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

Bob Tarling <[email protected]>
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: bobtarling
Date: 2009-12-04 04:17:34-0800
New Revision: 17596

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

Log:
Make consistent with other renderers - this fixes problem with comment edges - issue 5759

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/StateDiagramRenderer.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/StateDiagramRenderer.java?view=diff&pathrev=17596&r1=17595&r2=17596
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/StateDiagramRenderer.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/state/ui/StateDiagramRenderer.java	2009-12-04 04:17:34-0800
@@ -120,9 +120,27 @@
             newEdge = new FigEdgeNote(edge, settings); // TODO -> settings
         } 
         if (newEdge == null) {
-            LOG.debug("TODO: StateDiagramRenderer getFigEdgeFor");
-            return null;
+            throw new IllegalArgumentException(
+                    "Don't know how to create FigEdge for model type "
+                    + edge.getClass().getName());
         }
+        
+        setPorts(lay, newEdge);
+
+        assert newEdge != null : "There has been no FigEdge created";
+
+//        newEdge.setDiElement(
+//            GraphChangeAdapter.getInstance().createElement(gm, edge));
+
+        assert newEdge != null : "There has been no FigEdge created";
+        assert (newEdge.getDestFigNode() != null) 
+            : "The FigEdge has no dest node";
+        assert (newEdge.getDestPortFig() != null) 
+            : "The FigEdge has no dest port";
+        assert (newEdge.getSourceFigNode() != null) 
+            : "The FigEdge has no source node";
+        assert (newEdge.getSourcePortFig() != null) 
+            : "The FigEdge has no source port";
 
         lay.add(newEdge);
         return newEdge;

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

To unsubscribe from this discussion, e-mail: [[email protected]].
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.