Rename ManagementPermission to ManagementPermissionName

maliming · maliming · commit 5579fb7c543b · 2025-12-06T12:25:11.000+08:00
diff --git a/framework/src/Volo.Abp.Authorization.Abstractions/Volo/Abp/Authorization/Permissions/IPermissionDefinitionContext.cs b/framework/src/Volo.Abp.Authorization.Abstractions/Volo/Abp/Authorization/Permissions/IPermissionDefinitionContext.cs
@@ -52,7 +52,7 @@ PermissionGroupDefinition AddGroup(
     PermissionDefinition AddResourcePermission(
         string name,
         string resourceName,
-        string managementPermission,
+        string managementPermissionName,
         ILocalizableString? displayName = null,
         MultiTenancySides multiTenancySide = MultiTenancySides.Both,
         bool isEnabled = true);
diff --git a/framework/src/Volo.Abp.Authorization.Abstractions/Volo/Abp/Authorization/Permissions/PermissionDefinition.cs b/framework/src/Volo.Abp.Authorization.Abstractions/Volo/Abp/Authorization/Permissions/PermissionDefinition.cs
@@ -24,7 +24,7 @@ public class PermissionDefinition :
     /// <summary>
     ///  Management permission of the resource permission.
     /// </summary>
-    public string? ManagementPermission { get; set; }
+    public string? ManagementPermissionName { get; set; }
 
     /// <summary>
     /// Parent of this permission if one exists.
@@ -89,14 +89,14 @@ public object? this[string name] {
     protected internal PermissionDefinition(
         [NotNull] string name,
         string resourceName,
-        string managementPermission,
+        string managementPermissionName,
         ILocalizableString? displayName = null,
         MultiTenancySides multiTenancySide = MultiTenancySides.Both,
         bool isEnabled = true)
         : this(name, displayName, multiTenancySide, isEnabled)
     {
         ResourceName = Check.NotNull(resourceName, nameof(resourceName));
-        ManagementPermission = Check.NotNull(managementPermission, nameof(managementPermission));
+        ManagementPermissionName = Check.NotNull(managementPermissionName, nameof(managementPermissionName));
     }
 
     protected internal PermissionDefinition(
diff --git a/framework/src/Volo.Abp.Authorization.Abstractions/Volo/Abp/Authorization/Permissions/PermissionDefinitionContext.cs b/framework/src/Volo.Abp.Authorization.Abstractions/Volo/Abp/Authorization/Permissions/PermissionDefinitionContext.cs
@@ -95,14 +95,14 @@ public virtual void RemoveGroup(string name)
     public virtual PermissionDefinition AddResourcePermission(
         string name,
         string resourceName,
-        string managementPermission,
+        string managementPermissionName,
         ILocalizableString? displayName = null,
         MultiTenancySides multiTenancySide = MultiTenancySides.Both,
         bool isEnabled = true)
     {
         Check.NotNull(name, nameof(name));
         Check.NotNull(resourceName, nameof(resourceName));
-        Check.NotNull(managementPermission, nameof(managementPermission));
+        Check.NotNull(managementPermissionName, nameof(managementPermissionName));
 
         if (ResourcePermissions.Any(x => x.ResourceName == resourceName && x.Name == name))
         {
@@ -112,7 +112,7 @@ public virtual PermissionDefinition AddResourcePermission(
         var permission = new PermissionDefinition(
             name,
             resourceName,
-            managementPermission,
+            managementPermissionName,
             displayName,
             multiTenancySide,
             isEnabled)
diff --git a/framework/test/Volo.Abp.Authorization.Tests/Volo/Abp/Authorization/TestServices/Resources/AuthorizationTestResourcePermissionDefinitionProvider.cs b/framework/test/Volo.Abp.Authorization.Tests/Volo/Abp/Authorization/TestServices/Resources/AuthorizationTestResourcePermissionDefinitionProvider.cs
@@ -15,30 +15,32 @@ public override void Define(IPermissionDefinitionContext context)
         }
         getGroup.AddPermission("TestEntityManagementPermission");
 
-        var permission1 = context.AddResourcePermission("MyResourcePermission1", resourceName: TestEntityResource.ResourceName, managementPermission: "TestEntityManagementPermission");
+        var permission1 = context.AddResourcePermission("MyResourcePermission1", resourceName: TestEntityResource.ResourceName, "TestEntityManagementPermission");
         Assert.Throws<AbpException>(() =>
         {
             permission1.AddChild("MyResourcePermission1.ChildPermission1");
         }).Message.ShouldBe($"Resource permission cannot have child permissions. Resource: {TestEntityResource.ResourceName}");
         permission1.StateCheckers.Add(new TestRequireEditionPermissionSimpleStateChecker());;
         permission1[PermissionDefinitionContext.KnownPropertyNames.CurrentProviderName].ShouldBe(typeof(AuthorizationTestResourcePermissionDefinitionProvider).FullName);
 
-        context.AddResourcePermission("MyResourcePermission2", resourceName: typeof(TestEntityResource).FullName!, managementPermission: "TestEntityManagementPermission");
-        context.AddResourcePermission("MyResourcePermission3", resourceName: typeof(TestEntityResource).FullName!, managementPermission: "TestEntityManagementPermission");
-        context.AddResourcePermission("MyResourcePermission4", resourceName: typeof(TestEntityResource).FullName!, managementPermission: "TestEntityManagementPermission");
-        context.AddResourcePermission("MyResourcePermission5", resourceName: typeof(TestEntityResource).FullName!, managementPermission: "TestEntityManagementPermission");
-        context.AddResourcePermission("MyResourcePermission6", resourceName: typeof(TestEntityResource).FullName!, managementPermission: "TestEntityManagementPermission").WithProviders(nameof(TestResourcePermissionValueProvider1));
-        context.AddResourcePermission("MyResourcePermission7", resourceName: typeof(TestEntityResource).FullName!, managementPermission: "TestEntityManagementPermission").WithProviders(nameof(TestResourcePermissionValueProvider2));
+        context.AddResourcePermission("MyResourcePermission2", resourceName: typeof(TestEntityResource).FullName!, "TestEntityManagementPermission");
+        context.AddResourcePermission("MyResourcePermission3", resourceName: typeof(TestEntityResource).FullName!, "TestEntityManagementPermission");
+        context.AddResourcePermission("MyResourcePermission4", resourceName: typeof(TestEntityResource).FullName!, "TestEntityManagementPermission");
+        context.AddResourcePermission("MyResourcePermission5", resourceName: typeof(TestEntityResource).FullName!, "TestEntityManagementPermission");
+        context.AddResourcePermission("MyResourcePermission6", resourceName: typeof(TestEntityResource).FullName!, "TestEntityManagementPermission").WithProviders(nameof(TestResourcePermissionValueProvider1));
+        context.AddResourcePermission("MyResourcePermission7", resourceName: typeof(TestEntityResource).FullName!, "TestEntityManagementPermission").WithProviders(nameof(TestResourcePermissionValueProvider2));
 
         Assert.Throws<AbpException>(() =>
         {
-            context.AddResourcePermission("MyResourcePermission7", resourceName: typeof(TestEntityResource).FullName!, managementPermission: "TestEntityManagementPermission");
+            context.AddResourcePermission("MyResourcePermission7", resourceName: typeof(TestEntityResource).FullName!, "TestEntityManagementPermission");
         }).Message.ShouldBe($"There is already an existing resource permission with name: MyResourcePermission7 for resource: {typeof(TestEntityResource).FullName}");
 
-        context.AddResourcePermission("MyResourcePermission7", resourceName: typeof(TestEntityResource2).FullName!, managementPermission: "TestEntityManagementPermission").WithProviders(nameof(TestResourcePermissionValueProvider2));
+        context.AddResourcePermission("MyResourcePermission7", resourceName: typeof(TestEntityResource2).FullName!, "TestEntityManagementPermission").WithProviders(nameof(TestResourcePermissionValueProvider2));
 
         context.GetResourcePermissionOrNull(TestEntityResource.ResourceName, "MyResourcePermission1").ShouldNotBeNull();
         context.GetResourcePermissionOrNull(TestEntityResource.ResourceName, "MyResourcePermission7").ShouldNotBeNull();
         context.GetResourcePermissionOrNull(TestEntityResource2.ResourceName, "MyResourcePermission7").ShouldNotBeNull();
+        context.GetResourcePermissionOrNull(TestEntityResource.ResourceName, "MyResourcePermission9").ShouldBeNull();
+        context.GetResourcePermissionOrNull(TestEntityResource2.ResourceName, "MyResourcePermission6").ShouldBeNull();
     }
 }
diff --git a/modules/permission-management/src/Volo.Abp.PermissionManagement.Application/Volo/Abp/PermissionManagement/PermissionAppService.cs b/modules/permission-management/src/Volo.Abp.PermissionManagement.Application/Volo/Abp/PermissionManagement/PermissionAppService.cs
@@ -170,7 +170,7 @@ public virtual async Task<GetResourceProviderListResultDto> GetResourceProviderK
     {
         var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);
         if (!resourcePermissions.Any() ||
-            !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermission!).ToArray()))
+            !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermissionName!).ToArray()))
         {
             return new GetResourceProviderListResultDto();;
         }
