add a comment

wso2 · Aug 3, 2023 · a648f7a · a648f7a
1 parent 0a74fb0
commit a648f7a
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/...y.role.mgt.core/src/main/java/org/wso2/carbon/identity/role/mgt/core/dao/RoleDAOImpl.java b/...y.role.mgt.core/src/main/java/org/wso2/carbon/identity/role/mgt/core/dao/RoleDAOImpl.java
@@ -1449,6 +1449,7 @@ private Map<String, String> batchProcessRoleNames(List<String> roleNames, String
         return roleNamesToIDs;
     }
 
+    @SuppressWarnings("checkstyle:LineLength")
     @Override
     public String getRoleNameByID(String roleID, String tenantDomain) throws IdentityRoleManagementException {
 
@@ -1480,6 +1481,8 @@ public String getRoleNameByID(String roleID, String tenantDomain) throws Identit
                             + tenantDomain;
             throw new IdentityRoleManagementServerException(UNEXPECTED_SERVER_ERROR.getCode(), errorMessage, e);
         }
+        /* Verify whether the roleName is either null or a groupName
+        (excluding those with the 'Internal/Application' prefix). */
         if (roleName == null || !SCIMCommonUtils.isHybridRole(roleName)) {
             String errorMessage = "A role doesn't exist with id: " + roleID + " in the tenantDomain: " + tenantDomain;
             throw new IdentityRoleManagementClientException(ROLE_NOT_FOUND.getCode(), errorMessage);