Bug 535649 - Wrong representation name displayed in "Link Note" if targeted representation is deleted
Summary: Wrong representation name displayed in "Link Note" if targeted representation...
Status: CLOSED DUPLICATE of bug 535648
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 5.1.2   Edit
Hardware: PC Windows 10
: P3 minor (vote)
Target Milestone: 5.1.4   Edit
Assignee: Laurent Redor CLA
QA Contact: Laurent Redor CLA
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2018-06-07 08:15 EDT by Laurent Redor CLA
Modified: 2019-01-24 04:54 EST (History)
2 users (show)

See Also:

Attachments
SimpleModelingProject-5.1.2.zip (3.68 KB, application/x-zip-compressed)
2018-06-07 08:15 EDT, Laurent Redor CLA 		no flags Details
View All 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 Laurent Redor CLA 2018-06-07 08:15:48 EDT 
Created attachment 274369 [details]
SimpleModelingProject-5.1.2.zip

Steps to reproduce
* Import SimpleModelingProject (from SimpleModelingProject-5.1.2.zip)
* Set preferences "Do refresh on representation opening" and "Automatic refresh" to false
* Open diagram "p1Diagram"
* Delete class "C11FromP1" (use "Delete From Model" action of the tabbar)
* Save
* Open diagram "p2Diagram"
* OK: The node representing class "C11FromP1" is always here but with a red cross. This is expected as we are in manual refresh.
* Close diagram "p2Diagram"
* Delete diagram "p1Diagram"
* Open diagram "p2Diagram"
* KO: The Link Note pointing to the deleted diagram displays "p2Diagram" instead of "p1Diagram" or maybe a red cross (see bug 535648 for that). This bug is here to focus on the wrong name displayed.
Comment 1 Laurent Redor CLA 2018-06-07 09:05:44 EDT 
The cause of this problem is probably bug 412078 or linked to it.
This problem does not exist if Sirius is used with "team" as the Sirius CDO Native version of ViewImpl contains an adaptation to handle this case.
Comment 2 Felix Dorner CLA 2018-07-16 06:30:32 EDT 
I think we should first fix 535648 and get a fix for this one for free.
Comment 3 Felix Dorner CLA 2018-09-27 08:03:57 EDT 
Hi, (In reply to Laurent Redor from comment #1)
> The cause of this problem is probably bug 412078 or linked to it.
> This problem does not exist if Sirius is used with "team" as the Sirius CDO
> Native version of ViewImpl contains an adaptation to handle this case.

I think I can exploit the implementation of ViewImpl for the other bug, but I would need to see the Sirius CDO native implementation for ViewImpl to make sure it works also for team. You can send it by mail if you want.
Comment 4 Laurent Redor CLA 2018-10-11 09:47:09 EDT 
This bug is indirectly fixed through the bug 535648.

*** This bug has been marked as a duplicate of bug 535648 ***
Comment 5 Laurent Redor CLA 2018-10-11 09:47:34 EDT 
No validation needed
Comment 6 Pierre-Charles David CLA 2019-01-24 04:54:05 EST 
Available in Sirius 5.1.4, see https://wiki.eclipse.org/Sirius/5.1.4 for details.