This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 113573

Summary: API: Add ConnectionWidget.routingPolicy property.
Product: platform Reporter: David Kaspar <dkaspar>
Component: GraphAssignee: issues@platform <issues>
Status: RESOLVED FIXED    
Severity: blocker Keywords: API_REVIEW_FAST
Priority: P3    
Version: 6.x   
Hardware: All   
OS: All   
Issue Type: DEFECT Exception Reporter:
Bug Depends on:    
Bug Blocks: 108296    
Attachments: Patch

Description David Kaspar 2007-08-23 09:00:46 UTC 
The property should define policy for path routing. There should be 4 values available:
1) Always route
2) Update end points only (router is not invoked at all)
3) Disable routing until end point is moved (router is temporarily disabled until end point is moved)
4) Disable routing (router is not never invoked)

This feature helps to ease management of control points changed by an user or graph-oriented layout.
E.g.:
1) AddRemoveControlPointAction and MoveControlPointAction could allow you to set 2. or 3. policy to a connection widget
with modified control points. Then the "user change" is not discarded and a source or a target is changed.
2) GraphLayout may render connection paths and similarly to above, it can use 2. or 3. policy on connection routed by
graph-oriented layout so the path is not discarded and overriden by connection-widget-router immediately.
Comment 1 David Kaspar 2007-08-23 10:05:47 UTC 
Created attachment 47150 [details]
Patch
Comment 2 David Kaspar 2007-08-23 10:09:06 UTC 
A patch has been attached. It contains:
1) ConnectionWidget.routingPolicy property and its logic is added
2) AddRemoveControlPointAction and MoveControlPointAction has additional "routingPolicy" parameter.
3) Documentation has new "Routing Policy" section.
4) "test.routing.ActionsWithRoutingPolicyTest" and "test.routing.RoutingPolicyTest" examples.
5) JavaDoc, api-changes, manifest, ...

Asking for fast-api-review.
Comment 3 David Kaspar 2007-08-31 09:26:22 UTC 
The patch has been committed to the CVS.