svn commit: r14592 - trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java

[email protected]
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: bobtarling
Date: 2008-05-02 08:25:47-0700
New Revision: 14592

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java

Log:
Make sure any rebuild happens on AWT

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java?view=diff&rev=14592&p1=trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java&p2=trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java&r1=14591&r2=14592
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigCompositeState.java	2008-05-02 08:25:47-0700
@@ -34,6 +34,7 @@
 import java.util.Vector;
 
 import org.argouml.model.Model;
+import org.argouml.model.UmlChangeEvent;
 import org.argouml.ui.targetmanager.TargetManager;
 import org.argouml.uml.diagram.ui.ActionAddConcurrentRegion;
 import org.tigris.gef.graph.GraphModel;
@@ -322,22 +323,19 @@
     ////////////////////////////////////////////////////////////////
     // event processing
 
-    /*
-     * Update the text labels.
-     *
-     * @see org.argouml.uml.diagram.ui.FigNodeModelElement#modelChanged(java.beans.PropertyChangeEvent)
-     */
-    protected void modelChanged(PropertyChangeEvent mee) {
-        super.modelChanged(mee);
-
-        if (mee.getPropertyName().equals("isConcurrent")) {
+    @Override 
+    protected void updateLayout(UmlChangeEvent event) {
+        if (event.getPropertyName().equals("isConcurrent")) {
             // TODO: this should split the composite state into two
-            // regions. This must be implemented
+            // regions. This must be implemented. 
+            // Bob says - But it appears to me that there is no way to change
+            // isConcurrent of an existing CompositeState so do we
+            // really need to do this and do we need to redraw everything?
             renderingChanged();
         }
-
     }
 
+
     /*
      * @see org.argouml.uml.diagram.state.ui.FigState#getInitialHeight()
      */
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.