This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 126779 - messages are still showing on diagram after associated lifeline is deleted
Summary: messages are still showing on diagram after associated lifeline is deleted
Status: VERIFIED FIXED
Alias: None
Product: uml
Classification: Unclassified
Component: Diagram Sequence (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: Sergey Petrov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-06 01:43 UTC by Peter Lam
Modified: 2008-02-16 01:08 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
messages from deleted lifelines (5.85 KB, image/png)
2008-02-06 01:44 UTC, Peter Lam 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Lam 2008-02-06 01:43:58 UTC 
- create 2 lifelines on the diagram
- create a sync and an async messages from 1 lifeline to the other
- delete one of the lines or both
result: all messages left hanging on diagram. See attached screenshot.
Comment 1 Peter Lam 2008-02-06 01:44:40 UTC 
Created attachment 56118 [details]
messages from deleted lifelines
Comment 2 Sergey Petrov 2008-02-06 06:19:35 UTC 
investigating, sqd now use common delete action which isn't suitable in this case
Comment 3 Sergey Petrov 2008-02-11 23:37:54 UTC 
Checking in nodes/sqd/LifelineWidget.java;
/cvs/uml/diagrams/src/org/netbeans/modules/uml/diagrams/nodes/sqd/Attic/LifelineWidget.java,v  <--  LifelineWidget.java
new revision: 1.1.2.27; previous revision: 1.1.2.26
done
Checking in actions/sqd/MessagesConnectProvider.java;
/cvs/uml/diagrams/src/org/netbeans/modules/uml/diagrams/actions/sqd/Attic/MessagesConnectProvider.java,v  <-- 
MessagesConnectProvider.java
new revision: 1.1.2.74; previous revision: 1.1.2.73
done
Checking in ArrangeMoveWithBumping.java;
/cvs/uml/diagrams/src/org/netbeans/modules/uml/diagrams/actions/sqd/Attic/ArrangeMoveWithBumping.java,v  <-- 
ArrangeMoveWithBumping.java
new revision: 1.1.2.25; previous revision: 1.1.2.24

fix in progress
Comment 4 Sergey Petrov 2008-02-12 10:20:24 UTC 
Checking in actions/sqd/ArrangeMoveWithBumping.java;
/cvs/uml/diagrams/src/org/netbeans/modules/uml/diagrams/actions/sqd/Attic/ArrangeMoveWithBumping.java,v  <-- 
ArrangeMoveWithBumping.java
new revision: 1.1.2.26; previous revision: 1.1.2.25
done
Checking in nodes/sqd/LifelineWidget.java;
/cvs/uml/diagrams/src/org/netbeans/modules/uml/diagrams/nodes/sqd/Attic/LifelineWidget.java,v  <--  LifelineWidget.java
new revision: 1.1.2.28; previous revision: 1.1.2.27
done
IDE: [12.02.08 12:56] Committing "src" finished

IDE:-------------------------------------------------
IDE: [12.02.08 13:01] Committing "CombinedFragmentWidget.java" started
Checking in CombinedFragmentWidget.java;
/cvs/uml/diagrams/src/org/netbeans/modules/uml/diagrams/nodes/sqd/Attic/CombinedFragmentWidget.java,v  <-- 
CombinedFragmentWidget.java
new revision: 1.1.2.41; previous revision: 1.1.2.40
done
Comment 5 Sergey Petrov 2008-02-12 20:06:45 UTC 
will change logic and delete only direct access messages instead of recusive deletion for nested messages
Comment 6 Sergey Petrov 2008-02-12 22:39:45 UTC 
--- LifelineWidget.java	12 Feb 2008 10:44:26 -0000	1.1.2.29
+++ LifelineWidget.java	12 Feb 2008 21:20:57 -0000	1.1.2.30
--- CombinedFragmentWidget.java	12 Feb 2008 10:44:25 -0000	1.1.2.42
+++ CombinedFragmentWidget.java	12 Feb 2008 22:08:35 -0000	1.1.2.43

fixed, lifeline and combined fragment deletion delete messages now.
working on messages deletion but it's a separate issue, of cause may be filed too
also separate issues are deletion from project tree, but not regressions
Comment 7 Peter Lam 2008-02-16 01:08:28 UTC 
verified fix. messages are now removed from diagram.