svn commit: r18846 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java

Author: mvw
Date: 2010-11-15 23:52:51-0800
New Revision: 18846

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

Log:
Correct this mistake. Why did nobody see this before?
Still to do: check the implementation of updateBounds() by the descendents of FigNodeModelElement.

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java?view=diff&pathrev=18846&r1=18845&r2=18846
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigNodeModelElement.java	2010-11-15 23:52:51-0800
@@ -9,6 +9,7 @@
  * Contributors:
  *    Thomas Neustupny
  *    Bob Tarling
+ *    Michiel van der Wulp
  *****************************************************************************
  *
  * Some portions of this file was previously release using the BSD License:
@@ -109,8 +110,8 @@
 import org.argouml.uml.diagram.DiagramAppearance;
 import org.argouml.uml.diagram.DiagramElement;
 import org.argouml.uml.diagram.DiagramSettings;
-import org.argouml.uml.diagram.PathContainer;
 import org.argouml.uml.diagram.DiagramSettings.StereotypeStyle;
+import org.argouml.uml.diagram.PathContainer;
 import org.argouml.uml.ui.ActionDeleteModelElements;
 import org.argouml.util.IItemUID;
 import org.argouml.util.ItemUID;
@@ -1068,10 +1069,11 @@
             bbox.width = Math.max(bbox.width, minSize.width);
             bbox.height = Math.max(bbox.height, minSize.height);
         }
+        /* Only update the bounds if they change:  */
         if (bbox.x != getX()
-                && bbox.y != getY()
-                && bbox.width != getWidth()
-                && bbox.height != getHeight()) {
+                || bbox.y != getY()
+                || bbox.width != getWidth()
+                || bbox.height != getHeight()) {
             setBounds(bbox.x, bbox.y, bbox.width, bbox.height);
         }
     }

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

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