SettingManager in ControllerBase Constructor #1962

2 Answer(s)

• 

  0

  sayram created 8 years ago

  I found a working "hack" to pass data to all views.

  I created a service named ISettingService and implemented here.

  public class SettingService : ApplicationService, ISettingService
      {
          public string GetTheme()
          {
              return SettingManager.GetSettingValue(IndexSettingProvider.CurrentTheme);
          }
      }
  

  I inject ISettingService in ControllerBase.

  public abstract class IndexControllerBase : AbpController
      {
          private readonly ISettingService _settingService;
  
          protected IndexControllerBase(ISettingService settingService)
          {
              _settingService = settingService;
  
              LocalizationSourceName = IndexConsts.LocalizationSourceName;
              ViewBag.Theme = _settingService.GetTheme();
          }
  }
  

  And in each controller i created their constructor

  public class HomeController : IndexControllerBase
      {
  
          public HomeController(ISettingService settingService): base(settingService)
          {
  
          }
  
          public ActionResult Index()
          {
              return View();
          }
  
  	}
  

  this way is works. But the ugly way i believe.

  Save Cancel
• 

  0

  manuel created 8 years ago

  Maybe too late, but if you want to improve your code you can:

  public IndexControllerBase : AbpController
  {
      protected override void Initialize(RequestContext requestContext)
      {
          // inject with IOC the service
         // happily use the service :)
          base.Initialize(requestContext);
      }
  }
  

  But, answering your question:

  Can't we use AbpController.SettingManager in Constructor of ControllerBase?

  The service needs to be resolved and injected in some way, be careful when using services in the constructor, the services that depend on session specifics could not work fine there.

  Save Cancel