@@ -190,7 +190,7 @@ public virtual async Task<SearchProviderKeyListResultDto> SearchResourceProvider
     {
         var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);
         if (!resourcePermissions.Any() ||
-            !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermission!).ToArray()))
+            !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermissionName!).ToArray()))
         {
             return new SearchProviderKeyListResultDto();;
         }
@@ -217,7 +217,7 @@ public virtual async Task<GetResourcePermissionDefinitionListResultDto> GetResou
         var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);
         foreach (var resourcePermission in resourcePermissions)
         {
-            if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))
+            if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))
             {
                 result.Permissions.Add(new ResourcePermissionDefinitionDto()
                 {
@@ -257,7 +257,7 @@ public virtual async Task<GetResourcePermissionListResultDto> GetResourceAsync(s
                     continue;
                 }
 
-                if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))
+                if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))
                 {
                     resourcePermissionGrantInfoDto.Permissions.Add(new GrantedResourcePermissionDto()
                     {
@@ -290,7 +290,7 @@ public virtual async Task<GetResourcePermissionWithProviderListResultDto> GetRes
                 continue;
             }
 
-            if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))
+            if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))
             {
                 result.Permissions.Add(new ResourcePermissionWithProdiverGrantInfoDto
                 {
@@ -309,7 +309,7 @@ public virtual async Task UpdateResourceAsync(string resourceName, string resour
         var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);
         foreach (var resourcePermission in resourcePermissions)
         {
-            if (!await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))
+            if (!await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))
             {
                 continue;
             }
@@ -323,7 +323,7 @@ public virtual async Task DeleteResourceAsync(string resourceName, string resour
         var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);
         foreach (var resourcePermission in resourcePermissions)
         {
-            if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))
+            if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))
             {
                 await ResourcePermissionManager.DeleteAsync(resourcePermission.Name, resourceName, resourceKey, providerName, providerKey);
             }
