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

Bug 491216

Summary: Quick fix does not work with controlled resources
Product: [Modeling] Sirius Reporter: Maxime Porhel <maxime.porhel>
Component: DiagramAssignee: Laurent Redor <laurent.redor>
Status: CLOSED FIXED QA Contact: Florian Barbin <florian.barbin>
Severity: normal    
Priority: P3 CC: florian.barbin, laurent.redor, maxime.porhel, pierre-charles.david, stephane.thibaudeau
Version: 1.0.1Keywords: triaged
Target Milestone: 4.0.0M7   
Hardware: All   
OS: All   
See Also: https://git.eclipse.org/r/71086
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=dd2fe3b8a5f88f8e14c570cd6970f5c332487a32
https://git.eclipse.org/r/71324
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=06f8c8fee98961461201ca775212d6135923e3be
https://bugs.eclipse.org/bugs/show_bug.cgi?id=493975
Whiteboard:
Bug Depends on: 455244, 490907    
Bug Blocks:    

Description Maxime Porhel CLA 2016-04-07 05:12:49 EDT
+++ This bug was initially created as a clone of Bug #455244 +++

The same quick fix works on a "normal" resource, but when using it with a controlled resource the semantic modification is not done.

The problems ar e:
- no decorator on element with problem despite the problem actually shows up in Problems view
- quick fix does not update the semantic model


Steps to reproduce :

- import the VSM and test project
- in test project, open the diagram on the root EPackage
- if the name does not start with an uppercase letter, change it
- invoke "validate diagram"
- a warning shows up in problems view, a decorator is visible on diagram
- invoke the quick fix, the first letter is now in lowercase
=> OK

repeat the same steps on the last EPackage which is in a controlled resource :
=> warning shows up in problems view : OK
=> no decorator : KO
=> quick fix does not do anything : KO

Regards,
Stephane.
Comment 1 Eclipse Genie CLA 2016-04-20 12:05:24 EDT
New Gerrit change created: https://git.eclipse.org/r/71086
Comment 3 Eclipse Genie CLA 2016-04-25 08:49:54 EDT
New Gerrit change created: https://git.eclipse.org/r/71324
Comment 5 Florian Barbin CLA 2016-05-26 05:43:03 EDT
Verified on Sirius 4.0.0.RC1
Comment 6 Pierre-Charles David CLA 2016-06-24 08:03:11 EDT
Available in Sirius 4.0.0.