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]].
