svn commit: r14571 - trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java

[tfmorris]

Author: tfmorris
Date: 2008-05-01 13:33:51-0700
New Revision: 14571

Modified:
   trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java

Log:
TODO for issue 5054

Modified: trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java
Url: 
http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java?view=diff&rev=14571&p1=trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java&p2=trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java&r1=14570&r2=14571
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java      
(original)
+++ trunk/src/argouml-app/src/org/argouml/persistence/PGMLStackParser.java      
2008-05-01 13:33:51-0700
@@ -42,8 +42,6 @@
 import org.argouml.uml.diagram.StereotypeContainer;
 import org.argouml.uml.diagram.VisibilityContainer;
 import org.argouml.uml.diagram.activity.ui.FigPool;
-import org.argouml.uml.diagram.deployment.ui.FigMNodeInstance;
-import org.argouml.uml.diagram.deployment.ui.FigNodeInstance;
 import org.argouml.uml.diagram.ui.FigEdgeModelElement;
 import org.argouml.uml.diagram.ui.FigEdgePort;
 import org.tigris.gef.base.Diagram;
@@ -59,6 +57,8 @@
 import org.xml.sax.SAXException;
 import org.xml.sax.helpers.DefaultHandler;
 
+// TODO: Move to Diagram subsystem?
+
 /**
  * The PGML Parser. <p>
  * 
@@ -107,10 +107,9 @@
                 "org.argouml.gefext.ArgoFigRect");
         addTranslation("org.tigris.gef.presentation.FigRRect",
                 "org.argouml.gefext.ArgoFigRRect");
-        // Convert class reference to a string name when the class is removed
         addTranslation(
-                FigMNodeInstance.class.getName(),
-                FigNodeInstance.class.getName());
+                "org.argouml.uml.diagram.deployment.ui.FigMNodeInstance",
+                "org.argouml.uml.diagram.deployment.ui.FigNodeInstance");
     }
 
     /*
@@ -473,6 +472,9 @@
        
        Fig f = null;
        
+       // TODO: This low level parser shouldn't have a dependency on a specific
+       // activity diagram fig.  Whatever is special about FigPool needs to
+       // be represented by a core interface that we can look for. - tfm
         if (className.equals(FigPool.class.getName())) {
             f = new FigPool(bounds);
         } else {

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]