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

Bug 462538

Summary: Control failure if several models in the modeling project
Product: [Modeling] Sirius Reporter: Steve Monnier <steve.monnier>
Component: CoreAssignee: Steve Monnier <steve.monnier>
Status: CLOSED FIXED QA Contact: Maxime Porhel <maxime.porhel>
Severity: normal    
Priority: P3 CC: maxime.porhel, pierre-charles.david
Version: 2.0.4Keywords: triaged
Target Milestone: 3.0.0M7   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/44516
https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=0b10df793d2eb41ae2b61a2d44d5a9500c2b28d6
Whiteboard:
Bug Depends on: 462336    
Bug Blocks:    

Description Steve Monnier CLA 2015-03-19 05:24:12 EDT
+++ This bug was initially created as a clone of Bug #462336 +++

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 Steve Monnier CLA 2015-03-19 05:27:50 EDT
The fix from https://bugs.eclipse.org/bugs/show_bug.cgi?id=462336 need to be ported on the master branch
Comment 2 Maxime Porhel CLA 2015-03-19 07:02:17 EDT
Steve, could you add a reproduction /step to reproduce to help the homologation ?
Comment 3 Steve Monnier CLA 2015-03-19 11:04:44 EDT
For validation, you can use the JUnit test sample which is in the gerrit review of https://bugs.eclipse.org/bugs/show_bug.cgi?id=462336 and check that both semantic model 448373.ecore and 448373b.ecore can be controlled and uncontrolled.
Comment 4 Eclipse Genie CLA 2015-03-24 13:57:35 EDT
New Gerrit change created: https://git.eclipse.org/r/44516
Comment 6 Steve Monnier CLA 2015-03-30 11:09:52 EDT
The regression detected by SiriusControlTest has been fixed
Comment 7 Steve Monnier CLA 2015-03-30 11:10:51 EDT
Fixed by commit 0b10df793d2eb41ae2b61a2d44d5a9500c2b28d6
Comment 8 Maxime Porhel CLA 2015-05-27 04:45:22 EDT
Validated on Sirius 3.0.0 RC1
Comment 9 Pierre-Charles David CLA 2015-06-24 11:17:22 EDT
Available in Sirius 3.0.0. See https://wiki.eclipse.org/Sirius/3.0.0.