You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@syncope.apache.org by cs...@apache.org on 2013/01/23 15:00:01 UTC

svn commit: r1437396 - in /syncope/trunk/common/src/main/java/org/apache/syncope/common/to: AbstractAttributableTO.java ConnInstanceTO.java MappingTO.java NotificationTaskTO.java RoleTO.java WorkflowFormTO.java

Author: cschneider
Date: Wed Jan 23 14:00:01 2013
New Revision: 1437396

URL: http://svn.apache.org/viewvc?rev=1437396&view=rev
Log:
SYNCOPE-288 Changing setters of TO classes to make them compatible with jaxb

Modified:
    syncope/trunk/common/src/main/java/org/apache/syncope/common/to/AbstractAttributableTO.java
    syncope/trunk/common/src/main/java/org/apache/syncope/common/to/ConnInstanceTO.java
    syncope/trunk/common/src/main/java/org/apache/syncope/common/to/MappingTO.java
    syncope/trunk/common/src/main/java/org/apache/syncope/common/to/NotificationTaskTO.java
    syncope/trunk/common/src/main/java/org/apache/syncope/common/to/RoleTO.java
    syncope/trunk/common/src/main/java/org/apache/syncope/common/to/WorkflowFormTO.java

Modified: syncope/trunk/common/src/main/java/org/apache/syncope/common/to/AbstractAttributableTO.java
URL: http://svn.apache.org/viewvc/syncope/trunk/common/src/main/java/org/apache/syncope/common/to/AbstractAttributableTO.java?rev=1437396&r1=1437395&r2=1437396&view=diff
==============================================================================
--- syncope/trunk/common/src/main/java/org/apache/syncope/common/to/AbstractAttributableTO.java (original)
+++ syncope/trunk/common/src/main/java/org/apache/syncope/common/to/AbstractAttributableTO.java Wed Jan 23 14:00:01 2013
@@ -165,7 +165,9 @@ public abstract class AbstractAttributab
     }
 
     public void setPropagationTOs(final List<PropagationTO> propagationTOs) {
-        this.propagationTOs.clear();
-        this.propagationTOs.addAll(propagationTOs);
+        if (this.propagationTOs != propagationTOs) {
+            this.propagationTOs.clear();
+            this.propagationTOs.addAll(propagationTOs);
+        }
     }
 }

Modified: syncope/trunk/common/src/main/java/org/apache/syncope/common/to/ConnInstanceTO.java
URL: http://svn.apache.org/viewvc/syncope/trunk/common/src/main/java/org/apache/syncope/common/to/ConnInstanceTO.java?rev=1437396&r1=1437395&r2=1437396&view=diff
==============================================================================
--- syncope/trunk/common/src/main/java/org/apache/syncope/common/to/ConnInstanceTO.java (original)
+++ syncope/trunk/common/src/main/java/org/apache/syncope/common/to/ConnInstanceTO.java Wed Jan 23 14:00:01 2013
@@ -118,9 +118,11 @@ public class ConnInstanceTO extends Abst
     }
 
     public void setConfiguration(Set<ConnConfProperty> configuration) {
-        this.configuration.clear();
-        if (configuration != null && !configuration.isEmpty()) {
-            this.configuration.addAll(configuration);
+        if (this.configuration != configuration) {
+            this.configuration.clear();
+            if (configuration != null && !configuration.isEmpty()) {
+                this.configuration.addAll(configuration);
+            }
         }
     }
 
@@ -147,9 +149,11 @@ public class ConnInstanceTO extends Abst
     }
 
     public void setCapabilities(final Set<ConnectorCapability> capabilities) {
-        this.capabilities.clear();
-        if (capabilities != null && !capabilities.isEmpty()) {
-            this.capabilities.addAll(capabilities);
+        if (this.capabilities != capabilities) {
+            this.capabilities.clear();
+            if (capabilities != null && !capabilities.isEmpty()) {
+                this.capabilities.addAll(capabilities);
+            }
         }
     }
 

Modified: syncope/trunk/common/src/main/java/org/apache/syncope/common/to/MappingTO.java
URL: http://svn.apache.org/viewvc/syncope/trunk/common/src/main/java/org/apache/syncope/common/to/MappingTO.java?rev=1437396&r1=1437395&r2=1437396&view=diff
==============================================================================
--- syncope/trunk/common/src/main/java/org/apache/syncope/common/to/MappingTO.java (original)
+++ syncope/trunk/common/src/main/java/org/apache/syncope/common/to/MappingTO.java Wed Jan 23 14:00:01 2013
@@ -117,9 +117,11 @@ public class MappingTO extends AbstractB
     }
 
     public void setItems(final Collection<MappingItemTO> items) {
-        this.items.clear();
-        if (items != null && !items.isEmpty()) {
-            this.items.addAll(items);
+        if (this.items != items) {
+            this.items.clear();
+            if (items != null && !items.isEmpty()) {
+                this.items.addAll(items);
+            }
         }
     }
 }

