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

Bug 462336

Summary:	Control failure if several models in the modeling project
Product:	[Modeling] Sirius	Reporter:	Steve Monnier <steve.monnier>
Component:	Core	Assignee:	Project inbox <sirius.core-inbox>
Status:	CLOSED FIXED	QA Contact:	Belqassim Djafer <belqassim.djafer>
Severity:	normal
Priority:	P3	CC:	belqassim.djafer, laurent.redor, maxime.porhel
Version:	2.0.4	Keywords:	triaged
Target Milestone:	2.0.5
Hardware:	PC
OS:	Windows 7
See Also:	https://git.eclipse.org/r/44030 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=e5fc87bca5f9677ba62c8569ecb67eff7bd95241 https://git.eclipse.org/r/44559 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=23e88de105eb0e65627b369794407be9df3c24c1
Whiteboard:
Bug Depends on:
Bug Blocks:	462538

Description Steve Monnier

2015-03-17 06:03:28 EDT

A modeling project can have several models (several models tag in the aird model). However, in the execution of the control some tests are done on the "main" model. This causes failure to control element in a model that is not the first in the list of the session semantic models.

Comment 1 Eclipse Genie

2015-03-17 13:27:52 EDT

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

Comment 2 Eclipse Genie

2015-03-19 05:21:23 EDT

Gerrit change https://git.eclipse.org/r/44030 was merged to [v2.0.x].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=e5fc87bca5f9677ba62c8569ecb67eff7bd95241

Comment 3 Steve Monnier

2015-03-19 05:23:48 EDT

For validation, you can use the JUnit test sample which is in the gerrit review and check that both semantic model 448373.ecore and 448373b.ecore can be controlled and uncontrolled.

Comment 4 Maxime Porhel

2015-03-19 07:02:07 EDT

Steve, could you add a reproduction /step to reproduce to help the homologation ?

Comment 5 Maxime Porhel

2015-03-19 11:22:19 EDT

Reproduction case attached by Steve.

Comment 6 Steve Monnier

2015-03-23 12:55:59 EDT

Reopened because the fix causes issues in SiriusControlTest

Comment 7 Eclipse Genie

2015-03-25 05:03:02 EDT

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

Comment 8 Eclipse Genie

2015-03-30 08:35:39 EDT

Gerrit change https://git.eclipse.org/r/44559 was merged to [v2.0.x].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=23e88de105eb0e65627b369794407be9df3c24c1

Comment 9 Steve Monnier

2015-03-30 08:37:00 EDT

Fix by commit 23e88de105. Note that the gerrit verify was -1 because the mars target platform needs to be updated on the sirius 2.0.x branch

Comment 10 Belqassim Djafer

2015-03-31 03:52:39 EDT

Verified on Sirius 2.0.5 [1]

[1] http://download.eclipse.org/sirius/updates/stable/2.0.5-S20150331-000124/luna

Comment 11 Laurent Redor

2015-03-31 10:20:59 EDT

Available in Sirius 2.0.5.