Bug 467508 - Can not delete connected element on diagram after resource refresh
Summary: Can not delete connected element on diagram after resource refresh
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 2.0.5   Edit
Hardware: PC Windows 7
: P3 major (vote)
Target Milestone: 2.0.6   Edit
Assignee: Laurent Redor CLA
QA Contact: Belqassim Djafer CLA
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks: 466566 470083 470141
  Show dependency tree
 
Reported: 2015-05-18 08:52 EDT by Minh Tu Ton That CLA
Modified: 2015-06-17 05:34 EDT (History)
6 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Minh Tu Ton That CLA 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 CLA 2015-05-18 08:55:57 EDT 
Tested in 2.0.5
Comment 2 Laurent Fasani CLA 2015-05-22 11:27:47 EDT 
reproduced on sirius 2.0.3
not reproductible in sirius 3.0
Comment 3 Laurent Redor CLA 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 CLA 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 CLA 2015-05-28 12:38:59 EDT 
New Gerrit change created: https://git.eclipse.org/r/48894
Comment 7 Laurent Redor CLA 2015-06-09 09:24:30 EDT 
Fixed by above commit
Comment 8 Belqassim Djafer CLA 2015-06-12 11:15:59 EDT 
Verified with Sirius 2.0.6-S20150612-092529
Comment 9 Laurent Redor CLA 2015-06-17 05:34:54 EDT 
Available in Sirius 2.0.6.