Opened 10 years ago

Closed 10 years ago

#559 closed defect (fixed)

Invalid link expansion

Reported by: eblot Owned by: pkropf
Priority: high Component: GraphvizPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.9


The expand_wiki_links() method is far too restrictive.

For example, it prevents from using javascript-based URL, as the regular expression'href="(.*?)"', html_url)

silently fails, which least to a Python exception.

Attaching a patch to workaround this bug.

An typical case of failure happens if the URL in the dot graph is defined as follows:


Attachments (1)

expandlinks.patch (802 bytes) - added by eblot 10 years ago.
Allows Javascript URLs

Download all attachments as: .zip

Change History (4)

Changed 10 years ago by eblot

Allows Javascript URLs

comment:1 Changed 10 years ago by eblot

(sorry about not submitting a patch against the very last release of graphviz, trac-hacks is in trouble with https so I can't update my working copy through an http proxy. Nevertheless, the patch has been actually written against the latest zipped copy of graphviz)

comment:2 Changed 10 years ago by eblot

#560 to see why javascript URL is an important feature to support.

comment:3 Changed 10 years ago by pkropf

  • Resolution set to fixed
  • Status changed from new to closed

(In [1134]) Added support for javascript URLs. Fixes #559.

Add Comment

Modify Ticket

as closed The owner will remain pkropf.
The resolution will be deleted. Next status will be 'reopened'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.