svn commit: r18237 - trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/CoreHelperEUMLImpl.java

Author: bobtarling
Date: 2010-04-10 03:32:45-0700
New Revision: 18237

Modified:
   trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/CoreHelperEUMLImpl.java

Log:
Issue 6056: Delete any existing multiplicity before setting a new value.

Modified: trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/CoreHelperEUMLImpl.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/CoreHelperEUMLImpl.java?view=diff&pathrev=18237&r1=18236&r2=18237
==============================================================================
--- trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/CoreHelperEUMLImpl.java	(original)
+++ trunk/src/argouml-core-model-euml/src/org/argouml/model/euml/CoreHelperEUMLImpl.java	2010-04-10 03:32:45-0700
@@ -1493,6 +1493,14 @@
             final int lower_ = lower, upper_ = upper;
             RunnableClass run = new RunnableClass() {
                 public void run() {
+                    // TODO: We currently delete the old values before setting
+                    // to something new. This is a workaround to issue 6056.
+                    // We should consider giving an API to get the lower and
+                    // upper values so that controls can listen directly to
+                    // those rather than the element containing those values.
+                    ((MultiplicityElement) handle).setLowerValue(null);
+                    ((MultiplicityElement) handle).setUpperValue(null);
+                    //
                     ((MultiplicityElement) handle).setLower(lower_);
                     ((MultiplicityElement) handle).setUpper(upper_);
                 }

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

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