svn commit: r14327 - trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java

[email protected]
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: bobtarling
Date: 2008-04-11 18:43:22-0700
New Revision: 14327

Modified:
   trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java

Log:
Log if addition of a child Fig is rejected becaue it is the wrong type

Modified: trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java?view=diff&rev=14327&p1=trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java&p2=trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java&r1=14326&r2=14327
==============================================================================
--- trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java	(original)
+++ trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java	2008-04-11 18:43:22-0700
@@ -28,6 +28,7 @@
 import java.util.ArrayList;
 import java.util.List;
 
+import org.apache.log4j.Logger;
 import org.argouml.uml.diagram.sequence.MessageNode;
 import org.argouml.uml.diagram.ui.ArgoFigGroup;
 import org.tigris.gef.presentation.Fig;
@@ -42,6 +43,8 @@
 
     private static final long serialVersionUID = -7805833566723101923L;
     
+    private static final Logger LOG = Logger.getLogger(FigMessagePort.class);
+    
     private MessageNode node;
 
     /**
@@ -64,9 +67,13 @@
      * @see org.tigris.gef.presentation.FigGroup#addFig(org.tigris.gef.presentation.Fig)
      */
     public void addFig(Fig toAdd) {
-        if (toAdd instanceof FigLine && getFigs().size() == 0) {
-            toAdd.setVisible(false);
-            super.addFig(toAdd);
+        if (toAdd instanceof FigLine) {
+            if (getFigs().size() == 0) {
+                toAdd.setVisible(false);
+                super.addFig(toAdd);
+            }
+        } else {
+            LOG.error("Unexpect Fig " + toAdd);
         }
     }
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.