svn commit: r15236 - trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java

Author: tfmorris
Date: 2008-07-10 22:33:14-0700
New Revision: 15236

Modified:
   trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java

Log:
Issue 5211: Remove side effect from buildAttribute which attempts to force type into a different namespace.

Modified: trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java?view=diff&rev=15236&p1=trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java&p2=trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java&r1=15235&r2=15236
==============================================================================
--- trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java	(original)
+++ trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CoreFactoryMDRImpl.java	2008-07-10 22:33:14-0700
@@ -761,16 +761,6 @@
 
 
     public Attribute buildAttribute(Object model, Object theType) {
-        Classifier clsType = (Classifier) theType;
-        // Force type element into given namespace if not already there
-        // side effect!
-        /* TODO: (MVW) This is wrong! See issue 5211. 
-         * We have profiles now... */ 
-        if (model != clsType.getNamespace()
-                && !(modelImpl.getModelManagementHelper().getAllNamespaces(
-                        model).contains(clsType.getNamespace()))) {
-            clsType.setNamespace((Model) model);
-        }
         return buildAttribute2(theType);
     }