svn commit: r16848 - trunk/src/argouml-app/src/org/argouml/uml/diagram/activity/ui/FigPool.java

Author: bobtarling
Date: 2009-03-02 13:39:16-0800
New Revision: 16848

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

Log:
Issue 5705: Make sure owner is always null

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/activity/ui/FigPool.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/activity/ui/FigPool.java?view=diff&pathrev=16848&r1=16847&r2=16848
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/activity/ui/FigPool.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/activity/ui/FigPool.java	2009-03-02 13:39:16-0800
@@ -38,7 +38,14 @@
 import org.tigris.gef.presentation.FigRect;
 
 /**
- * This class represents a Pool of Swimlanes for Activity diagrams.
+ * This class represents a Pool of Swimlanes for Activity diagrams. This is
+ * exists only to act as the GEF style encloser. Other nodes in the same
+ * muist be placed entirely within or outside the boundaries of this Fig
+ * but cannot intersect with the boundary.
+ * TODO: There is no actual model element being represented here so we are
+ * inheriting a lot of behaviour from FigNodeModelElement that we don't want.
+ * We require to split FigNodeModelElement to separate the code that requires
+ * a model element owner. See issue 
  *
  * @author mkl
  */
@@ -75,7 +82,7 @@
      * @param settings (ignored since this is just an empty rectangle)
      */
     public FigPool(Object owner, Rectangle bounds, DiagramSettings settings) {
-        super(owner, bounds, settings);
+        super(null, bounds, settings);
         initialize(bounds);
     }

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

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