Refactor code

wso2 · Oct 20, 2023 · b85e1c4 · b85e1c4
1 parent 561431f
commit b85e1c4
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/modules/charon-core/src/main/java/org/wso2/charon3/core/protocol/BulkRequestProcessor.java b/modules/charon-core/src/main/java/org/wso2/charon3/core/protocol/BulkRequestProcessor.java
@@ -290,7 +290,7 @@ private void processBulkRequestContent(BulkRequestContent bulkRequestContent, Ma
 
         try {
             if (userIDMappings == null || userIDMappings.isEmpty() ||
-                    method.equals(SCIMConstants.OperationalConstants.DELETE)) {
+                    SCIMConstants.OperationalConstants.DELETE.equals(method)) {
                 return;
             }
 
@@ -354,11 +354,10 @@ private JSONArray getUserArrayForPatch(JSONObject dataJson, String usersOrMember
             if (isValidOperation(operation, usersOrMembersKey)) {
                 if (operation.has(SCIMConstants.OperationalConstants.PATH)) {
                     return operation.optJSONArray(SCIMConstants.OperationalConstants.VALUE);
-                } else {
-                    JSONObject valueObject = operation.optJSONObject(SCIMConstants.OperationalConstants.VALUE);
-                    if (valueObject != null) {
-                        return valueObject.optJSONArray(usersOrMembersKey);
-                    }
+                }
+                JSONObject valueObject = operation.optJSONObject(SCIMConstants.OperationalConstants.VALUE);
+                if (valueObject != null) {
+                    return valueObject.optJSONArray(usersOrMembersKey);
                 }
             }
         }