Merge pull request #495 from PasinduYeshan/fix/v2-role-replace

Fixed empty permission list replace issue in v2 Role patch
wso2-extensions · Oct 26, 2023 · 20d8dba · 20d8dba
2 parents 6175145 + 868e78d
commit 20d8dba
Showing 1 changed file with 0 additions and 4 deletions.
diff --git a/...m2.common/src/main/java/org/wso2/carbon/identity/scim2/common/impl/SCIMRoleManagerV2.java b/...m2.common/src/main/java/org/wso2/carbon/identity/scim2/common/impl/SCIMRoleManagerV2.java
@@ -849,10 +849,6 @@ private void prepareAddedRemovedPermissionLists(Set<String> addedPermissions, Se
     private void prepareReplacedPermissionLists(List<Permission> permissionsOfRole, Set<String> addedPermissions,
                                                 Set<String> removedPermissions, Set<String> replacedPermissions) {
 
-        if (replacedPermissions.isEmpty()) {
-            return;
-        }
-
         if (!permissionsOfRole.isEmpty()) {
             for (Permission permission : permissionsOfRole) {
                 if (!replacedPermissions.contains(permission.getName())) {