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

Bug 456080

Summary: False "The semantic model was not correctly updated" error with edge reconnection
Product: [Modeling] Sirius Reporter: Pierre-Charles David <pierre-charles.david>
Component: DiagramAssignee: Maxime Porhel <maxime.porhel>
Status: CLOSED FIXED QA Contact: Maxime Porhel <maxime.porhel>
Severity: normal    
Priority: P3 CC: esteban.dugueperoux, florian.barbin, jessy.mallet, maxime.porhel, stephane.thibaudeau
Version: 1.0.1Keywords: triaged
Target Milestone: 2.0.3   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on: 453036    
Bug Blocks:    

Description Pierre-Charles David CLA 2014-12-23 09:29:43 EST
+++ This bug was initially created as a clone of Bug #453036 +++

Eclipse Luna : 4.4.0
Sirius : 1.0.1
Ecore tools : 2.0.0.201406112035

When reconnecting, a warning "The semantic model was not correctly updated by the reconnect tool, the diagram part of the reconnect cannot be done" sometimes pops up in Error log. The reconnection is cancelled and the command stack is flushed (no Ctrl-Z anymore).

This behaviour is easily seen using ecore tools.

Steps to reproduce : 

1 - import the given test project
2 - open the class diagram
3 - try some reconnections
4 - sometimes the error pops up in error log and the reconnection is cancelled.

Cases in which I get the error on my computer :

Case 1 :
- from the starting point (reference betwenn EClass1 and EClass2)
- try to reconnect the Eclass1's side to have an edge between EClass3 and ECLass2 => error

Case 2 :
- from the starting point (EClass1 <-> Eclass2)
- do a few reconnections which work (EClass1 <-> Eclass3, EClass1 <-> EClass4, EClass2 <-> EClass4, EClass3 <-> EClass4)
- now try to do EClass3 <-> Eclass2 => Error.
Comment 1 Pierre-Charles David CLA 2014-12-23 09:30:16 EST
Cloned from bug #453036 to backport into Sirius 2.0.3.
Comment 2 Esteban DUGUEPEROUX CLA 2015-01-13 09:05:57 EST
Cherry picked through https://git.eclipse.org/r/#/c/39488/ and merged through 	eaf04b6ba9779c38358882681dc0535daa49082a and a4534ac9431db62747e63f6ccf842d9282943cda for the obselete test deletion.
Comment 3 Jessy Mallet CLA 2015-01-19 10:39:57 EST
Validated on Sirius Stable pre-2.0.3 (2.0.3-S20150116-162634)
Comment 4 Maxime Porhel CLA 2015-01-22 10:03:28 EST
Available in Sirius 2.0.3.