Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 518440 - Bug during edge label location computation
Summary: Bug during edge label location computation
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 3.1.0   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: 3.1.9   Edit
Assignee: Laurent Redor CLA
QA Contact: Laurent Redor CLA
URL:
Whiteboard:
Keywords:
Depends on: 517437 518073 518441 518870 518871
Blocks:
  Show dependency tree
 
Reported: 2017-06-19 05:34 EDT by Laurent Redor CLA
Modified: 2017-07-07 07:20 EDT (History)
0 users

See Also:


Attachments
Project for steps to reproduce compatible with Sirius 3.1.x (66.80 KB, application/x-zip-compressed)
2017-06-30 03:04 EDT, 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-06-19 05:34:03 EDT
+++ This bug was initially created as a clone of Bug #517437 +++

Since bug 465328, there are some labels computation problems. When moving an edge the label is sometimes moved very far from its edge. Currently 2 scenarios have been identified. The label is potentially not visible but the diagram has strange scrollbar without any visible reason.

I completed the extisting data for automatic tests of 465328 for the below scenarios.

Scenario 1:
* Import project "edgeLabelsMoveTest" from "edgeLabelsMoveTest.zip"
* Open the diagram "DiagramWithSmallObliqueEdge"
* Move class "L" to the bottom right corner of the diagram
* KO: The center label of the edge, "toMCenter", is moved around {-5700, -1900}; far from its edge.

Scenario 2:
* Import project "edgeLabelsMoveTest" from "edgeLabelsMoveTest.zip"
* Open the diagram "DiagWithNodeAndTreeLayout"
* Set the zoom level to 25% (to see all green logical functions)
* Select all the green logical functions
* Move them under the blue logical functions (you can you the scroll mouse to zoom out, to facilitate the move)
* Everything is OK.
* Open the diagram "DiagWithNodeAndTreeLayoutAndIconOnEdgeLabel"
* Set the zoom level to 25% (to see all green logical functions)
* Select all the green logical functions
* Move them under the blue logical functions (you can you the scroll mouse to zoom out, to facilitate the move)
* KO: A big horizontal scroll appears. This reveals that some edge labels, not visible in this kind of diagram, have "big" coordinates.
Comment 1 Eclipse Genie CLA 2017-06-19 10:03:01 EDT
New Gerrit change created: https://git.eclipse.org/r/99593
Comment 2 Eclipse Genie CLA 2017-06-19 10:03:03 EDT
New Gerrit change created: https://git.eclipse.org/r/99592
Comment 3 Eclipse Genie CLA 2017-06-19 10:03:04 EDT
New Gerrit change created: https://git.eclipse.org/r/99595
Comment 4 Eclipse Genie CLA 2017-06-19 10:03:05 EDT
New Gerrit change created: https://git.eclipse.org/r/99591
Comment 5 Eclipse Genie CLA 2017-06-19 10:03:06 EDT
New Gerrit change created: https://git.eclipse.org/r/99596
Comment 11 Laurent Redor CLA 2017-06-26 09:39:21 EDT
This bug can cause other problems concerning layout of diagram. Indeed, the very huge coordinates of label can lead to computing error caused by limit (MIN or MAX) of INTEGER that is reached.
The migration participant must at least moved the corrupted nodes to a more readable layout.
Comment 12 Eclipse Genie CLA 2017-06-27 05:12:18 EDT
New Gerrit change created: https://git.eclipse.org/r/100096
Comment 14 Laurent Redor CLA 2017-06-30 03:04:39 EDT
Created attachment 269140 [details]
Project for steps to reproduce compatible with Sirius 3.1.x
Comment 15 Laurent Redor CLA 2017-07-03 03:40:57 EDT
Verified with Sirius 3.1.9-S20170627-085227
Comment 16 Pierre-Charles David CLA 2017-07-07 07:20:08 EDT
Available in Sirius 3.1.9, see https://wiki.eclipse.org/Sirius/3.1.9 for details.