From 1ac196427c4cbf8a5d36bba0861ab1ab053ffa38 Mon Sep 17 00:00:00 2001
From: liangshiwei <liangshiw@outlook.com>
Date: Tue, 2 Mar 2021 14:46:30 +0800
Subject: [PATCH] Use RequiredPermissionName for all ApplicationMenuItem of
 Blazor

---
 .../AbpIdentityWebMainMenuContributor.cs      | 38 +++++++++----------
 .../TenantManagementBlazorMenuContributor.cs  | 18 +++++----
 2 files changed, 30 insertions(+), 26 deletions(-)

diff --git a/modules/identity/src/Volo.Abp.Identity.Blazor/AbpIdentityWebMainMenuContributor.cs b/modules/identity/src/Volo.Abp.Identity.Blazor/AbpIdentityWebMainMenuContributor.cs
index 5e5714a851..609da7e6a7 100644
--- a/modules/identity/src/Volo.Abp.Identity.Blazor/AbpIdentityWebMainMenuContributor.cs
+++ b/modules/identity/src/Volo.Abp.Identity.Blazor/AbpIdentityWebMainMenuContributor.cs
@@ -6,35 +6,35 @@ namespace Volo.Abp.Identity.Blazor
 {
     public class AbpIdentityWebMainMenuContributor : IMenuContributor
     {
-        public virtual async Task ConfigureMenuAsync(MenuConfigurationContext context)
+        public virtual Task ConfigureMenuAsync(MenuConfigurationContext context)
         {
             if (context.Menu.Name != StandardMenus.Main)
             {
-                return;
+                return Task.CompletedTask;
             }
 
-            var hasRolePermission = await context.IsGrantedAsync(IdentityPermissions.Roles.Default);
-            var hasUserPermission = await context.IsGrantedAsync(IdentityPermissions.Users.Default);
+            var administrationMenu = context.Menu.GetAdministration();
 
-            if (hasRolePermission || hasUserPermission)
-            {
-                var administrationMenu = context.Menu.GetAdministration();
+            var l = context.GetLocalizer<IdentityResource>();
 
-                var l = context.GetLocalizer<IdentityResource>();
+            var identityMenuItem = new ApplicationMenuItem(IdentityMenuNames.GroupName, l["Menu:IdentityManagement"],
+                icon: "far fa-id-card");
+            administrationMenu.AddItem(identityMenuItem);
 
-                var identityMenuItem = new ApplicationMenuItem(IdentityMenuNames.GroupName, l["Menu:IdentityManagement"], icon: "far fa-id-card");
-                administrationMenu.AddItem(identityMenuItem);
+            identityMenuItem.AddItem(new ApplicationMenuItem(
+                    IdentityMenuNames.Roles,
+                    l["Roles"],
+                    url: "identity/roles",
+                    requiredPermissionName: IdentityPermissions.Roles.Default));
 
-                if (hasRolePermission)
-                {
-                    identityMenuItem.AddItem(new ApplicationMenuItem(IdentityMenuNames.Roles, l["Roles"], url: "identity/roles"));
-                }
 
-                if (hasUserPermission)
-                {
-                    identityMenuItem.AddItem(new ApplicationMenuItem(IdentityMenuNames.Users, l["Users"], url: "identity/users"));
-                }
-            }
+            identityMenuItem.AddItem(new ApplicationMenuItem(
+                IdentityMenuNames.Users,
+                l["Users"],
+                url: "identity/users",
+                requiredPermissionName: IdentityPermissions.Users.Default));
+
+            return Task.CompletedTask;
         }
     }
 }
diff --git a/modules/tenant-management/src/Volo.Abp.TenantManagement.Blazor/Navigation/TenantManagementBlazorMenuContributor.cs b/modules/tenant-management/src/Volo.Abp.TenantManagement.Blazor/Navigation/TenantManagementBlazorMenuContributor.cs
index 03bfd6b2d2..32cac00b2c 100644
--- a/modules/tenant-management/src/Volo.Abp.TenantManagement.Blazor/Navigation/TenantManagementBlazorMenuContributor.cs
+++ b/modules/tenant-management/src/Volo.Abp.TenantManagement.Blazor/Navigation/TenantManagementBlazorMenuContributor.cs
@@ -6,24 +6,28 @@ namespace Volo.Abp.TenantManagement.Blazor.Navigation
 {
     public class TenantManagementBlazorMenuContributor : IMenuContributor
     {
-        public virtual async Task ConfigureMenuAsync(MenuConfigurationContext context)
+        public virtual Task ConfigureMenuAsync(MenuConfigurationContext context)
         {
             if (context.Menu.Name != StandardMenus.Main)
             {
-                return;
+                return Task.CompletedTask;
             }
 
             var administrationMenu = context.Menu.GetAdministration();
 
             var l = context.GetLocalizer<AbpTenantManagementResource>();
 
-            var tenantManagementMenuItem = new ApplicationMenuItem(TenantManagementMenuNames.GroupName, l["Menu:TenantManagement"], icon: "fa fa-users");
+            var tenantManagementMenuItem = new ApplicationMenuItem(TenantManagementMenuNames.GroupName,
+                l["Menu:TenantManagement"], icon: "fa fa-users");
             administrationMenu.AddItem(tenantManagementMenuItem);
 
-            if (await context.IsGrantedAsync(TenantManagementPermissions.Tenants.Default))
-            {
-                tenantManagementMenuItem.AddItem(new ApplicationMenuItem(TenantManagementMenuNames.Tenants, l["Tenants"], url: "tenant-management/tenants"));
-            }
+            tenantManagementMenuItem.AddItem(new ApplicationMenuItem(
+                TenantManagementMenuNames.Tenants,
+                l["Tenants"],
+                url: "tenant-management/tenants",
+                requiredPermissionName: TenantManagementPermissions.Tenants.Default));
+
+            return Task.CompletedTask;
         }
     }
 }