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

Bug 465954

Summary:	Provide variable types analysis in TableDescription and the corresponding tools
Product:	[Modeling] Sirius	Reporter:	Cedric Brun <cedric.brun>
Component:	Table	Assignee:	Project Inbox <sirius.table-inbox>
Status:	CLOSED FIXED	QA Contact:	Belqassim Djafer <belqassim.djafer>
Severity:	enhancement
Priority:	P3	CC:	laurent.fasani, maxime.porhel
Version:	unspecified	Keywords:	triaged
Target Milestone:	3.0.0
Hardware:	PC
OS:	Linux
See Also:	https://git.eclipse.org/r/46898 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=95512ea160b7a31b2a30ef957acc1518787dba00
Whiteboard:
Bug Depends on:	465561
Bug Blocks:

Description Cedric Brun

2015-04-30 11:09:18 EDT

This ticket is about providing type analysis for variables in the context of the table description tools. Notably in 
CreateLineTool
CreateColumnTool
DeleteLineTool
DeleteColumnTool

Comment 1 Maxime Porhel

2015-05-04 05:59:32 EDT

Implementer will have to provide a test case to validate the issue.

Comment 2 Maxime Porhel

2015-05-04 05:59:37 EDT

Implementer will have to provide a test case to validate the issue.

Comment 3 Cedric Brun

2015-05-06 05:30:48 EDT

(In reply to Maxime Porhel from comment #2)
> Implementer will have to provide a test case to validate the issue.

The patchset includes an update of model used by the JUnit tests to check for the covered situations.

https://git.eclipse.org/r/#/c/46898/3/plugins/org.eclipse.sirius.tests.junit/data/unit/vsm/validateVariableTypes.odesign

One can test manually by opening this .odesign and making sure the Tables (Create|Delete)(Line|Column) are valid.

Comment 4 Eclipse Genie

2015-05-07 04:08:58 EDT

Gerrit change https://git.eclipse.org/r/46898 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=95512ea160b7a31b2a30ef957acc1518787dba00

Comment 5 Cedric Brun

2015-05-07 12:05:36 EDT

Fixed for RC1

Comment 6 Belqassim Djafer

2015-05-26 03:55:26 EDT

Verified with Sirius 3.0.0 RC1

Comment 7 Pierre-Charles David

2015-06-24 11:12:44 EDT

Available in Sirius 3.0.0. See https://wiki.eclipse.org/Sirius/3.0.0.