Bug 510199 - Wrong GMF bendpoints after edge, with tree layout, creation
Summary: Wrong GMF bendpoints after edge, with tree layout, creation
Status: NEW
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 4.1.0   Edit
Hardware: PC Windows NT
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2017-01-10 11:10 EST by Laurent Redor CLA
Modified: 2022-07-20 05:41 EDT (History)
1 user (show)

See Also:


Attachments
EcoreProject (2.14 KB, application/x-zip-compressed)
2017-01-10 11:10 EST, Laurent Redor CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Laurent Redor CLA 2017-01-10 11:10:02 EST
Created attachment 266217 [details]
EcoreProject

After the creation of an edge with a tree layout: 
* the source vectors seems OK for first and last point.
* the target vectors seems wrong for first and last point.

Steps to reproduce:
* Import the project EcoreProject (this project uses the VSM of org.eclipse.sirius.sample.ecore.design)
* Open the diagram "root package entities"
* Check the rulers & grid properties:
    * Ruler Units: Pixels
    * Grid Spacing: 25
    * Snap To Grid: true
    * Snap To Shapes: true
* Create a relation of kind SuperType between center of C2 and center of C1
* Save
* Open the aird in text editor
* The bendpoins of the edge must be something like points="[0, 0, 49, 75]$[-50, -52, -1, 23]" and not like points="[0, 0, 50, 52]$[-50, -52, 0, 0]"
* You can also use the "Sirius Debugging View" to get the edge data (instead of looking in the aird text file).

Ideally, the bendpoints should contain 4 points instead of two as this edge uses a tree layout. But this is not the main goal of this issue.

This problem has been discovered during the analysis of bug 495659.
Comment 1 Eclipse Genie CLA 2017-01-10 11:43:06 EST
New Gerrit change created: https://git.eclipse.org/r/88387
Comment 2 Maxime Porhel CLA 2017-01-19 07:49:05 EST
Thank your for the detail bug report and provided draft patch set. 
Nevertheless it is not yet in the scope of a future release.

Regards
Comment 3 Laurent Redor CLA 2022-07-20 05:41:26 EDT
The problem is always here. The gerrit has just been rebase (without new analysis). There are conflicts so maybe the rebase is not OK and it is strange because with the steps to reproduce, in debug the modified code is not used...