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

Bug 518441

Summary:	Wrong first or last GMF bendpoints in case of Straight routing style with only one segment
Product:	[Modeling] Sirius	Reporter:	Laurent Redor <laurent.redor>
Component:	Diagram	Assignee:	Laurent Redor <laurent.redor>
Status:	CLOSED FIXED	QA Contact:	Guillaume Doux <guillaume.doux>
Severity:	normal
Priority:	P3	CC:	guillaume.doux, pierre-charles.david
Version:	3.1.0	Keywords:	triaged
Target Milestone:	3.1.9
Hardware:	All
OS:	All
See Also:	https://git.eclipse.org/r/99594 https://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=b173984e0e8b5768e167c36306a79e361971e1c6
Whiteboard:
Bug Depends on:	518073, 518871
Bug Blocks:	517437, 518440, 518870

Description Laurent Redor

2017-06-19 05:40:37 EDT

+++ This bug was initially created as a clone of Bug #518073 +++

In case of an edge with a Straight routing style and with only one segment, the GMF bendpoints are false if the edge is "reverted". For example, in the case of "1-beforeMovingEdge.png", the GMF bendpoints are correct (same as draw2d). But if the node C2 is moved on left of C1, the GMF bendpoints are wrong (see "2-afterMovingEdge.png").
These false bendpoints can be problematic for other calculations (like label location).

Steps to reproduce:
* Open the diagram "newDiag3" of the project "SampleProject" (from SampleProject.zip)
* Open the "Sirius Debugging View" (from plugin org.eclipse.sirius.ui.debug)
* Select the edge between C1 and C2
* In the "Sirius Debugging View", the GMF computed points are the same as Draw2D points.
* Move C2 to the left of C1
* KO: In the "Sirius Debugging View", the GMF computed points are not the same as Draw2D points.

Comment 1 Eclipse Genie

2017-06-19 10:03:00 EDT

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

Comment 2 Eclipse Genie

2017-06-20 10:40:43 EDT

Gerrit change https://git.eclipse.org/r/99594 was merged to [v3.1.x].
Commit: http://git.eclipse.org/c/sirius/org.eclipse.sirius.git/commit/?id=b173984e0e8b5768e167c36306a79e361971e1c6

Comment 3 Guillaume Doux

2017-06-30 11:13:49 EDT

Validated with Sirius 3.1.9-S20170627-085227

Comment 4 Pierre-Charles David

2017-07-07 07:20:09 EDT

Available in Sirius 3.1.9, see https://wiki.eclipse.org/Sirius/3.1.9 for details.