From 5bbcbb85b64091197db983b5f9ce56dd78c3c951 Mon Sep 17 00:00:00 2001
From: Sebastien Ros <none@none>
Date: Thu, 21 Jun 2012 10:57:46 -0700
Subject: [PATCH] Removing unnecessary code from MessageSettingsBanner
 constructor

--HG--
branch : 1.x
---
 .../Services/MissingSettingsBanner.cs         | 27 +++++++------------
 1 file changed, 10 insertions(+), 17 deletions(-)

diff --git a/src/Orchard.Web/Modules/Orchard.Email/Services/MissingSettingsBanner.cs b/src/Orchard.Web/Modules/Orchard.Email/Services/MissingSettingsBanner.cs
index f2d639cb1..0e95da148 100644
--- a/src/Orchard.Web/Modules/Orchard.Email/Services/MissingSettingsBanner.cs
+++ b/src/Orchard.Web/Modules/Orchard.Email/Services/MissingSettingsBanner.cs
@@ -6,32 +6,25 @@ using Orchard.Localization;
 using Orchard.UI.Admin.Notification;
 using Orchard.UI.Notify;
 
-namespace Orchard.Email.Services
-{
-    public class MissingSettingsBanner : INotificationProvider
-    {
+namespace Orchard.Email.Services {
+    public class MissingSettingsBanner : INotificationProvider {
         private readonly IOrchardServices _orchardServices;
-        private readonly WorkContext _workContext;
 
-        public MissingSettingsBanner(IOrchardServices orchardServices, IWorkContextAccessor workContextAccessor)
-        {
+        public MissingSettingsBanner(IOrchardServices orchardServices) {
             _orchardServices = orchardServices;
-            _workContext = workContextAccessor.GetContext();
             T = NullLocalizer.Instance;
         }
 
         public Localizer T { get; set; }
 
-        public IEnumerable<NotifyEntry> GetNotifications()
-        {
+        public IEnumerable<NotifyEntry> GetNotifications() {
+            var workContext = _orchardServices.WorkContext;
+            var smtpSettings = workContext.CurrentSite.As<SmtpSettingsPart>();
 
-            var smtpSettings = _orchardServices.WorkContext.CurrentSite.As<SmtpSettingsPart>();
-
-            if (smtpSettings == null || !smtpSettings.IsValid())
-            {
-                var urlHelper = new UrlHelper(_workContext.HttpContext.Request.RequestContext);
-                var url = urlHelper.Action("Email", "Admin", new { Area = "Settings" });
-                yield return new NotifyEntry { Message = T("The <a href=\"{0}\">SMTP settings</a> needs to be configured.", url), Type = NotifyType.Warning };
+            if (smtpSettings == null || !smtpSettings.IsValid()) {
+                var urlHelper = new UrlHelper(workContext.HttpContext.Request.RequestContext);
+                var url = urlHelper.Action("Email", "Admin", new {Area = "Settings"});
+                yield return new NotifyEntry {Message = T("The <a href=\"{0}\">SMTP settings</a> needs to be configured.", url), Type = NotifyType.Warning};
             }
         }
     }