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

Bug 467508

Summary:	Can not delete connected element on diagram after resource refresh
Product:	[Modeling] Sirius	Reporter:	Minh Tu Ton That <minhtutonthat>
Component:	Diagram	Assignee:	Laurent Redor <laurent.redor>
Status:	CLOSED FIXED	QA Contact:	Belqassim Djafer <belqassim.djafer>
Severity:	major
Priority:	P3	CC:	belqassim.djafer, damien.filliat, laurent.fasani, laurent.redor, maxime.porhel, minhtutonthat
Version:	2.0.5	Keywords:	triaged
Target Milestone:	2.0.6
Hardware:	PC
OS:	Windows 7
See Also:	https://git.eclipse.org/r/48894 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=225fa65a5470d65f3355a931a1328103c342b4c6
Whiteboard:
Bug Depends on:
Bug Blocks:	466566, 470083, 470141

Description Minh Tu Ton That

2015-05-18 08:52:15 EDT

Steps to reproduce:
1. With Sirius, create a Family model,
2. Add a "Persons diagram",
3. Save,
4. Open the .aird file with a text editor and change a "name" attribute,
5. In Sirius, do F5 on the .aird file,
6. In the diagram, try to delete a connected element (a node linked with an edge)
==> Deletion is not possible, even when the diagram is reopened.

Comment 1 Minh Tu Ton That

2015-05-18 08:55:57 EDT

Tested in 2.0.5

Comment 2 Laurent Fasani

2015-05-22 11:27:47 EDT

reproduced on sirius 2.0.3
not reproductible in sirius 3.0

Comment 3 Laurent Redor

2015-05-28 03:23:37 EDT

A "git bisect" on master branch identify the commit [1] has "responsible" of the fix. This commit corresponds to the bug 427017 (remove the AirDCrossReferenceAdapter). So the goal of this commit is not linked to the current issue. A more complete analysis of this commit is needed.

[1] http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=94b51052cdab27eb399b7c7655c66fecb50a3648

Comment 4 Laurent Redor

2015-05-28 10:29:35 EDT

This is a regression, the scenario is OK on Sirius 1.0.x.

The problem appears with commit http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=5b294ef32d426ab47c5339379f6a293a8d75044d

Comment 5 Eclipse Genie

2015-05-28 12:38:59 EDT

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

Comment 6 Eclipse Genie

2015-06-09 09:17:25 EDT

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

Comment 7 Laurent Redor

2015-06-09 09:24:30 EDT

Fixed by above commit

Comment 8 Belqassim Djafer

2015-06-12 11:15:59 EDT

Verified with Sirius 2.0.6-S20150612-092529

Comment 9 Laurent Redor

2015-06-17 05:34:54 EDT

Available in Sirius 2.0.6.