Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328407 - [Element Edit Service] Using edit service for delete in Class Diagram
Summary: [Element Edit Service] Using edit service for delete in Class Diagram
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Patrick Tessier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 324966
  Show dependency tree
 
Reported: 2010-10-21 16:24 EDT by Yann Tanguy CLA
Modified: 2013-07-05 12:20 EDT (History)
1 user (show)

See Also:

Attachments
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 Yann Tanguy CLA 2010-10-21 16:24:29 EDT 
The delete command used by the contextual menu on GraphicalEditParts in Class Diagram should use the delete command provided by the Edit Service so that the deletion of an element destroy not only the current view, but any view referencing the deleted element.

Patrick, can you plan to spend some time on this ?
1. make some not trivial tests on Class diagram with the delete command from the model explorer
2. for element the are normally deleted, refer their edit part with a "EditPartUsingDeleteService" node and re-generate the diagram
3. Let me know if some element deletion are not correctly handled by the edit service in Class diagram
Comment 1 Patrick Tessier CLA 2010-10-27 10:18:33 EDT 
the step 1 and 2 have been done.
before close this task, I prefer let some days to continue the test.
Done in the revision 3035
Comment 2 Yann Tanguy CLA 2010-11-04 15:22:46 EDT 
In r3122:
- Diagram specific advice added in order to clean inconsistent views during Generalization re-orient action (remove no more inherited member).
Comment 3 Yann Tanguy CLA 2010-11-21 16:05:47 EST 
In r3280:
- Bug fix for Generalization deletion (loop if Generalization cycle exists)
- Deprecated code removed
Comment 4 Yann Tanguy CLA 2010-11-21 16:45:27 EST 
In r3282:
- Generalization item semantic edit policy corrected
Comment 5 Yann Tanguy CLA 2010-11-29 05:36:36 EST 
(In reply to comment #3)
> In r3280:
> - Bug fix for Generalization deletion (loop if Generalization cycle exists)
> - Deprecated code removed
In r3375:
- NPE could occur if the model is not well formed.
Comment 6 Camille Letavernier CLA 2013-07-05 12:20:28 EDT 
This task is fixed