Bug 509763 - IBendableContentPart#setContentBendPoints() should be combined with attachTo-/detachFromContentAnchorage().
Summary: IBendableContentPart#setContentBendPoints() should be combined with attachTo-...
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF MVC (show other bugs)
Version: 1.0.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-29 10:00 EST by Alexander Nyßen CLA
Modified: 2016-12-29 10:00 EST (History)
0 users

See Also:

Attachments
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 Alexander Nyßen CLA 2016-12-29 10:00:34 EST 
When setting content bendpoints to an IBendableContentPart, in case the content anchorages changed, attachTo-/detachFromContentAnchorage needs to be called. 

Currently this is left to client code (EdgePart for instance implements the functionality), while this contract is not properly documented. An adopter asked for the intended contract in https://www.eclipse.org/forums/index.php/t/1083396/.

We could think of providing a default implementation within IBendableContentPart that delegates to attachTo/detachFromContentAnchorage if the content anchorages change, or we should properly document what is expected from the client. We could also impose the contract of calling both to the related transaction policies as an alternative.