Add permission support for menu items

src/OrchardCore.Modules/OrchardCore.AdminMenu/AdminNodes/LinkAdminNodeDriver.cs

@@ -1,17 +1,17 @@
-using OrchardCore.AdminMenu.Services;
 using OrchardCore.DisplayManagement.Handlers;
 using OrchardCore.DisplayManagement.Views;
 using OrchardCore.Navigation;
+using OrchardCore.Security.Permissions;
 
 namespace OrchardCore.AdminMenu.AdminNodes;
 
 public sealed class LinkAdminNodeDriver : DisplayDriver<MenuItem, LinkAdminNode>
 {
-    private readonly IAdminMenuPermissionService _adminMenuPermissionService;
+    private readonly IPermissionService permissionService;
 
-    public LinkAdminNodeDriver(IAdminMenuPermissionService adminMenuPermissionService)
+    public LinkAdminNodeDriver(IPermissionService permissionService)
     {
-        _adminMenuPermissionService = adminMenuPermissionService;
+        this.permissionService = permissionService;
     }
 
     public override Task<IDisplayResult> DisplayAsync(LinkAdminNode treeNode, BuildDisplayContext context)
@@ -31,7 +31,7 @@ public override IDisplayResult Edit(LinkAdminNode treeNode, BuildEditorContext c
             model.IconClass = treeNode.IconClass;
             model.Target = treeNode.Target;
 
-            var permissions = await _adminMenuPermissionService.GetPermissionsAsync();
+            var permissions = await permissionService.GetPermissionsAsync();
 
             var selectedPermissions = permissions.Where(p => treeNode.PermissionNames.Contains(p.Name));
 
@@ -69,7 +69,7 @@ await context.Updater.TryUpdateModelAsync(model, Prefix,
             ? []
             : model.SelectedPermissionNames.Split(',', StringSplitOptions.RemoveEmptyEntries);
 
-        var permissions = await _adminMenuPermissionService.GetPermissionsAsync();
+        var permissions = await permissionService.GetPermissionsAsync();
         treeNode.PermissionNames = permissions
             .Where(p => selectedPermissions.Contains(p.Name))
             .Select(p => p.Name).ToArray();

src/OrchardCore.Modules/OrchardCore.AdminMenu/AdminNodes/LinkAdminNodeNavigationBuilder.cs

@@ -4,22 +4,23 @@
 using OrchardCore.Admin;
 using OrchardCore.AdminMenu.Services;
 using OrchardCore.Navigation;
+using OrchardCore.Security.Permissions;
 
 namespace OrchardCore.AdminMenu.AdminNodes;
 
 public class LinkAdminNodeNavigationBuilder : IAdminNodeNavigationBuilder
 {
     private readonly ILogger _logger;
-    private readonly IAdminMenuPermissionService _adminMenuPermissionService;
+    private readonly IPermissionService _permissionService;
     private readonly AdminOptions _adminOptions;
 
 
     public LinkAdminNodeNavigationBuilder(
-        IAdminMenuPermissionService adminMenuPermissionService,
+        IPermissionService permissionService,
         IOptions<AdminOptions> adminOptions,
         ILogger<LinkAdminNodeNavigationBuilder> logger)
     {
-        _adminMenuPermissionService = adminMenuPermissionService;
+        _permissionService = permissionService;
         _adminOptions = adminOptions.Value;
         _logger = logger;
     }
@@ -61,7 +62,7 @@ public Task BuildNavigationAsync(MenuItem menuItem, NavigationBuilder builder, I
 
             if (node.PermissionNames.Length > 0)
             {
-                var permissions = await _adminMenuPermissionService.GetPermissionsAsync();
+                var permissions = await _permissionService.GetPermissionsAsync();
 
                 // Find the actual permissions and apply them to the menu.
                 var selectedPermissions = permissions.Where(p => node.PermissionNames.Contains(p.Name));

src/OrchardCore.Modules/OrchardCore.AdminMenu/AdminNodes/PlaceholderAdminNodeDriver.cs

@@ -1,17 +1,17 @@
-using OrchardCore.AdminMenu.Services;
 using OrchardCore.DisplayManagement.Handlers;
 using OrchardCore.DisplayManagement.Views;
 using OrchardCore.Navigation;
+using OrchardCore.Security.Permissions;
 
 namespace OrchardCore.AdminMenu.AdminNodes;
 
 public sealed class PlaceholderAdminNodeDriver : DisplayDriver<MenuItem, PlaceholderAdminNode>
 {
-    private readonly IAdminMenuPermissionService _adminMenuPermissionService;
+    private readonly IPermissionService _permissionService;
 
-    public PlaceholderAdminNodeDriver(IAdminMenuPermissionService adminMenuPermissionService)
+    public PlaceholderAdminNodeDriver(IPermissionService permissionService)
     {
-        _adminMenuPermissionService = adminMenuPermissionService;
+        _permissionService = permissionService;
     }
 
     public override Task<IDisplayResult> DisplayAsync(PlaceholderAdminNode treeNode, BuildDisplayContext context)
@@ -29,7 +29,7 @@ public override IDisplayResult Edit(PlaceholderAdminNode treeNode, BuildEditorCo
             model.LinkText = treeNode.LinkText;
             model.IconClass = treeNode.IconClass;
 
-            var permissions = await _adminMenuPermissionService.GetPermissionsAsync();
+            var permissions = await _permissionService.GetPermissionsAsync();
 
             var selectedPermissions = permissions.Where(p => treeNode.PermissionNames.Contains(p.Name));
 
@@ -60,7 +60,7 @@ await context.Updater.TryUpdateModelAsync(model, Prefix,
         treeNode.IconClass = model.IconClass;
 
         var selectedPermissions = (model.SelectedPermissionNames == null ? Array.Empty<string>() : model.SelectedPermissionNames.Split(',', StringSplitOptions.RemoveEmptyEntries));
-        var permissions = await _adminMenuPermissionService.GetPermissionsAsync();
+        var permissions = await _permissionService.GetPermissionsAsync();
         treeNode.PermissionNames = permissions
             .Where(p => selectedPermissions.Contains(p.Name))
             .Select(p => p.Name).ToArray();

src/OrchardCore.Modules/OrchardCore.AdminMenu/AdminNodes/PlaceholderAdminNodeNavigationBuilder.cs

@@ -2,17 +2,20 @@
 using Microsoft.Extensions.Logging;
 using OrchardCore.AdminMenu.Services;
 using OrchardCore.Navigation;
+using OrchardCore.Security.Permissions;
 
 namespace OrchardCore.AdminMenu.AdminNodes;
 
 public class PlaceholderAdminNodeNavigationBuilder : IAdminNodeNavigationBuilder
 {
     private readonly ILogger _logger;
-    private readonly IAdminMenuPermissionService _adminMenuPermissionService;
+    private readonly IPermissionService _permissionService;
 
-    public PlaceholderAdminNodeNavigationBuilder(IAdminMenuPermissionService adminMenuPermissionService, ILogger<PlaceholderAdminNodeNavigationBuilder> logger)
+    public PlaceholderAdminNodeNavigationBuilder(
+        IPermissionService permissionService,
+        ILogger<PlaceholderAdminNodeNavigationBuilder> logger)
     {
-        _adminMenuPermissionService = adminMenuPermissionService;
+        _permissionService = permissionService;
         _logger = logger;
     }
 
@@ -34,7 +37,7 @@ public Task BuildNavigationAsync(MenuItem menuItem, NavigationBuilder builder, I
 
             if (node.PermissionNames.Length > 0)
             {
-                var permissions = await _adminMenuPermissionService.GetPermissionsAsync();
+                var permissions = await _permissionService.GetPermissionsAsync();
                 // Find the actual permissions and apply them to the menu.
                 var selectedPermissions = permissions.Where(p => node.PermissionNames.Contains(p.Name));
                 itemBuilder.Permissions(selectedPermissions);

src/OrchardCore.Modules/OrchardCore.AdminMenu/Services/AdminMenuPermissionService.cs

@@ -2,6 +2,7 @@
 
 namespace OrchardCore.AdminMenu.Services;
 
+[Obsolete("This service is obsolete and will be removed in version 4. Instead, please use IPermissionService")]
 public sealed class AdminMenuPermissionService : IAdminMenuPermissionService
 {
     private readonly IEnumerable<IPermissionProvider> _permissionProviders;

src/OrchardCore.Modules/OrchardCore.AdminMenu/Startup.cs

@@ -17,7 +17,10 @@ public override void ConfigureServices(IServiceCollection services)
     {
         services.AddPermissionProvider<Permissions>();
         services.AddNavigationProvider<AdminMenu>();
+
+#pragma warning disable CS0618 // Type or member is obsolete
         services.AddScoped<IAdminMenuPermissionService, AdminMenuPermissionService>();
+#pragma warning restore CS0618 // Type or member is obsolete
 
         services.AddScoped<IAdminMenuService, AdminMenuService>();
         services.AddScoped<AdminMenuNavigationProvidersCoordinator>();

src/OrchardCore.Modules/OrchardCore.AdminMenu/Views/Items/LinkAdminNode.Fields.TreeEdit.cshtml

@@ -113,19 +113,19 @@
         <div class="mb-3">
             <div class="w-xl-50">
                 <input asp-for="SelectedPermissionNames" type="hidden" v-model="selectedNames" />
-                <vue-multiselect v-model="value" 
-                                :options="options" 
-                                track-by="name"
-                                label="displayText" 
-                                placeholder="@T["Type to search"]"
-                                @@select="onSelect"
-                                :searchable="true" 
-                                :close-on-select="true" 
-                                :reset-after="true"
-                                :show-labels="true" 
-                                :hide-selected="false"
-                                select-label="@T["Select"]" 
-                                deselect-label="@T["Remove"]">
+                <vue-multiselect v-model="value"
+                                 :options="options"
+                                 track-by="name"
+                                 label="displayText"
+                                 placeholder="@T["Type to search"]"
+                                 @@select="onSelect"
+                                 :searchable="true"
+                                 :close-on-select="true"
+                                 :reset-after="true"
+                                 :show-labels="true"
+                                 :hide-selected="false"
+                                 select-label="@T["Select"]"
+                                 deselect-label="@T["Remove"]">
                     <template slot="option" slot-scope="props">
                         <div v-cloak><span>{{ props.option.displayText }}</span></div>
                     </template>
@@ -137,7 +137,11 @@
             </div>
         </div>
     </div>
-    <script at="Foot" depends-on="admin-menu-permission-picker">initAdminMenuPermissionsPicker(document.querySelector("#PermissionPicker"))</script>
+    <script at="Foot" depends-on="admin-menu-permission-picker">
+        document.addEventListener('DOMContentLoaded', () => {
+            initAdminMenuPermissionsPicker(document.querySelector("#PermissionPicker"));
+        });
+    </script>
 </div>
 
 <script at="Foot">

src/OrchardCore.Modules/OrchardCore.AdminMenu/Views/Items/PlaceholderAdminNode.Fields.TreeEdit.cshtml

@@ -115,7 +115,11 @@
             </div>
         </div>
     </div>
-    <script at="Foot" depends-on="admin-menu-permission-picker">initAdminMenuPermissionsPicker(document.querySelector("#PermissionPicker"))</script>
+    <script at="Foot" depends-on="admin-menu-permission-picker">
+        document.addEventListener('DOMContentLoaded', () => {
+            initAdminMenuPermissionsPicker(document.querySelector("#PermissionPicker"))
+        });
+    </script>
 </div>
 
 <script at="Foot">

src/OrchardCore.Modules/OrchardCore.Menu/Assets.json

@@ -16,5 +16,17 @@
       "Assets/js/activate-links.js"
     ],
     "output": "wwwroot/Scripts/activate-links.js"
+  },
+  {
+    "inputs": [
+      "Assets/scss/menu-permission-picker.scss"
+    ],
+    "output": "wwwroot/Styles/menu-permission-picker.css"
+  },
+  {
+    "inputs": [
+      "Assets/js/menu-permission-picker.js"
+    ],
+    "output": "wwwroot/Scripts/menu-permission-picker.js"
   }
 ]

src/OrchardCore.Modules/OrchardCore.Menu/Assets/js/menu-permission-picker.js

@@ -0,0 +1,45 @@
+function initMenuPermissionsPicker(element) {
+    // only run script if element exists
+    if (element) {
+        var elementId = element.id;
+        var selectedItems = JSON.parse(element.dataset.selectedItems || "[]");
+        var allItems = JSON.parse(element.dataset.allItems || "[]");
+
+        var vueMultiselect = Vue.component('vue-multiselect', window.VueMultiselect.default);
+
+        var vm = new Vue({
+            el: '#' + elementId,
+            components: { 'vue-multiselect': vueMultiselect },
+            data: {
+                value: null,
+                arrayOfItems: selectedItems,
+                options: allItems,
+            },
+            computed: {
+                selectedNames: function () {
+                    return this.arrayOfItems.map(function (x) { return x.name }).join(',');
+                }
+            },
+            methods: {
+                onSelect: function (selectedOption, name) {
+                    var self = this;
+
+                    for (i = 0; i < self.arrayOfItems.length; i++) {
+                        if (self.arrayOfItems[i].name === selectedOption.name) {
+                            return;
+                        }
+                    }
+
+                    self.arrayOfItems.push(selectedOption);
+                },
+                remove: function (item) {
+                    this.arrayOfItems.splice(this.arrayOfItems.indexOf(item), 1)
+                }
+            }
+        })
+
+        /*Hook for other scripts that might want to have access to the view model*/
+        var event = new CustomEvent("menu-permission-picker-created", { detail: { vm: vm } });
+        document.querySelector("body").dispatchEvent(event);
+    }
+}

src/OrchardCore.Modules/OrchardCore.Menu/Assets/scss/menu-permission-picker.scss

@@ -0,0 +1,7 @@
+.vue-multiselect .multiselect--active {
+    z-index: var(--bs-dropdown-zindex, 1000);
+}
+
+[v-cloak] {
+    display: none;
+}

src/OrchardCore.Modules/OrchardCore.Menu/Drivers/MenuItemPermissionPartDisplayDriver.cs

@@ -0,0 +1,60 @@
+using OrchardCore.ContentManagement.Display.ContentDisplay;
+using OrchardCore.ContentManagement.Display.Models;
+using OrchardCore.DisplayManagement.Views;
+using OrchardCore.Menu.Models;
+using OrchardCore.Menu.ViewModels;
+using OrchardCore.Security.Permissions;
+
+namespace OrchardCore.Menu.Drivers;
+
+public sealed class MenuItemPermissionPartDisplayDriver : ContentPartDisplayDriver<MenuItemPermissionPart>
+{
+    private readonly IPermissionService _permissionService;
+
+    public MenuItemPermissionPartDisplayDriver(IPermissionService permissionService)
+    {
+        _permissionService = permissionService;
+    }
+
+    public override IDisplayResult Edit(MenuItemPermissionPart part, BuildPartEditorContext context)
+    {
+        return Initialize<MenuItemPermissionViewModel>("MenuItemPermissionPart_Edit", async model =>
+        {
+            var permissions = await _permissionService.GetPermissionsAsync();
+
+            var selectedPermissions = permissions.Where(p => part.PermissionNames.Contains(p.Name));
+
+            model.SelectedItems = selectedPermissions
+                .Select(p => new PermissionViewModel
+                {
+                    Name = p.Name,
+                    DisplayText = p.Description
+                }).ToArray();
+
+            model.AllItems = permissions
+                .Select(p => new PermissionViewModel
+                {
+                    Name = p.Name,
+                    DisplayText = p.Description
+                }).ToArray();
+        });
+    }
+
+    public override async Task<IDisplayResult> UpdateAsync(MenuItemPermissionPart part, UpdatePartEditorContext context)
+    {
+        var model = new MenuItemPermissionViewModel();
+        await context.Updater.TryUpdateModelAsync(model, Prefix,
+            x => x.SelectedPermissionNames);
+
+        var selectedPermissions = model.SelectedPermissionNames == null
+            ? []
+            : model.SelectedPermissionNames.Split(',', StringSplitOptions.RemoveEmptyEntries);
+
+        var permissions = await _permissionService.GetPermissionsAsync();
+        part.PermissionNames = permissions
+            .Where(p => selectedPermissions.Contains(p.Name))
+            .Select(p => p.Name).ToArray();
+
+        return Edit(part, context);
+    }
+}