svn commit: r18399 - trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CollaborationsHelperMDRImpl.java

Author: bobtarling
Date: 2010-05-18 14:20:28-0700
New Revision: 18399

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

Log:
Allow multiple classifierroles with same base

Modified: trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CollaborationsHelperMDRImpl.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CollaborationsHelperMDRImpl.java?view=diff&pathrev=18399&r1=18398&r2=18399
==============================================================================
--- trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CollaborationsHelperMDRImpl.java	(original)
+++ trunk/src/argouml-core-model-mdr/src/org/argouml/model/mdr/CollaborationsHelperMDRImpl.java	2010-05-18 14:20:28-0700
@@ -350,13 +350,13 @@
             throw new IllegalArgumentException("In addBase: either the role "
                     + "or the base is null");
         }
-        // TODO: This check probably belongs in a critic instead of here since
-        // the rule can be violated later if the names are cleared
-        if (!isNamedOrUnique(role, base)) {
-            throw new IllegalArgumentException("In addBase: base is "
-                    + "already part of " + "another role and "
-                    + "role does not have " + "a name");
-        }
+//        // TODO: This check probably belongs in a critic instead of here since
+//        // the rule can be violated later if the names are cleared
+//        if (!isNamedOrUnique(role, base)) {
+//            throw new IllegalArgumentException("In addBase: base is "
+//                    + "already part of " + "another role and "
+//                    + "role does not have " + "a name");
+//        }
         role.getBase().add(base);
         if (modelImpl.getFacade().getBases(role).size() == 1) {
             role.getAvailableContents().clear();

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=2610550

To unsubscribe from this discussion, e-mail: [[email protected]].