diff --git a/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain.Shared/Volo/Abp/PermissionManagement/PermissionDefinitionRecordConsts.cs b/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain.Shared/Volo/Abp/PermissionManagement/PermissionDefinitionRecordConsts.cs
@@ -15,5 +15,5 @@ public class PermissionDefinitionRecordConsts
 
     public static int MaxResourceNameLength { get; set; } = 256;
 
-    public static int MaxManagementPermissionLength { get; set; } = 128;
+    public static int MaxManagementPermissionNameLength { get; set; } = 128;
 }
diff --git a/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/DynamicPermissionDefinitionStoreInMemoryCache.cs b/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/DynamicPermissionDefinitionStoreInMemoryCache.cs
@@ -53,7 +53,7 @@ public Task FillAsync(
         {
             context.AddResourcePermission(resourcePermission.Name,
                 resourcePermission.ResourceName,
-                resourcePermission.ManagementPermission,
+                resourcePermission.ManagementPermissionName,
                 resourcePermission.DisplayName != null ? LocalizableStringSerializer.Deserialize(resourcePermission.DisplayName) : null,
                 resourcePermission.MultiTenancySide,
                 resourcePermission.IsEnabled);
diff --git a/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/PermissionDefinitionRecord.cs b/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/PermissionDefinitionRecord.cs
@@ -14,7 +14,7 @@ public class PermissionDefinitionRecord : BasicAggregateRoot<Guid>, IHasExtraPro
 
     public string ResourceName { get; set; }
 
-    public string ManagementPermission { get; set; }
+    public string ManagementPermissionName { get; set; }
 
     public string ParentName { get; set; }
 
@@ -47,7 +47,7 @@ public PermissionDefinitionRecord(
         string groupName,
         string name,
         string resourceName,
-        string managementPermission,
+        string managementPermissionName,
         string parentName,
         string displayName,
         bool isEnabled = true,
@@ -63,7 +63,7 @@ public PermissionDefinitionRecord(
         }
         Name = Check.NotNullOrWhiteSpace(name, nameof(name), PermissionDefinitionRecordConsts.MaxNameLength);
         ResourceName = resourceName;
-        ManagementPermission = managementPermission;
+        ManagementPermissionName = managementPermissionName;
         ParentName = Check.Length(parentName, nameof(parentName), PermissionDefinitionRecordConsts.MaxNameLength);
         DisplayName =  Check.NotNullOrWhiteSpace(displayName, nameof(displayName), PermissionDefinitionRecordConsts.MaxDisplayNameLength);
         IsEnabled = isEnabled;
@@ -87,7 +87,7 @@ public bool HasSameData(PermissionDefinitionRecord otherRecord)
             return false;
         }
 
-        if (ManagementPermission != otherRecord.ManagementPermission)
+        if (ManagementPermissionName != otherRecord.ManagementPermissionName)
         {
             return false;
         }
@@ -147,9 +147,9 @@ public void Patch(PermissionDefinitionRecord otherRecord)
             ResourceName = otherRecord.ResourceName;
         }
 
-        if (ManagementPermission != otherRecord.ManagementPermission)
+        if (ManagementPermissionName != otherRecord.ManagementPermissionName)
         {
-            ManagementPermission = otherRecord.ManagementPermission;
+            ManagementPermissionName = otherRecord.ManagementPermissionName;
         }
 
         if (GroupName != otherRecord.GroupName)
diff --git a/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/PermissionDefinitionSerializer.cs b/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/PermissionDefinitionSerializer.cs
@@ -87,7 +87,7 @@ public virtual Task<PermissionDefinitionRecord> SerializeAsync(
                 permissionGroup?.Name,
                 permission.Name,
                 permission.ResourceName,
-                permission.ManagementPermission,
+                permission.ManagementPermissionName,
                 permission.Parent?.Name,
                 LocalizableStringSerializer.Serialize(permission.DisplayName),
                 permission.IsEnabled,
diff --git a/modules/permission-management/src/Volo.Abp.PermissionManagement.EntityFrameworkCore/Volo/Abp/PermissionManagement/EntityFrameworkCore/AbpPermissionManagementDbContextModelBuilderExtensions.cs b/modules/permission-management/src/Volo.Abp.PermissionManagement.EntityFrameworkCore/Volo/Abp/PermissionManagement/EntityFrameworkCore/AbpPermissionManagementDbContextModelBuilderExtensions.cs
@@ -72,7 +72,7 @@ public static void ConfigurePermissionManagement(
                 b.Property(x => x.GroupName).HasMaxLength(PermissionGroupDefinitionRecordConsts.MaxNameLength);
                 b.Property(x => x.Name).HasMaxLength(PermissionDefinitionRecordConsts.MaxNameLength).IsRequired();
                 b.Property(x => x.ResourceName).HasMaxLength(PermissionDefinitionRecordConsts.MaxResourceNameLength);
-                b.Property(x => x.ManagementPermission).HasMaxLength(PermissionDefinitionRecordConsts.MaxManagementPermissionLength);
+                b.Property(x => x.ManagementPermissionName).HasMaxLength(PermissionDefinitionRecordConsts.MaxManagementPermissionNameLength);
                 b.Property(x => x.ParentName).HasMaxLength(PermissionDefinitionRecordConsts.MaxNameLength);
                 b.Property(x => x.DisplayName).HasMaxLength(PermissionDefinitionRecordConsts.MaxDisplayNameLength).IsRequired();
                 b.Property(x => x.Providers).HasMaxLength(PermissionDefinitionRecordConsts.MaxProvidersLength);
diff --git a/modules/permission-management/test/Volo.Abp.PermissionManagement.Domain.Tests/Volo/Abp/PermissionManagement/PermissionDefinitionSerializer_Tests.cs b/modules/permission-management/test/Volo.Abp.PermissionManagement.Domain.Tests/Volo/Abp/PermissionManagement/PermissionDefinitionSerializer_Tests.cs
@@ -111,7 +111,7 @@ public async Task Serialize_Complex_Resource_Permission_Definition()
         permissionRecord.Name.ShouldBe("ResourcePermission1");
         permissionRecord.GroupName.ShouldBe(null);
         permissionRecord.ResourceName.ShouldBe(TestEntityResource.ResourceName);
-        permissionRecord.ManagementPermission.ShouldBe("Permission1");
+        permissionRecord.ManagementPermissionName.ShouldBe("Permission1");
         permissionRecord.DisplayName.ShouldBe("L:AbpPermissionManagement,ResourcePermission1");
         permissionRecord.GetProperty("CustomProperty2").ShouldBe("CustomValue2");
         permissionRecord.Providers.ShouldBe("ProviderA,ProviderB");

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ public virtual async Task<GetResourceProviderListResultDto> GetResourceProviderK`
`170`	`170`	`{`
`171`	`171`	`var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);`
`172`	`172`	`if (!resourcePermissions.Any() \|\|`
`173`		`- !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermission!).ToArray()))`
	`173`	`+ !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermissionName!).ToArray()))`
`174`	`174`	`{`
`175`	`175`	`return new GetResourceProviderListResultDto();;`
`176`	`176`	`}`
`@@ -190,7 +190,7 @@ public virtual async Task<SearchProviderKeyListResultDto> SearchResourceProvider`
`190`	`190`	`{`
`191`	`191`	`var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);`
`192`	`192`	`if (!resourcePermissions.Any() \|\|`
`193`		`- !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermission!).ToArray()))`
	`193`	`+ !await AuthorizationService.IsGrantedAnyAsync(resourcePermissions.Select(p => p.ManagementPermissionName!).ToArray()))`
`194`	`194`	`{`
`195`	`195`	`return new SearchProviderKeyListResultDto();;`
`196`	`196`	`}`
`@@ -217,7 +217,7 @@ public virtual async Task<GetResourcePermissionDefinitionListResultDto> GetResou`
`217`	`217`	`var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);`
`218`	`218`	`foreach (var resourcePermission in resourcePermissions)`
`219`	`219`	`{`
`220`		`- if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))`
	`220`	`+ if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))`
`221`	`221`	`{`
`222`	`222`	`result.Permissions.Add(new ResourcePermissionDefinitionDto()`
`223`	`223`	`{`
`@@ -257,7 +257,7 @@ public virtual async Task<GetResourcePermissionListResultDto> GetResourceAsync(s`
`257`	`257`	`continue;`
`258`	`258`	`}`
`259`	`259`
`260`		`- if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))`
	`260`	`+ if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))`
`261`	`261`	`{`
`262`	`262`	`resourcePermissionGrantInfoDto.Permissions.Add(new GrantedResourcePermissionDto()`
`263`	`263`	`{`
`@@ -290,7 +290,7 @@ public virtual async Task<GetResourcePermissionWithProviderListResultDto> GetRes`
`290`	`290`	`continue;`
`291`	`291`	`}`
`292`	`292`
`293`		`- if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))`
	`293`	`+ if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))`
`294`	`294`	`{`
`295`	`295`	`result.Permissions.Add(new ResourcePermissionWithProdiverGrantInfoDto`
`296`	`296`	`{`
`@@ -309,7 +309,7 @@ public virtual async Task UpdateResourceAsync(string resourceName, string resour`
`309`	`309`	`var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);`
`310`	`310`	`foreach (var resourcePermission in resourcePermissions)`
`311`	`311`	`{`
`312`		`- if (!await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))`
	`312`	`+ if (!await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))`
`313`	`313`	`{`
`314`	`314`	`continue;`
`315`	`315`	`}`
`@@ -323,7 +323,7 @@ public virtual async Task DeleteResourceAsync(string resourceName, string resour`
`323`	`323`	`var resourcePermissions = await ResourcePermissionManager.GetAvailablePermissionsAsync(resourceName);`
`324`	`324`	`foreach (var resourcePermission in resourcePermissions)`
`325`	`325`	`{`
`326`		`- if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermission!))`
	`326`	`+ if (await AuthorizationService.IsGrantedAsync(resourcePermission.ManagementPermissionName!))`
`327`	`327`	`{`
`328`	`328`	`await ResourcePermissionManager.DeleteAsync(resourcePermission.Name, resourceName, resourceKey, providerName, providerKey);`
`329`	`329`	`}`
Original file line number	Diff line number	Diff line change
`@@ -15,5 +15,5 @@ public class PermissionDefinitionRecordConsts`
`15`	`15`
`16`	`16`	`public static int MaxResourceNameLength { get; set; } = 256;`
`17`	`17`
`18`		`- public static int MaxManagementPermissionLength { get; set; } = 128;`
	`18`	`+ public static int MaxManagementPermissionNameLength { get; set; } = 128;`
`19`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ public Task FillAsync(`
`53`	`53`	`{`
`54`	`54`	`context.AddResourcePermission(resourcePermission.Name,`
`55`	`55`	`resourcePermission.ResourceName,`
`56`		`- resourcePermission.ManagementPermission,`
	`56`	`+ resourcePermission.ManagementPermissionName,`
`57`	`57`	`resourcePermission.DisplayName != null ? LocalizableStringSerializer.Deserialize(resourcePermission.DisplayName) : null,`
`58`	`58`	`resourcePermission.MultiTenancySide,`
`59`	`59`	`resourcePermission.IsEnabled);`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ public class PermissionDefinitionRecord : BasicAggregateRoot<Guid>, IHasExtraPro`
`14`	`14`
`15`	`15`	`public string ResourceName { get; set; }`
`16`	`16`
`17`		`- public string ManagementPermission { get; set; }`
	`17`	`+ public string ManagementPermissionName { get; set; }`
`18`	`18`
`19`	`19`	`public string ParentName { get; set; }`
`20`	`20`
`@@ -47,7 +47,7 @@ public PermissionDefinitionRecord(`
`47`	`47`	`string groupName,`
`48`	`48`	`string name,`
`49`	`49`	`string resourceName,`
`50`		`- string managementPermission,`
	`50`	`+ string managementPermissionName,`
`51`	`51`	`string parentName,`
`52`	`52`	`string displayName,`
`53`	`53`	`bool isEnabled = true,`
`@@ -63,7 +63,7 @@ public PermissionDefinitionRecord(`
`63`	`63`	`}`
`64`	`64`	`Name = Check.NotNullOrWhiteSpace(name, nameof(name), PermissionDefinitionRecordConsts.MaxNameLength);`
`65`	`65`	`ResourceName = resourceName;`
`66`		`- ManagementPermission = managementPermission;`
	`66`	`+ ManagementPermissionName = managementPermissionName;`
`67`	`67`	`ParentName = Check.Length(parentName, nameof(parentName), PermissionDefinitionRecordConsts.MaxNameLength);`
`68`	`68`	`DisplayName = Check.NotNullOrWhiteSpace(displayName, nameof(displayName), PermissionDefinitionRecordConsts.MaxDisplayNameLength);`
`69`	`69`	`IsEnabled = isEnabled;`
`@@ -87,7 +87,7 @@ public bool HasSameData(PermissionDefinitionRecord otherRecord)`
`87`	`87`	`return false;`
`88`	`88`	`}`
`89`	`89`
`90`		`- if (ManagementPermission != otherRecord.ManagementPermission)`
	`90`	`+ if (ManagementPermissionName != otherRecord.ManagementPermissionName)`
`91`	`91`	`{`
`92`	`92`	`return false;`
`93`	`93`	`}`
`@@ -147,9 +147,9 @@ public void Patch(PermissionDefinitionRecord otherRecord)`
`147`	`147`	`ResourceName = otherRecord.ResourceName;`
`148`	`148`	`}`
`149`	`149`
`150`		`- if (ManagementPermission != otherRecord.ManagementPermission)`
	`150`	`+ if (ManagementPermissionName != otherRecord.ManagementPermissionName)`
`151`	`151`	`{`
`152`		`- ManagementPermission = otherRecord.ManagementPermission;`
	`152`	`+ ManagementPermissionName = otherRecord.ManagementPermissionName;`
`153`	`153`	`}`
`154`	`154`
`155`	`155`	`if (GroupName != otherRecord.GroupName)`