diff options
author | joey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071> | 2007-07-27 09:45:27 +0000 |
---|---|---|
committer | joey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071> | 2007-07-27 09:45:27 +0000 |
commit | 71cd4921c1b5ace031c3235d2dc99f813db289e3 (patch) | |
tree | 3d40c7184e038f2310e3ee2d15141605677ae971 | |
parent | 8f991679d2212887aaf6929eaed36b55e966325f (diff) |
web commit by HenrikBrixAndersen: This patch is still needed
-rw-r--r-- | doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn b/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn index 9eca41faf..0c679a910 100644 --- a/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn +++ b/doc/bugs/Problems_with_graphviz.pm_plug-in.mdwn @@ -5,12 +5,42 @@ The graphviz.pm plug-in currently attempts to read PNG data in UTF-8 mode, which > Ok, will remove the binmode IN then. [[done]] --[[Joey]] +>> Thanks --[[HenrikBrixAndersen]] + 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. +>> +>> Here is an updated patch againt ikiwiki-2.5: + + + --- 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 + @@ -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 (@) { #{{{ + + +>> --[[HenrikBrixAndersen]] + + The patch below fixes these two issues. --- graphviz.pm.orig Thu Jun 7 15:45:16 2007 |