* Fix problems with previewing in the graphviz plugin. Thanks,
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sun, 29 Jul 2007 22:34:42 +0000 (22:34 +0000)
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sun, 29 Jul 2007 22:34:42 +0000 (22:34 +0000)
  HenrikBrixAndersen.

IkiWiki/Plugin/graphviz.pm
debian/changelog
doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn

index ee97c71d1d9e109fca01673766007e2f091ba438..fb32392279cbff0e0e4aa7ad434ce9b3a6acc03d 100644 (file)
@@ -69,7 +69,12 @@ sub render_graph (\%) { #{{{
                }
        }
 
                }
        }
 
-       return "<img src=\"".urlto($dest, $params{page})."\" />\n";
+       if ($params{preview}) {
+               return "<img src=\"".urlto($dest, "")."\" />\n";
+       }
+       else {
+               return "<img src=\"".urlto($dest, $params{page})."\" />\n";
+       }
 } #}}}
 
 sub graph (@) { #{{{
 } #}}}
 
 sub graph (@) { #{{{
index 17412d9d0e16476fe5b680a8e83568181b375d30..ae575b916424dcc2c7d49cb6709a71e3f19cd6cf 100644 (file)
@@ -2,8 +2,10 @@ ikiwiki (2.6) UNRELEASED; urgency=low
 
   * Various minor bug fixes for silly mistakes in the code, thanks to the
     various reviewers.
 
   * Various minor bug fixes for silly mistakes in the code, thanks to the
     various reviewers.
+  * Fix problems with previewing in the graphviz plugin. Thanks,
+    HenrikBrixAndersen.
 
 
- -- Joey Hess <joeyh@debian.org>  Sat, 28 Jul 2007 17:01:35 -0400
+ -- Joey Hess <joeyh@debian.org>  Sun, 29 Jul 2007 18:34:13 -0400
 
 ikiwiki (2.5) unstable; urgency=low
 
 
 ikiwiki (2.5) unstable; urgency=low
 
index 1ec3e32bceb5f2cd064ff25e5d4553878b221759..9a26e505af2df9f4b9a0dd47e5d57c4da31a76f8 100644 (file)
@@ -9,21 +9,9 @@ The graphviz.pm plug-in currently attempts to read PNG data in UTF-8 mode, which
 
 It also generates image URLs relative to the page being rendered, which means the URLs wont work when previewing a graph from the CGI script.
 
 
 It also generates image URLs relative to the page being rendered, which means the URLs wont work when previewing a graph from the CGI script.
 
-> You seem to be using an old version of ikiwiki, these preview isues have
-> been fixed for some time, and the code doesn't look like what you patch
-> in your second hunk. --[[Joey]]
-
->> I have just tested ikiwiki-2.5. The code in question still looks
->> the same, and the patch is still needed for making the graphviz
->> plug-in work in preview mode here.
-
->>> No, the relevant code is just above the bit you patched. Notice that in
->>> preview mode, it returns an img tag with the graphviz drawing embedded
->>> in it, using base64 encoding. Unless you can show me why that doesn't
->>> work, I still think this bug is done. --[[Joey]]
-
 >> Here is an updated patch againt ikiwiki-2.5:
 
 >> Here is an updated patch againt ikiwiki-2.5:
 
+>>> [[Applied|done]], thanks. --[[Joey]]
 
     --- IkiWiki/Plugin/graphviz.pm.orig        2007-07-27 11:35:05.000000000 +0200
     +++ IkiWiki/Plugin/graphviz.pm     2007-07-27 11:36:02.000000000 +0200
 
     --- IkiWiki/Plugin/graphviz.pm.orig        2007-07-27 11:35:05.000000000 +0200
     +++ IkiWiki/Plugin/graphviz.pm     2007-07-27 11:36:02.000000000 +0200