Modified: syncope/trunk/common/src/main/java/org/apache/syncope/common/to/NotificationTaskTO.java
URL: http://svn.apache.org/viewvc/syncope/trunk/common/src/main/java/org/apache/syncope/common/to/NotificationTaskTO.java?rev=1437396&r1=1437395&r2=1437396&view=diff
==============================================================================
--- syncope/trunk/common/src/main/java/org/apache/syncope/common/to/NotificationTaskTO.java (original)
+++ syncope/trunk/common/src/main/java/org/apache/syncope/common/to/NotificationTaskTO.java Wed Jan 23 14:00:01 2013
@@ -60,9 +60,11 @@ public class NotificationTaskTO extends 
     }
 
     public void setRecipients(final Set<String> recipients) {
-        this.recipients.clear();
-        if (recipients != null) {
-            this.recipients.addAll(recipients);
+        if (this.recipients != recipients) {
+            this.recipients.clear();
+            if (recipients != null) {
+                this.recipients.addAll(recipients);
+            }
         }
     }
 

Modified: syncope/trunk/common/src/main/java/org/apache/syncope/common/to/RoleTO.java
URL: http://svn.apache.org/viewvc/syncope/trunk/common/src/main/java/org/apache/syncope/common/to/RoleTO.java?rev=1437396&r1=1437395&r2=1437396&view=diff
==============================================================================
--- syncope/trunk/common/src/main/java/org/apache/syncope/common/to/RoleTO.java (original)
+++ syncope/trunk/common/src/main/java/org/apache/syncope/common/to/RoleTO.java Wed Jan 23 14:00:01 2013
@@ -135,9 +135,11 @@ public class RoleTO extends AbstractAttr
     }
 
     public void setEntitlements(final List<String> entitlements) {
-        this.entitlements.clear();
-        if (entitlements != null && !entitlements.isEmpty()) {
-            this.entitlements.addAll(entitlements);
+        if (this.entitlements != entitlements) {
+            this.entitlements.clear();
+            if (entitlements != null && !entitlements.isEmpty()) {
+                this.entitlements.addAll(entitlements);
+            }
         }
     }
 

Modified: syncope/trunk/common/src/main/java/org/apache/syncope/common/to/WorkflowFormTO.java
URL: http://svn.apache.org/viewvc/syncope/trunk/common/src/main/java/org/apache/syncope/common/to/WorkflowFormTO.java?rev=1437396&r1=1437395&r2=1437396&view=diff
==============================================================================
--- syncope/trunk/common/src/main/java/org/apache/syncope/common/to/WorkflowFormTO.java (original)
+++ syncope/trunk/common/src/main/java/org/apache/syncope/common/to/WorkflowFormTO.java Wed Jan 23 14:00:01 2013
@@ -113,10 +113,11 @@ public class WorkflowFormTO extends Abst
     }
 
     public void setProperties(final Collection<WorkflowFormPropertyTO> properties) {
-
-        this.properties.clear();
-        if (properties != null) {
-            this.properties.addAll(properties);
+        if (this.properties != properties) {
+            this.properties.clear();
+            if (properties != null) {
+                this.properties.addAll(properties);
+            }
         }
     }