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

Bug 471814

Summary:	Reconnect source point of oblique edge moves the target point of edge
Product:	[Modeling] Sirius	Reporter:	Laurent Redor <laurent.redor>
Component:	Diagram	Assignee:	Florian Barbin <florian.barbin>
Status:	CLOSED FIXED	QA Contact:	Jessy Mallet <jessy.mallet>
Severity:	normal
Priority:	P3	CC:	belqassim.djafer, florian.barbin, jessy.mallet, maxime.porhel, pierre-charles.david
Version:	3.0.0	Keywords:	triaged
Target Milestone:	3.1.0
Hardware:	PC
OS:	Linux
See Also:	https://git.eclipse.org/r/51337 https://bugs.eclipse.org/bugs/show_bug.cgi?id=467663 https://git.eclipse.org/r/55515 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=f7332b46d093fb6b109450662c84cd0657882c55
Whiteboard:

Description Laurent Redor

2015-07-03 10:17:49 EDT

When a target point, of an oblique edge, is reconnected, the source point is stable. But when the source point, of an oblique edge, is reconnected, the target point is also moved. The edge is considered as new edge and the target points in direction of the center of the targeted node.

Comment 1 Eclipse Genie

2015-07-03 10:20:20 EDT

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

Comment 2 Laurent Redor

2015-07-03 10:21:07 EDT

The test in gerrit of comment 1 reveals the problem.

Comment 3 Maxime Porhel

2015-07-06 05:49:22 EDT

This is also true when there are bendpoints on the edge.

Comment 4 Maxime Porhel

2015-07-06 05:51:18 EDT

IT could be nice to have a behavior close to https://www.eclipse.org/sirius/doc/images/moveNode-edgeImpact-changedBehavior.gif

Comment 5 Belqassim Djafer

2015-07-09 11:01:59 EDT

This issue also exists in Sirius 2.0 and 1.0.1

Comment 6 Florian Barbin

2015-09-09 03:59:55 EDT

The issue also exists when reconnecting the target. That is because when reconnecting an edge a new one is created. We kept the bendpoints but not the source or target anchor. A proposed patch to fix this issue: https://git.eclipse.org/r/55515

Comment 7 Eclipse Genie

2015-09-10 03:13:14 EDT

Gerrit change https://git.eclipse.org/r/55515 was merged to [master].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=f7332b46d093fb6b109450662c84cd0657882c55

Comment 8 Florian Barbin

2015-09-10 03:18:10 EDT

Fixed by f7332b46d093fb6b109450662c84cd0657882c55

Comment 9 Jessy Mallet

2015-09-28 12:31:00 EDT

Validated with Sirius 3.1.0.201509241504

Comment 10 Pierre-Charles David

2015-10-16 09:12:30 EDT

Available in Sirius 3.1.0.