(syncope) branch master updated: [SYNCOPE-1803] Allow removing of all membership plain schemas value from console (#622)

[sgarofalo]

This is an automated email from the ASF dual-hosted git repository.

sgarofalo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/syncope.git


The following commit(s) were added to refs/heads/master by this push:
     new df729ffd2c [SYNCOPE-1803] Allow removing of all membership plain 
schemas value from console (#622)
df729ffd2c is described below

commit df729ffd2c46001de81e8d0de9113b427bdf7e3a
Author: Samuel Garofalo <72073457+samuelg...@users.noreply.github.com>
AuthorDate: Mon Feb 19 11:39:56 2024 +0100

    [SYNCOPE-1803] Allow removing of all membership plain schemas value from 
console (#622)
---
 .../java/org/apache/syncope/common/lib/AnyOperations.java  | 14 ++------------
 1 file changed, 2 insertions(+), 12 deletions(-)

diff --git 
a/common/idrepo/lib/src/main/java/org/apache/syncope/common/lib/AnyOperations.java
 
b/common/idrepo/lib/src/main/java/org/apache/syncope/common/lib/AnyOperations.java
index a7e9220f05..5baff5a480 100644
--- 
a/common/idrepo/lib/src/main/java/org/apache/syncope/common/lib/AnyOperations.java
+++ 
b/common/idrepo/lib/src/main/java/org/apache/syncope/common/lib/AnyOperations.java
@@ -198,12 +198,7 @@ public final class AnyOperations {
                     operation(PatchOperation.ADD_REPLACE).build();
 
             diff(value, membershipPatch);
-
-            if (!originalMembs.containsKey(key)
-                    || (!membershipPatch.getPlainAttrs().isEmpty() || 
!membershipPatch.getVirAttrs().isEmpty())) {
-
-                result.getMemberships().add(membershipPatch);
-            }
+            result.getMemberships().add(membershipPatch);
         });
 
         if (!incremental) {
@@ -309,12 +304,7 @@ public final class AnyOperations {
                     operation(PatchOperation.ADD_REPLACE).build();
 
             diff(value, membershipPatch);
-
-            if (!originalMembs.containsKey(key)
-                    || (!membershipPatch.getPlainAttrs().isEmpty() || 
!membershipPatch.getVirAttrs().isEmpty())) {
-
-                result.getMemberships().add(membershipPatch);
-            }
+            result.getMemberships().add(membershipPatch);
         });
 
         if (!incremental) {