Author: bobtarling
Date: 2010-04-29 01:18:48-0700
New Revision: 18355

Modified:
   
trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/UMLSequenceDiagram.java

Log:
Don't delete anything during fix. This may leave redundant elements around but 
safer it seems than deleting parts of the user model.

Modified: 
trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/UMLSequenceDiagram.java
Url: 
http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/UMLSequenceDiagram.java?view=diff&pathrev=18355&r1=18354&r2=18355
==============================================================================
--- 
trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/UMLSequenceDiagram.java
 (original)
+++ 
trunk/src/argouml-core-diagrams-sequence2/src/org/argouml/sequence2/diagram/UMLSequenceDiagram.java
 2010-04-29 01:18:48-0700
@@ -383,20 +383,6 @@
                         collabHelper.addMessage(correctInteraction, message);
                         Model.getCoreHelper().setNamespace(
                                 action, collaboration);
-                        // If this leaves the interaction empty then
-                        // delete it.
-                        if (facade.getMessages(interaction).isEmpty()) {
-                            LOG.warn("Deleting empty interaction "
-                                    + interaction);
-                            Model.getUmlFactory().delete(interaction);
-                            // If that in turn leaves the collaboration empty
-                            // then delete that also.
-                            if (context != null && 
facade.getOwnedElements(context).isEmpty()) {
-                                LOG.warn("Deleting empty collaboration "
-                                        + context);
-                                Model.getUmlFactory().delete(context);
-                            }
-                        }
                     }
                 } else if (f instanceof FigClassifierRole) {
                     final Object cr = f.getOwner();

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

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

Reply via email to