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

Bug 454228

Summary: Validation of VSM detects errors on the use of newly defined variables
Product: [Modeling] Sirius Reporter: Melanie Bats <melanie.bats>
Component: DiagramAssignee: Project Inbox <sirius.diagram-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: belqassim.djafer, laurent.redor, pierre-charles.david
Version: 2.0.0Keywords: triaged
Target Milestone: 3.0.0M5   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
VSM validation unexpected error on create view variable usage
none
Label Edit tool variables validation issue none

Description Melanie Bats CLA 2014-12-05 06:05:30 EST
Created attachment 249187 [details]
VSM validation unexpected error on create view variable usage

If I define a new variable thanks to a create view and if I use this variable in a valid context, the VSM validation detects an unexpected error on this variable : "The current context does not contains variable named : newView". Also, the content assist does not provide the newly created variable.

This error occurs on "create view" and "create edge view" tools. It seems that it works fine with "for" and "create instance".

Attached you will find an odesign example to reproduce.
Comment 1 Melanie Bats CLA 2014-12-08 04:14:37 EST
Created attachment 249228 [details]
Label Edit tool variables validation issue
Comment 2 Melanie Bats CLA 2014-12-08 04:16:06 EST
(In reply to Melanie Bats from comment #1)
> Created attachment 249228 [details]
> Label Edit tool variables validation issue

It seems that the problem already exists in Table definition on Label Edit tool variableS.
Comment 3 Belqassim Djafer CLA 2014-12-11 09:02:14 EST
currently working on it
Comment 4 Belqassim Djafer CLA 2014-12-12 08:51:15 EST
I propose the fix : https://git.eclipse.org/r/#/c/38122/
Comment 5 Pierre-Charles David CLA 2015-01-26 05:39:01 EST
Fixed by 8647d279cfd45201156543d10199e8b233d55f06.
Comment 6 Belqassim Djafer CLA 2015-04-10 04:00:08 EDT
Verified on Sirius 3.0.0M6
Comment 7 Pierre-Charles David CLA 2015-06-24 11:15:14 EDT
Available in Sirius 3.0.0. See https://wiki.eclipse.org/Sirius/3.0.0.