Page Index Toggle Pages: 1 Send TopicPrint
Normal Topic RouteDescriptor Constructor (Read 2230 times)
Julien_Dechene
YaBB Newbies
*
Offline


I love YaBB 1G - SP1!

Posts: 17
Joined: Jun 20th, 2011
RouteDescriptor Constructor
Jun 20th, 2011 at 7:14pm
Print Post  
Hi, sifting through the source code of Mindfusion.Diagramming I found out something strange that I am not sure if it is to be that way. In the RouteDescriptor class constructor (contained in DiagramLink.cs) the last two lines read :

this.originConnection = originConnection;
this.destConnection = originConnection;

Is this right ?? Intuitively I would expect it to be :

this.originConnection = originConnection;
[b]this.destConnection = destConnection ;[/b]

Is there a reason why it is like that ? Enlighten me please.

Thanks,

Julien Dechêne
  
Back to top
 
IP Logged
 
Stoyo
God Member
*****
Offline


MindFusion support

Posts: 13230
Joined: Jul 20th, 2005
Re: RouteDescriptor Constructor
Reply #1 - Jun 20th, 2011 at 7:39pm
Print Post  
Hi,

Yes, I think it should be set to destConnection. It seems to affect the feature where orthogonal bends in polyline links are chopped by GridRouter, and causes the bend at destination to not be removed. Anyway if you need to access the destination node through this descriptor from custom code, feel free to fix that in your copy.

Stoyan
  
Back to top
 
IP Logged
 
Julien_Dechene
YaBB Newbies
*
Offline


I love YaBB 1G - SP1!

Posts: 17
Joined: Jun 20th, 2011
Re: RouteDescriptor Constructor
Reply #2 - Jun 20th, 2011 at 8:42pm
Print Post  
Ok thanks for the response, I am using cascading linkstyle so from what you are saying (or from what I think you are saying) it won't affect the behavior of my links. Did I get this right ?

Thanks !
  
Back to top
 
IP Logged
 
Stoyo
God Member
*****
Offline


MindFusion support

Posts: 13230
Joined: Jul 20th, 2005
Re: RouteDescriptor Constructor
Reply #3 - Jun 21st, 2011 at 7:33am
Print Post  
Right, from what I can see the incorrect value is used for polyline links when SmartPolylineEnds and GridRouter are enabled. It doesn't affect the routing much even then, but of course we'll fix that for the next release.

Stoyan
  
Back to top
 
IP Logged
 
Page Index Toggle Pages: 1
Send TopicPrint