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

Bug 465303

Summary:	lineSemantic/columnSemantic variables in a CreateCellTool/LabelEditTool have no inferred type
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:	normal
Priority:	P3	CC:	laurent.redor, pierre-charles.david
Version:	unspecified	Keywords:	triaged
Target Milestone:	3.0.0M7
Hardware:	PC
OS:	Linux
See Also:	https://git.eclipse.org/r/46400 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=7212f6912f5d24e26b072fa0cb0f95ed907d2e21 https://git.eclipse.org/r/46425 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=794a1ae15bdc6fb50f136cff49a787e239f97a50
Whiteboard:
Bug Depends on:
Bug Blocks:	462490

Description Cedric Brun

2015-04-23 09:53:28 EDT

Steps to reproduce :
- import the source of org.eclipse.emf.ecoretools.design
- open ecore.odesign
- right-click, validate on the 'EModelElements to Doc Annotation' intersection mapping of the 'Documentation' matrix
=> you are getting validation errors : 
 "feature eAnnotations not found in EClass EObject" about the lineSemantic variable

whereas there is all the required information in the VSM to infer that lineSemantic will be either of type ecore.EStructuralFeature or ecore.EClassifier

Comment 1 Pierre-Charles David

2015-04-24 04:16:39 EDT

Reproduced on current master:
* Sirius: c4d0f60c12d04e1c5c7bef9b1a772e8aa24d6dde
* Ecore Tools: 470abd6b80fe22cc0050cd8422df26be484e22f4
* AQL: org.eclipse.acceleo.query_3.6.0.201504232036

Comment 2 Cedric Brun

2015-04-24 05:15:27 EDT

The problem also exists for columnSemantic, and also in the case of a LabelEditTool bound to an IntersectionMapping.

Comment 3 Eclipse Genie

2015-04-24 05:34:03 EDT

New Gerrit change created: https://git.eclipse.org/r/46400

Comment 4 Eclipse Genie

2015-04-24 08:09:29 EDT

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

Comment 5 Cedric Brun

2015-04-24 08:33:16 EDT

Fixed for M7

Comment 6 Eclipse Genie

2015-04-24 09:06:34 EDT

New Gerrit change created: https://git.eclipse.org/r/46425

Comment 7 Cedric Brun

2015-04-24 09:07:27 EDT

(In reply to Eclipse Genie from comment #6)
> New Gerrit change created: https://git.eclipse.org/r/46425

These are the corresponding JUnit-Plugin tests

Comment 8 Eclipse Genie

2015-04-24 11:24:37 EDT

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

Comment 9 Belqassim Djafer

2015-05-21 11:09:39 EDT

Verified with Sirius 3.0.0 RC1

Comment 10 Pierre-Charles David

2015-06-24 11:16:09 EDT

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