svn commit: r14568 - trunk/src/argouml-app/src/org/argouml/uml/diagram: state/ui ui

Author: tfmorris
Date: 2008-05-01 12:45:17-0700
New Revision: 14568

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigConcurrentRegion.java
   trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionAddConcurrentRegion.java
   trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionDeleteConcurrentRegion.java

Log:
Add TODOs for issue 5050

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigConcurrentRegion.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigConcurrentRegion.java?view=diff&rev=14568&p1=trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigConcurrentRegion.java&p2=trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigConcurrentRegion.java&r1=14567&r2=14568
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigConcurrentRegion.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/FigConcurrentRegion.java	2008-05-01 12:45:17-0700
@@ -148,6 +148,8 @@
         popUpActions.remove(
                 ProjectActions.getInstance().getRemoveFromDiagramAction());
         popUpActions.add(new JSeparator());
+        // TODO: There's a cyclic dependency between FigConcurrentRegion and
+        // the actions ActionAddConcurrentRegion and ActionDeleteConcurrentRegion
         popUpActions.addElement(
                 new ActionAddConcurrentRegion());
         popUpActions.addElement(

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionAddConcurrentRegion.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionAddConcurrentRegion.java?view=diff&rev=14568&p1=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionAddConcurrentRegion.java&p2=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionAddConcurrentRegion.java&r1=14567&r2=14568
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionAddConcurrentRegion.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionAddConcurrentRegion.java	2008-05-01 12:45:17-0700
@@ -49,6 +49,8 @@
 import org.tigris.gef.presentation.Fig;
 import org.tigris.gef.undo.UndoableAction;
 
+//TODO: There is a cyclic dependency between this class and FigConcurrentRegion
+
 /**
  * Add a concurrent region to a concurrent composite state
  *

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionDeleteConcurrentRegion.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionDeleteConcurrentRegion.java?view=diff&rev=14568&p1=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionDeleteConcurrentRegion.java&p2=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionDeleteConcurrentRegion.java&r1=14567&r2=14568
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionDeleteConcurrentRegion.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/ActionDeleteConcurrentRegion.java	2008-05-01 12:45:17-0700
@@ -43,6 +43,8 @@
 import org.tigris.gef.presentation.Fig;
 import org.tigris.gef.undo.UndoableAction;
 
+//TODO: There is a cyclic dependency between this class and FigConcurrentRegion
+
 /**
  * Delete a concurrent region of a concurrent composite state
  *