Merge

--HG--
branch : dev

src/Orchard.Web/Core/Dashboard/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Dashboard:
         Description: Standard admin dashboard.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Core/Feeds/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Feeds:
         Description: RSS feeds for content items.
-        Dependencies: Common
         Category: Syndication

src/Orchard.Web/Core/HomePage/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     HomePage:
         Description: Standard site home page that allows a specified content type or container to *be* the home page.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Core/Navigation/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Navigation:
         Description: Menu management.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Core/Scheduling/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Scheduling:
         Description: Scheduled background tasks.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Core/Settings/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Settings:
         Description: Site settings.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Core/XmlRpc/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     XmlRpc:
         Description: XML-RPC opt-in implementation.
-        Dependencies: Common
-        Category: Core
+        Category: Content Management

src/Orchard.Web/Modules/Futures.Widgets/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Futures.Widgets:
         Description: Widgets container with simple inline content editing widget.
-        Dependencies: Common
         Category: Widget

src/Orchard.Web/Modules/Orchard.Blogs/Module.txt

@@ -7,5 +7,5 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Blogs:
         Description: A simple web log.
-        Dependencies: Common, XmlRpc
+        Dependencies: XmlRpc
         Category: Content

src/Orchard.Web/Modules/Orchard.Comments/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Comments:
         Description: Standard content item comments.
-        Dependencies: Common
         Category: Social

src/Orchard.Web/Modules/Orchard.DevTools/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.DevTools:
         Description: An assortment of debuging tools.
-        Dependencies: Common
         Category: Developer

src/Orchard.Web/Modules/Orchard.Media/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Media:
         Description: File system based media upload, storage and management.
-        Dependencies: Common
         Category: Media

src/Orchard.Web/Modules/Orchard.Modules/Controllers/AdminController.cs

@@ -1,11 +1,8 @@
 using System.Linq;
-using System.Reflection;
 using System.Web.Mvc;
 using Orchard.Localization;
 using Orchard.Modules.ViewModels;
-using Orchard.Mvc.AntiForgery;
 using Orchard.Mvc.Results;
