Opened 13 years ago

Closed 13 years ago

#4409 closed defect (duplicate)

bug in plugin _launch method

Reported by: jholg Owned by: Peter Kropf
Priority: normal Component: GraphvizPlugin
Severity: normal Keywords:
Cc: Christian Boos Trac Release: 0.11


The check on the input in the _launch method is wrong, which leads to preview not working correctly on graphs (you can workaround it by reloading the previewed page, iirc). Should be changed to check on encoded_input instead.

The relevant 0.7.2 based diff portion is:

@@ -529,7 +532,7 @@ class Graphviz(Component):
         p = subprocess.Popen(encoded_cmd, stdin=subprocess.PIPE,
                              stdout=subprocess.PIPE, stderr=subprocess.PIPE)
-        if input:
+        if encoded_input:
         out =

This looks still wrong in the 0.7.4 source.

Cheers, Holger

Attachments (0)

Change History (2)

comment:1 Changed 13 years ago by Christian Boos

This was fixed some time ago in r4536.

Duplicate of #3912 - I should really find 5 minutes to remove that graphviz-0.7.4 folder...

comment:2 Changed 13 years ago by Christian Boos

Resolution: duplicate
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Peter Kropf.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

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

Note: See TracTickets for help on using tickets.