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

Bug 508613

Summary: Decorator with precondition does not reappear properly in manual refresh
Product: [Modeling] Sirius Reporter: Steve Monnier <steve.monnier>
Component: DiagramAssignee: Project Inbox <sirius.diagram-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: jessy.mallet, maxime.porhel, pierre-charles.david
Version: 4.1.1Keywords: triaged
Target Milestone: 5.0.0   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/86389
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=ff8d58acdf9a21293033b8c92617c68ee1801032
Whiteboard:
Attachments:
Description Flags
decorator_issue_design_and_sample none

Description Steve Monnier CLA 2016-12-02 11:55:25 EST
Created attachment 265703 [details]
decorator_issue_design_and_sample

Have a diagram in manual refresh, with a decorator on diagram elements that is displayed depending on a precondition. When the precondition is false, the decorator disappear properly. Then change the diagram or semantic element to have the precondition true. The decorator does not reappear by itself. You have to refresh the editor in order to see it.

Using the given sample (showing a decorator on EClasses with a name starting with the letter 'c'), follow this scenario:
- import both project in your workspace. 
- Open the test_decorator project and then its diagram 'new Bug_Decorator'. 
- A class named 'c1' should be display with a decorator (a yellow car).
- Rename the eclass to 'a1'. The decorator disappear. OK
- Rename the eclass to 'c1'. The decorator does not sappear. KO
Note that clicking on the refresh button shows the decorator.
Comment 1 Maxime Porhel CLA 2016-12-05 11:41:02 EST
Thanks for the detailed bug report Steve. 
Feel free to continue to work on the issue. 
The identified target is currently Sirius 5.0.
Comment 2 Eclipse Genie CLA 2016-12-05 12:22:35 EST
New Gerrit change created: https://git.eclipse.org/r/86389
Comment 4 Jessy Mallet CLA 2017-05-18 12:10:43 EDT
Validated with Sirius 5.0.0.201705151305
Comment 5 Pierre-Charles David CLA 2017-05-19 03:08:33 EDT
Verified by Jessy.
Comment 6 Pierre-Charles David CLA 2017-06-29 03:32:52 EDT
Available in Sirius 5.0.0, see https://wiki.eclipse.org/Sirius/5.0.0 for details.