-using Orchard.UI.Notify;
 
 namespace Orchard.Modules.Controllers {
     public class AdminController : Controller {
@@ -37,9 +34,7 @@ namespace Orchard.Modules.Controllers {
             if (module == null)
                 return new NotFoundResult();
 
-            return View(new ModuleEditViewModel {
-                                               Name = module.DisplayName
-                                           });
+            return View(new ModuleEditViewModel {Name = module.DisplayName});
         }
 
         public ActionResult Features() {
@@ -50,43 +45,30 @@ namespace Orchard.Modules.Controllers {
             return View(new FeaturesViewModel {Features = features});
         }
 
-        [ValidateAntiForgeryTokenOrchard]
-        public ActionResult Enable(string id) {
+        [HttpPost]
+        public ActionResult Enable(string id, bool? force) {
             if (!Services.Authorizer.Authorize(Permissions.ManageFeatures, T("Not allowed to manage features")))
                 return new HttpUnauthorizedResult();
 
             if (string.IsNullOrEmpty(id))
                 return new NotFoundResult();
 
-            _moduleService.EnableFeatures(new [] {id});
+            _moduleService.EnableFeatures(new[] {id}, force != null && (bool) force);
 
             return RedirectToAction("Features");
         }
 
-        [ValidateAntiForgeryTokenOrchard]
-        public ActionResult Disable(string id) {
+        [HttpPost]
+        public ActionResult Disable(string id, bool? force) {
             if (!Services.Authorizer.Authorize(Permissions.ManageFeatures, T("Not allowed to manage features")))
                 return new HttpUnauthorizedResult();
 
             if (string.IsNullOrEmpty(id))
                 return new NotFoundResult();
 
-            _moduleService.DisableFeatures(new[] { id });
+            _moduleService.DisableFeatures(new[] {id}, force != null && (bool) force);
 
             return RedirectToAction("Features");
         }
-
-        private class FormValueRequiredAttribute : ActionMethodSelectorAttribute {
-            private readonly string _submitButtonName;
-
-            public FormValueRequiredAttribute(string submitButtonName) {
-                _submitButtonName = submitButtonName;
-            }
-
-            public override bool IsValidForRequest(ControllerContext controllerContext, MethodInfo methodInfo) {
-                var value = controllerContext.HttpContext.Request.Form[_submitButtonName];
-                return !string.IsNullOrEmpty(value);
-            }
-        }
     }
 }

src/Orchard.Web/Modules/Orchard.Modules/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Modules:
         Description: Standard module and feature management.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Modules/Orchard.Modules/Services/ModuleService.cs

@@ -16,7 +16,8 @@ namespace Orchard.Modules.Services {
         private readonly IExtensionManager _extensionManager;
         private readonly IShellDescriptorManager _shellDescriptorManager;
 
-        public ModuleService(IOrchardServices orchardServices,IExtensionManager extensionManager, IShellDescriptorManager shellDescriptorManager) {
+        public ModuleService(IOrchardServices orchardServices, IExtensionManager extensionManager,
+                             IShellDescriptorManager shellDescriptorManager) {
             Services = orchardServices;
             _extensionManager = extensionManager;
             _shellDescriptorManager = shellDescriptorManager;
@@ -27,14 +28,19 @@ namespace Orchard.Modules.Services {
         public IOrchardServices Services { get; set; }
 
         public IModule GetModuleByName(string moduleName) {
-            return _extensionManager.AvailableExtensions().Where(e => string.Equals(e.Name, moduleName, StringComparison.OrdinalIgnoreCase) && string.Equals(e.ExtensionType, ModuleExtensionType, StringComparison.OrdinalIgnoreCase)).Select(
+            return
+                _extensionManager.AvailableExtensions().Where(
+                    e =>
+                    string.Equals(e.Name, moduleName, StringComparison.OrdinalIgnoreCase) &&
+                    string.Equals(e.ExtensionType, ModuleExtensionType, StringComparison.OrdinalIgnoreCase)).Select(
                         descriptor => AssembleModuleFromDescriptor(descriptor)).FirstOrDefault();
         }
 
         public IEnumerable<IModule> GetInstalledModules() {
             return
                 _extensionManager.AvailableExtensions().Where(
-                    e => String.Equals(e.ExtensionType, ModuleExtensionType, StringComparison.OrdinalIgnoreCase)).Select(
+                    e => String.Equals(e.ExtensionType, ModuleExtensionType, StringComparison.OrdinalIgnoreCase)).Select
+                    (
                         descriptor => AssembleModuleFromDescriptor(descriptor));
         }
 
@@ -46,86 +52,145 @@ namespace Orchard.Modules.Services {
             _extensionManager.UninstallExtension(ModuleExtensionType, moduleName);
         }
 
-        public IModule GetModuleByFeatureName(string featureName) {
-            return GetInstalledModules()
-                .Where(
-                m =>
-                m.Features.FirstOrDefault(f => string.Equals(f.Name, featureName, StringComparison.OrdinalIgnoreCase)) !=
-                null).FirstOrDefault();
-        }
-
         public IEnumerable<IModuleFeature> GetAvailableFeatures() {
             var enabledFeatures = _shellDescriptorManager.GetShellDescriptor().EnabledFeatures;
             return GetInstalledModules()
                 .SelectMany(m => _extensionManager.LoadFeatures(m.Features))
-                .Select(f => AssembleModuleFromDescriptor(f, enabledFeatures.FirstOrDefault(sf => string.Equals(sf.Name, f.Descriptor.Name, StringComparison.OrdinalIgnoreCase)) != null));
-        }
-
-        public IEnumerable<Feature> GetAvailableFeaturesByModule(string moduleName) {
-            throw new NotImplementedException();
+                .Select(
+                    f =>
+                    AssembleModuleFromDescriptor(f,
+                                                 enabledFeatures.FirstOrDefault(
+                                                     sf =>
+                                                     string.Equals(sf.Name, f.Descriptor.Name,
+                                                                   StringComparison.OrdinalIgnoreCase)) != null));
         }
 
         public void EnableFeatures(IEnumerable<string> featureNames) {
+            EnableFeatures(featureNames, false);
+        }
+
+        public void EnableFeatures(IEnumerable<string> features, bool force) {
             var shellDescriptor = _shellDescriptorManager.GetShellDescriptor();
             var enabledFeatures = shellDescriptor.EnabledFeatures.ToList();
-            var features = GetAvailableFeatures().ToList();
 
-            foreach (var name in featureNames) {
-                var featureName = name;
-                var feature = features.Single(f => f.Descriptor.Name == featureName);
-                var sleepingDependencies =
-                    feature.Descriptor.Dependencies.Where(s => enabledFeatures.FirstOrDefault(sf => sf.Name == s) == null);
+            var featuresToEnable =
+                features.Select(s => EnableFeature(s, GetAvailableFeatures(), force)).
+                    SelectMany(ies => ies.Select(s => s));
 
-                if (sleepingDependencies.Count() != 0) {
-                    Services.Notifier.Warning(T(
-                        "If you want to enable {0}, then you'll also need {1} (and I won't let you flip everything on in one go yet).",
-                        featureName,
-                        sleepingDependencies.Count() > 1
-                            ? string.Join("",
-                                          sleepingDependencies.Select(
-                                              (s, i) =>
-                                              i == sleepingDependencies.Count() - 2
-                                                  ? T("{0} and ", s).ToString()
-                                                  : T("{0}, ", s).ToString()).ToArray())
-                            : sleepingDependencies.First()));
-                } else if (enabledFeatures.FirstOrDefault(f => f.Name == featureName) == null) {
-                    enabledFeatures.Add(new ShellFeature {Name = featureName});
-                    Services.Notifier.Information(T("{0} was enabled", featureName));
-                }
+            if (featuresToEnable.Count() == 0)
+                return;
+
+            foreach (var featureToEnable in featuresToEnable) {
+                enabledFeatures.Add(new ShellFeature {Name = featureToEnable});
+                Services.Notifier.Information(T("{0} was enabled", featureToEnable));
             }
 
-            _shellDescriptorManager.UpdateShellDescriptor(shellDescriptor.SerialNumber, enabledFeatures, shellDescriptor.Parameters);
+            _shellDescriptorManager.UpdateShellDescriptor(shellDescriptor.SerialNumber, enabledFeatures,
+                                                          shellDescriptor.Parameters);
         }
 
         public void DisableFeatures(IEnumerable<string> featureNames) {
+            DisableFeatures(featureNames, false);
+        }
+
+        public void DisableFeatures(IEnumerable<string> features, bool force) {
             var shellDescriptor = _shellDescriptorManager.GetShellDescriptor();
             var enabledFeatures = shellDescriptor.EnabledFeatures.ToList();
-            var features = GetAvailableFeatures().ToList();
 
-            foreach (var name in featureNames) {
-                var featureName = name;
-                var dependants = features.Where(f => f.IsEnabled && f.Descriptor.Dependencies != null && f.Descriptor.Dependencies.Contains(featureName));
+            var featuresToDisable =
+                features.Select(s => DisableFeature(s, GetAvailableFeatures(), force)).SelectMany(
+                    ies => ies.Select(s => s));
 
-                if (dependants.Count() != 0) {
-                    Services.Notifier.Warning(T(
-                        "If you want to disable {0}, then you'll also lose {1} (and I won't let you do that yet).",
+            if (featuresToDisable.Count() == 0)
+                return;
+
+            foreach (var featureToDisable in featuresToDisable) {
+                var feature = featureToDisable;
+                enabledFeatures.RemoveAll(f => f.Name == feature);
+                Services.Notifier.Information(T("{0} was disabled", feature));
+            }
+
+            _shellDescriptorManager.UpdateShellDescriptor(shellDescriptor.SerialNumber, enabledFeatures,
+                                                          shellDescriptor.Parameters);
+        }
+
+        public IModule GetModuleByFeatureName(string featureName) {
+            return GetInstalledModules()
+                .Where(
+                    m =>
+                    m.Features.FirstOrDefault(
+                        f => string.Equals(f.Name, featureName, StringComparison.OrdinalIgnoreCase)) !=
+                    null).FirstOrDefault();
+        }
+
+        private IEnumerable<string> EnableFeature(string featureName, IEnumerable<IModuleFeature> features, bool force) {
+            var featuresList = features.ToList();
+            var getDisabledDependencies =
+                new Func<string, IEnumerable<IModuleFeature>, IEnumerable<IModuleFeature>>(
+                    (n, fs) => {
+                        var feature = fs.Single(f => f.Descriptor.Name == n);
+                        return feature.Descriptor.Dependencies != null
+                                   ? feature.Descriptor.Dependencies.Select(
+                                       fn => fs.Single(f => f.Descriptor.Name == fn)).Where(f => !f.IsEnabled)
+                                   : Enumerable.Empty<IModuleFeature>();
+                    });
+
+            var featuresToEnable = GetAffectedFeatures(featureName, featuresList, getDisabledDependencies);
+
+            if (featuresToEnable.Count() > 1 && !force) {
+                GenerateWarning("If you want {0} enabled, then you'll also need to enable {1}.",
                                 featureName,
-                        dependants.Count() > 0
-                            ? string.Join("",
-                                          dependants.Select(
-                                              (f, i) =>
-                                              i == dependants.Count() - 2
-                                                  ? T("{0} and ", f.Descriptor.Name).ToString()
-                                                  : T("{0}, ", f.Descriptor.Name).ToString()).ToArray())
-                            : dependants.First().Descriptor.Name));
-                }
-                else {
-                    enabledFeatures.RemoveAll(f => f.Name == featureName);
-                    Services.Notifier.Information(T("{0} was disabled", featureName));
-                }
+                                featuresToEnable.Where(fn => fn != featureName));
+                return Enumerable.Empty<string>();
             }
 
-            _shellDescriptorManager.UpdateShellDescriptor(shellDescriptor.SerialNumber, enabledFeatures, shellDescriptor.Parameters);
+            return featuresToEnable;
+        }
+
+        private IEnumerable<string> DisableFeature(string featureName, IEnumerable<IModuleFeature> features, bool force) {
+            var featuresList = features.ToList();
+            var getEnabledDependants =
+                new Func<string, IEnumerable<IModuleFeature>, IEnumerable<IModuleFeature>>(
+                    (n, fs) => fs.Where(f => f.IsEnabled && f.Descriptor.Dependencies != null && f.Descriptor.Dependencies.Contains(n)));
+
+            var featuresToDisable = GetAffectedFeatures(featureName, featuresList, getEnabledDependants);
+
+            if (featuresToDisable.Count() > 1 && !force) {
+                GenerateWarning("If {0} is disabled, then you'll also lose {1}.",
+                                featureName,
+                                featuresToDisable.Where(fn => fn != featureName));
+                return Enumerable.Empty<string>();
+            }
+
+            return featuresToDisable;
+        }
+
+        private static IEnumerable<string> GetAffectedFeatures(string featureName, IEnumerable<IModuleFeature> features, Func<string, IEnumerable<IModuleFeature>, IEnumerable<IModuleFeature>> getAffectedDependencies) {
+            var dependencies = new List<string> {featureName};
+
+            foreach (var dependency in getAffectedDependencies(featureName, features))
+                dependencies.AddRange(GetAffectedFeatures(dependency.Descriptor.Name, features, getAffectedDependencies));
+
+            return dependencies;
+        }
+
+        private void GenerateWarning(string messageFormat, string featureName, IEnumerable<string> featuresInQuestion) {
+            if (featuresInQuestion.Count() < 1)
+                return;
+
+            Services.Notifier.Warning(T(
+                messageFormat,
+                featureName,
+                featuresInQuestion.Count() > 1
+                    ? string.Join("",
+                                  featuresInQuestion.Select(
+                                      (fn, i) =>
+                                      T(i == featuresInQuestion.Count() - 1
+                                            ? "{0}"
+                                            : (i == featuresInQuestion.Count() - 2
+                                                   ? "{0} and "
+                                                   : "{0}, "), fn).ToString()).ToArray())
+                    : featuresInQuestion.First()));
         }
 
         private static IModule AssembleModuleFromDescriptor(ExtensionDescriptor extensionDescriptor) {

src/Orchard.Web/Modules/Orchard.Modules/Views/Admin/Features.ascx

@@ -13,7 +13,10 @@
         if (featureGroup == featureGroups.First())
             categoryClassName += " first";
         if (featureGroup == featureGroups.Last())
-            categoryClassName += " last"; %>
+            categoryClassName += " last";
+        
+        //temporarily "disable" actions on core features
+        var showActions = categoryName.ToString() != "Core"; %>
     <li class="<%=categoryClassName %>">
         <h2><%=Html.Encode(categoryName) %></h2>
         <ul><%
@@ -42,7 +45,8 @@
                                 "") %>
                         </div><%
                         } %>
-                    </div>
+                    </div><%
+                    if (showActions) { %>
                     <div class="actions"><%
                         if (feature.IsEnabled) {
                         using (Html.BeginFormAntiForgeryPost(string.Format("{0}", Url.Action("Disable", new { area = "Orchard.Modules" })), FormMethod.Post, new {@class = "inline link"})) { %>
@@ -56,6 +60,23 @@
                         }
                         } %>
                     </div>
+                    <div class="cathedral"><%
+                        //temporary forceful feature actions
+                        if (feature.IsEnabled) {
+                        using (Html.BeginFormAntiForgeryPost(string.Format("{0}", Url.Action("Disable", new { area = "Orchard.Modules" })), FormMethod.Post, new {@class = "inline link"})) { %>
+                            <%=Html.Hidden("id", feature.Descriptor.Name, new { id = "" })%>
+                            <%=Html.Hidden("force", true)%>
+                            <button type="submit"><%=_Encoded("π")%></button><%
+                        }
+                        } else {
+                        using (Html.BeginFormAntiForgeryPost(string.Format("{0}", Url.Action("Enable", new { area = "Orchard.Modules" })), FormMethod.Post, new {@class = "inline link"})) { %>
+                            <%=Html.Hidden("id", feature.Descriptor.Name, new { id = "" })%>
+                            <%=Html.Hidden("force", true)%>
+                            <button type="submit"><%=_Encoded("π")%></button><%
+                        }
+                        } %>
+                    </div><%
+                    } %>
                 </div>
             </li><%
             } %>

src/Orchard.Web/Modules/Orchard.Modules/styles/admin.css

@@ -1,11 +1,16 @@
-.features .category {
+.orchard-modules #main h2 {
+    border:0;
+    margin-bottom:.2em;
+}
+
+.features .category {
     overflow:hidden;
 }
 .features .feature {
     border:1px solid #EAEAEA;
     display:block;
     float:left;
-    height:6.8em;
+    height:5em;
     margin:0 .5% 1% .5%;
     position:relative;
     width:32.1%;
@@ -28,8 +33,8 @@
 }
 
 .features .enabled.feature {
-    background:#D1F2A5;
-    border-color:#BCD994;
+    background:#FFF;
+    border-color:#CFE493;cfe493
 }
 .features .disabled.feature {
     background:#EAEAEA;
@@ -40,7 +45,7 @@
     padding:.4em .5em;
 }
 .features .dependencies li,
-.features .actions a {
+.features .actions {
     font-size:1.4em;
 }
 .features .dependencies {
@@ -52,7 +57,7 @@
 }
 .features .dependencies li {
     display:inline;
-    margin-left:.5ex;
+    margin-left:.5em;
 }
 .features .dependencies li::after {
     content:", ";
@@ -62,6 +67,19 @@
 }
 .features .feature .actions {
     position:absolute;
-    right:.8em;
+    right:.4em;
     top:.6em;
 }
+
+.cathedral {
+    bottom:0;
+    font-size:.8em;
+    position:absolute;
+    right:3px;
+}
+.cathedral a,
+.cathedral a:link,
+.cathedral a:visited,
+.cathedral form.inline.link button {
+    color:#aeaeae;
+}

src/Orchard.Web/Modules/Orchard.MultiTenancy/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.MultiTenancy:
         Description: Configure multiple site tenants.
-        Dependencies: Common
-        Category: Core
+        Category: Hosting

src/Orchard.Web/Modules/Orchard.Pages/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Pages:
         Description: Simple pages.
-        Dependencies: Common
         Category: Content

src/Orchard.Web/Modules/Orchard.Roles/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Roles:
         Description: Standard user roles.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Modules/Orchard.Sandbox/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Sandbox:
         Description: A module to mess around with. Currently wiki-like.
-        Dependencies: Common
         Category: Developer

src/Orchard.Web/Modules/Orchard.Setup/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Setup:
         Description: Standard site setup.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Modules/Orchard.Tags/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Tags:
         Description: Tag a content item.
-        Dependencies: Common
         Category: Navigation

src/Orchard.Web/Modules/Orchard.Themes/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Themes:
         Description: Basic theming capability.
-        Dependencies: Common
         Category: Display

src/Orchard.Web/Modules/Orchard.Users/Module.txt

@@ -7,5 +7,4 @@ orchardversion: 0.1.2010.0312
 features:
     Orchard.Users:
         Description: Standard users.
-        Dependencies: Common
         Category: Core

src/Orchard.Web/Themes/TheAdmin/Styles/site.css

@@ -337,8 +337,8 @@ span.message {
 	color:#fff;
 }
 .info.message {
-    background:#D1F2A5; /* green*/
-	border:1px solid #BCD994;
+    background:#e6f1c9; /* green */
+	border:1px solid #cfe493;
 	color:#062232;
 }
 .debug.message {

src/Orchard/Modules/IModuleService.cs

@@ -1,6 +1,5 @@
 using System.Collections.Generic;
 using System.Web;
-using Orchard.Environment.Extensions.Models;
 
 namespace Orchard.Modules {
     public interface IModuleService : IDependency {
@@ -8,9 +7,10 @@ namespace Orchard.Modules {
         IEnumerable<IModule> GetInstalledModules();
         void InstallModule(HttpPostedFileBase file);
         void UninstallModule(string moduleName);
-        IModule GetModuleByFeatureName(string featureName);
         IEnumerable<IModuleFeature> GetAvailableFeatures();
         void EnableFeatures(IEnumerable<string> featureNames);
+        void EnableFeatures(IEnumerable<string> featureNames, bool force);
         void DisableFeatures(IEnumerable<string> featureNames);
+        void DisableFeatures(IEnumerable<string> featureNames, bool force);
     }
 }