Thu Dec 31 15:07:19 2015NHORNE [...] cpan.org - Ticket created
Subject:
Hyperlinks
It would be great if the URLs could be added to the nodes to the tree. I believe GraphViz supports that.
Fri Jan 08 14:42:47 2016bduggan [...] matatu.org - Correspondence added
On Thu Dec 31 15:07:19 2015, NHORNE wrote:
Show quoted text
> It would be great if the URLs could be added to the nodes to the tree.
> I believe GraphViz supports that.
There's some rudimentary support for this (see _person_node in lib/Tree/Family.pm) -- I'd be happy to incorporate any patches which improve this.
thanks
Brian
Fri Jan 08 14:42:47 2016The RT System itself - Status changed from 'new' to 'open'
Fri Jan 08 14:42:48 2016bduggan [...] matatu.org - Status changed from 'open' to 'resolved'
Fri Jan 08 20:27:06 2016NHORNE [...] cpan.org - Correspondence added
That nearly works (setting $Tree::Family::urlBase), but not quite. Instead of setting the hyperlink to "http://localhost/?id=XXX" where XXX is the ID in the Person's constructor, it actually sets XXX to the *name* of the person.
On a personal note, I'd have preferred to have "XXX.html", but I guess every person will want there own. What would be best would be to pass a URL into the T:F:P constructor.
Fri Jan 08 20:27:08 2016NHORNE [...] cpan.org - Status changed from 'resolved' to 'open'
Fri Jan 08 20:36:28 2016bduggan [...] matatu.org - Correspondence added
Subject:
Re: [rt.cpan.org #110822] Hyperlinks
Date:
Fri, 8 Jan 2016 20:36:10 -0500
To:
Nigel Horne via RT <bug-Tree-Family [...] rt.cpan.org>
From:
Brian Duggan <bduggan [...] matatu.org>
On Friday, January 8, Nigel Horne via RT wrote:
Show quoted text
> On a personal note, I'd have preferred to have "XXX.html", but I guess every person will want there own. What would be best would be to pass a URL into the T:F:P constructor.
Thanks, yes, I agree, that would be nice; much of this code
needs to be refactored to use attributes and better OO
practices in general.
Brian
RT for rt.cpan.org