Bug 445324 - [LinksLF] Limited rectilinear routing with Snap To Shapes option activated
Summary: [LinksLF] Limited rectilinear routing with Snap To Shapes option activated
Status: NEW
Alias: None
Product: Papyrus
Classification: Modeling
Component: Diagram (show other bugs)
Version: 1.0.0   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 442157
  Show dependency tree
 
Reported: 2014-09-29 05:54 EDT by Sebastien GABEL CLA
Modified: 2017-09-08 10:54 EDT (History)
3 users (show)

See Also:

Attachments
Rectilinear routing in Papyrus 1.0.0 (18.86 KB, image/jpeg)
2014-09-29 05:54 EDT, Sebastien GABEL CLA 		no flags Details
NPE trace (2.62 KB, text/plain)
2014-09-29 09:35 EDT, Sebastien GABEL CLA 		no flags Details
Show Obsolete (1) 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 Sebastien GABEL CLA 2014-09-29 05:54:22 EDT 
Created attachment 247436 [details]
Rectilinear routing in Papyrus 1.0.0

Found on dev branch bugs/luna/anchors.

When an edge has a rectilinear routing, it is not possible to move the last segment anymore to turn around the figure.
At least one, sometimes more, side(s) of the figure is/are inaccessible.
See attached screenshot that illustrates the former behavior.
Comment 1 Sebastien GABEL CLA 2014-09-29 06:39:05 EDT 
I was certainly on the bad dev branch.
Actually work as before.
Comment 2 Sebastien GABEL CLA 2014-09-29 09:35:40 EDT 
Created attachment 247442 [details]
NPE trace
Comment 3 Sebastien GABEL CLA 2014-09-29 09:37:02 EDT 
I re-open this bug since I managed to reproduce it with the 'Snap To Grid' option activated.

When this option is checked, it becomes impossible to access to certain sides of the figure (Cf. description below) moving the latest created segment.

NPEs are raised (Cf. attachment)
Comment 4 Sebastien GABEL CLA 2014-09-29 10:27:04 EDT 
After a deeper investigation, the responsible is the 'Snap To Shapes' option and not the 'Snap To Grid' option as said above.
Comment 5 Vincent Lorenzo CLA 2014-10-23 10:56:48 EDT 
After discussing with Sebastien GABEL, we decided than 'Snap to Shape' feature must be ignored for Link elements (anchors, bendpoints and LineSeg)
Comment 6 Sebastien GABEL CLA 2014-12-01 08:45:27 EST 
I've picked up the last commits from the dev branch (1st december)
Good news, I don't manage to reproduce this problem anymore. Is this bug fixed ?
Comment 7 Michael Golubev CLA 2014-12-01 09:23:38 EST 
(In reply to Sebastien GABEL from comment #6)
> Is this bug fixed ?

Well. exception is fixed long time ago. 
This bug is open only due to comment 5, that is due to the difference between a) and b) below: 
- a) bendpoint operations (moving the inner bendpoint or recti-segment) respect Snap To Shape's in addition to Snap to Grid 
- b) while anchor operations (moving existing anchor or creating anchor for new links) only respect Snap To Grid and does NOT respect Snap to Shape. 

With respect to comment 5 only a) is a bug. I see it as a minor one and it hopefully can be extracted from this bug and re-scheduled for Mars. What do you think?
Comment 8 Sebastien GABEL CLA 2014-12-01 11:09:59 EST 
(In reply to Michael Golubev from comment #7)
> What do you think?

I think it's totally acceptable and the missing point can be re-scheduled as you propose.