Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 328407

Summary:	[Element Edit Service] Using edit service for delete in Class Diagram
Product:	[Modeling] Papyrus	Reporter:	Yann Tanguy <yann.tanguy>
Component:	Core	Assignee:	Patrick Tessier <Patrick.Tessier>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	sebastien.gerard
Version:	unspecified
Target Milestone:	---
Hardware:	PC
OS:	Windows 7
Whiteboard:
Bug Depends on:
Bug Blocks:	324966

Description Yann Tanguy

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

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

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

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

2010-11-21 16:45:27 EST

In r3282:
- Generalization item semantic edit policy corrected

Comment 5 Yann Tanguy

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

2013-07-05 12:20:28 EDT

This task is fixed