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

Bug 364804

Summary: [Sequence Diagram] - Deletion of combined fragments
Product: [Modeling] Papyrus Reporter: Chokri Mraidha <Chokri.Mraidha>
Component: CoreAssignee: Chokri Mraidha <Chokri.Mraidha>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: rschnekenburger, yves.yang
Version: 0.9.0   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Patch fixes the bug
none
Fix the bug of redo
none
bug fixes rschnekenburger: iplog+

Description Chokri Mraidha CLA 2011-11-25 05:23:12 EST 
Currently, deleting a combined fragments deletes all its contents. Sometimes, the user just wants to delete the combined fragment and keeping its content. It would be useful to give the possibility to not delete the combined fragment content. This would imply a refactoring of the model.
Comment 1 Yves YANG CLA 2012-04-20 09:05:55 EDT 
Created attachment 214304 [details]
Patch fixes the bug

Add a popup menu to select the choice between "Delete all" or "Keep content".

In case of "Keep content", Keep the nested combined fragments and contained action/behavior execution specifications.
Comment 2 Chokri Mraidha CLA 2012-04-23 10:54:21 EDT 
Changes committed on the trunk r.7965
Comment 3 Yves YANG CLA 2012-04-24 07:56:56 EDT 
Created attachment 214460 [details]
Fix the bug of redo
Comment 4 Yves YANG CLA 2012-04-24 07:57:35 EDT 
And also the location of nested combined fragment
Comment 5 Chokri Mraidha CLA 2012-04-24 08:26:13 EDT 
Changes committed on the trunk r.7968.
Comment 6 Yves YANG CLA 2012-06-07 07:39:54 EDT 
Created attachment 217013 [details]
bug fixes

We have found some errors with ConsiderIgnoreFrament when destroying ConsiderIgnoreFrament, popup menu does not display, and also bug relative to DnD bug 364696. The patch fixes both bugs.
Comment 7 Yves YANG CLA 2012-06-07 07:40:54 EDT 
Bug with ConsiderIgnoreFrament.
Comment 8 Vincent Lorenzo CLA 2012-06-12 09:57:33 EDT 
Comment on attachment 217013 [details]
bug fixes

Hello, I can't apply this patch, I get "Unmatched patch segment". 
Please could you redo it? Thank you.
Comment 9 Chokri Mraidha CLA 2012-06-12 14:32:18 EDT 
The patch has been committed on the trunk.
Comment 10 Sébastien Gérard CLA 2013-02-12 04:51:32 EST 
closed
Comment 11 Yves YANG CLA 2013-05-30 08:16:46 EDT 
a. I, Yves Yang, wrote 100% of the code I've provided.
b. I have the right to contribute the code to Eclipse.
c. I contribute the content under the EPL.
d. This contribution contains no Cryptography features.