Merge pull request #24531 from abpframework/ResourcePermissionManager-Set

yagmurcelk · web-flow · commit 4e628ede9d9e · 2026-01-05T09:00:34.000+03:00
Fix condition check in ResourcePermissionManager to ensure provider match before returning
diff --git a/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/ResourcePermissionManager.cs b/modules/permission-management/src/Volo.Abp.PermissionManagement.Domain/Volo/Abp/PermissionManagement/ResourcePermissionManager.cs
@@ -270,7 +270,7 @@ maybe they were removed from dynamic permission definition store */
         }
 
         var currentGrantInfo = await GetInternalAsync(permission, resourceName, resourceKey, providerName, providerKey);
-        if (currentGrantInfo.IsGranted == isGranted)
+        if (currentGrantInfo.IsGranted == isGranted && currentGrantInfo.Providers.Any(x => x.Name == providerName && x.Key == providerKey))
         {
             return;
         }

Original file line number	Diff line number	Diff line change
`@@ -270,7 +270,7 @@ maybe they were removed from dynamic permission definition store */`
`270`	`270`	`}`
`271`	`271`
`272`	`272`	`var currentGrantInfo = await GetInternalAsync(permission, resourceName, resourceKey, providerName, providerKey);`
`273`		`- if (currentGrantInfo.IsGranted == isGranted)`
	`273`	`+ if (currentGrantInfo.IsGranted == isGranted && currentGrantInfo.Providers.Any(x => x.Name == providerName && x.Key == providerKey))`
`274`	`274`	`{`
`275`	`275`	`return;`
`276`	`276`	`}`