Bug 302064 - [EditPolicy] LayoutEditPolicies should check whether the content pane's layout manager is compatible.
Summary: [EditPolicy] LayoutEditPolicies should check whether the content pane's layou...
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy GEF (MVC) (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 302062
Blocks:
  Show dependency tree
 
Reported: 2010-02-07 11:39 EST by Alexander Nyßen CLA
Modified: 2010-11-04 17:22 EDT (History)
1 user (show)

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 2010-02-07 11:39:39 EST 
Build Identifier: 

Up to now, layout edit policies do not assert that the layout manager of their layout container (the host's content pane) is a compatible one. This is a likely pitfall that could easily be prevented by adding a respective assertion within setHost().

Just a comment: bug #302062 would allow to also perform such a check within ConstrainedLayoutEditPolicy be means of something like: Assert.isLegal(getLayoutContainer().getLayoutManager() instanceof ConstrainedLayoutManager);

Reproducible: Always