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

Bug 274808

Summary: [Sequence Diagram] Improve Combined Fragment functionality
Product: [Modeling] Papyrus Reporter: Gabriel Merin Cubero <gmerin>
Component: CoreAssignee: Thibault Landré <thibault.landre>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: raynerv, sebastien.gerard, yaang.liu
Version: 0.7.0Keywords: plan
Target Milestone: M2   
Hardware: All   
OS: All   
Whiteboard: UML 2.x Diagrams

Description Gabriel Merin Cubero CLA 2009-05-04 06:25:24 EDT
Improve Combined Fragment functionality in the Sequence Diagram.
Comment 1 Thibault Landré CLA 2009-10-16 04:42:54 EDT
CombinedFragment is improved. 

It managed the different InteractionOperator. It is also possible to create a ConsideredIgnoreFragment

Depending on the InteractionOperator, the user is grant the right to add one or more Interaction Operand. 
If the user try to add more than one InteractionOperand on a CombinedFragment that only accept one InteractionOperand (for exemple loop), then a popup shows up to explain why he can't create this new InteractionOperand. 

If the combinedFragment contains more than one InteractionOperand, the user is not allowed anymore to change the InteractionOperator.
Comment 2 Vincent Hémery CLA 2010-05-07 12:21:36 EDT
Validation has revealed new bugs or problems :

- 1/ The background color is initialized to black with default preferences, which is very confusing.
- 2/ When a Combined Fragment is added in an Interaction Operand of a CF, the new CF can not be distinguished from the Interaction Operand (and label is not common to the two CF). Hence, when trying to add new Interaction Operands to the new CF, they are added to the oldest one.
- 3/ When an Interaction Operand in a CF is deleted from the Model Explorer view, the figure is not removed.
- 4/ When trying to set Interaction Operator to Consider or Ignore, a wrong reason is invoked for failure :
 The message tells it is because there is "more than one Interaction Operand" instead of explaining the value is reserved for a ConsiderIgnoreFragment.
- 5/ When trying to set Interaction Operator when there is more than one Interaction Operand, the wrong reason is invoked. Message is switched with the case in 4/.
- 6/ No min value nor guard is created on a loop Combined Fragment
- 7/ When a lifeline is added to the property "covered", the figure is not enlarged. We must resize the figure to correct it.
- 8/ CF can overlap each over
- 9/ CF can not be drag n dropped.
- 10/ When deleting a CF from Model Explorer, no popup ask whether executions and messages must be delete. Executions are deleted , Messages are not.
- 11/ Delete from model does not work on CF

Same bugs exist on ConsiderIgnoreFragment.
Comment 3 Vincent Hémery CLA 2010-06-10 04:55:02 EDT
*** Bug 298773 has been marked as a duplicate of this bug. ***
Comment 4 Vincent Hémery CLA 2010-06-10 05:04:23 EDT
- 1/  solved.
Comment 5 Thibault Landré CLA 2010-07-02 05:52:28 EDT
-2/ solved
Comment 6 Thibault Landré CLA 2010-07-02 11:03:50 EDT
-3/ Solved, it was missing the correct policy (removeOrphanView).
Comment 7 Thibault Landré CLA 2010-07-02 11:11:29 EDT
- 4 / - 5/ Solved
Comment 8 Thibault Landré CLA 2010-07-05 15:12:45 EDT
-11/ Delete from model has been improved. 

It works now, and several improvement have also been made  on interaction operand "delete from model" action
Comment 9 Yang Liu CLA 2011-10-10 03:21:15 EDT
(In reply to comment #2)
> Validation has revealed new bugs or problems :
> 
> - 1/ The background color is initialized to black with default preferences,
> which is very confusing.
> - 2/ When a Combined Fragment is added in an Interaction Operand of a CF, the
> new CF can not be distinguished from the Interaction Operand (and label is not
> common to the two CF). Hence, when trying to add new Interaction Operands to
> the new CF, they are added to the oldest one.
> - 3/ When an Interaction Operand in a CF is deleted from the Model Explorer
> view, the figure is not removed.
> - 4/ When trying to set Interaction Operator to Consider or Ignore, a wrong
> reason is invoked for failure :
>  The message tells it is because there is "more than one Interaction Operand"
> instead of explaining the value is reserved for a ConsiderIgnoreFragment.
> - 5/ When trying to set Interaction Operator when there is more than one
> Interaction Operand, the wrong reason is invoked. Message is switched with the
> case in 4/.
> - 6/ No min value nor guard is created on a loop Combined Fragment
> - 7/ When a lifeline is added to the property "covered", the figure is not
> enlarged. We must resize the figure to correct it.
> - 8/ CF can overlap each over
> - 9/ CF can not be drag n dropped.
> - 10/ When deleting a CF from Model Explorer, no popup ask whether executions
> and messages must be delete. Executions are deleted , Messages are not.
> - 11/ Delete from model does not work on CF
> 
> Same bugs exist on ConsiderIgnoreFragment.

Will 9 be fixed in the future release? Any schedule?
This issue still active in 0.8.0
Thank you.
Comment 10 Sébastien Gérard CLA 2013-05-27 04:45:21 EDT
Closed as fixed in 0.10.0.