summaryrefslogtreecommitdiffstats
path: root/sca-java-2.x/trunk/modules/assembly/src
diff options
context:
space:
mode:
Diffstat (limited to 'sca-java-2.x/trunk/modules/assembly/src')
-rw-r--r--sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/DataTypeImpl.java14
1 files changed, 14 insertions, 0 deletions
diff --git a/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/DataTypeImpl.java b/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/DataTypeImpl.java
index 572dd7e356..5d084d9289 100644
--- a/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/DataTypeImpl.java
+++ b/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/DataTypeImpl.java
@@ -168,6 +168,20 @@ public class DataTypeImpl<L> implements DataType<L> {
@Override
public Object clone() throws CloneNotSupportedException {
DataTypeImpl copy = (DataTypeImpl)super.clone();
+
+ //
+ // When using a DataTypeImpl with java:array databinding,
+ // 'logical' will be another DataTypeImpl. Doing only
+ // a shallow copy means that resetting the databinding will
+ // have an unexpected side effect on the original. Though
+ // we could special case the java:array-databinding case,
+ // instead do it more generally when the logical is another
+ // DataType.
+ //
+ if (logical instanceof DataType) {
+ DataType logicalDT = (DataType)logical;
+ copy.logical = logicalDT.clone();
+ }
return copy;
}