X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/7ae2deacdf5ebf7274da8c77daf42b43aca985c0..ced9e33776f74d1190a104dcaf839229ba204137:/IkiWiki/Plugin/sparkline.pm?ds=sidebyside

diff --git a/IkiWiki/Plugin/sparkline.pm b/IkiWiki/Plugin/sparkline.pm
index a44e06252..fb4849492 100644
--- a/IkiWiki/Plugin/sparkline.pm
+++ b/IkiWiki/Plugin/sparkline.pm
@@ -3,7 +3,7 @@ package IkiWiki::Plugin::sparkline;
 
 use warnings;
 use strict;
-use IkiWiki;
+use IkiWiki 3.00;
 use IPC::Open2;
 
 my $match_num=qr/[-+]?[0-9]+(?:\.[0-9]+)?/;
@@ -14,11 +14,20 @@ my %locmap=(
 	left => 'TEXT_LEFT',
 );
 
-sub import { #{{{
+sub import {
+	hook(type => "getsetup", id => "sparkline", call => \&getsetup);
 	hook(type => "preprocess", id => "sparkline", call => \&preprocess);
-} # }}}
+}
 
-sub preprocess (@) { #{{{
+sub getsetup () {
+	return
+		plugin => {
+			safe => 1,
+			rebuild => undef,
+		},
+}
+
+sub preprocess (@) {
 	my %params=@_;
 
 	my $php;
@@ -60,13 +69,13 @@ sub preprocess (@) { #{{{
 			}
 		}
 		elsif (! length $value) {
-			return "[[".gettext("sparkline parse error")." \"$key\"]]";
+			error gettext("parse error")." \"$key\"";
 		}
 		elsif ($key eq 'featurepoint') {
 			my ($x, $y, $color, $diameter, $text, $location)=
 				split(/\s*,\s*/, $value);
 			if (! defined $diameter || $diameter < 0) {
-				return "[[".gettext("sparkline bad featurepoint diameter")."]]";
+				error gettext("invalid featurepoint diameter");
 			}
 			$x=int($x);
 			$y=int($y);
@@ -76,7 +85,7 @@ sub preprocess (@) { #{{{
 			if (defined $location) {
 				$location=$locmap{$location};
 				if (! defined $location) {
-					return "[[".gettext("sparkline bad featurepoint location")."]]";
+					error gettext("invalid featurepoint location");
 				}
 			}
 			$php.=qq{\$sparkline->SetFeaturePoint($x, $y, '$color', $diameter};
@@ -87,31 +96,27 @@ sub preprocess (@) { #{{{
 	}
 
 	if ($c eq 0) {
-		return "[[".gettext("sparkline missing values")."]]";
+		error gettext("missing values");
 	}
 
 	my $height=int($params{height} || 20);
 	if ($height < 2 || $height > 100) {
-		return "[[".gettext("sparkline bad height value")."]]";
+		error gettext("invalid height value");
 	}
 	if ($style eq "Bar") {
 		$php.=qq{\$sparkline->Render($height);\n};
 	}
 	else {
 		if (! exists $params{width}) {
-			return "[[".gettext("sparkline missing width parameter")."]]";
+			error gettext("missing width parameter");
 		}
 		my $width=int($params{width});
 		if ($width < 2 || $width > 1024) {
-			return "[[".gettext("sparkline bad width value")."]]";
+			error gettext("invalid width value");
 		}
 		$php.=qq{\$sparkline->RenderResampled($width, $height);\n};
 	}
 	
-	if ($params{preview}) {
-		return "[[".gettext("sparkline previewing not implemented")."]]";
-	}
-	
 	$php.=qq{\$sparkline->Output();\n?>\n};
 
 	# Use the sha1 of the php code that generates the sparkline as
@@ -125,7 +130,7 @@ sub preprocess (@) { #{{{
 
 	if (! -e "$config{destdir}/$fn") {
 		my $pid;
-		my $sigpipe=0;;
+		my $sigpipe=0;
 		$SIG{PIPE}=sub { $sigpipe=1 };
 		$pid=open2(*IN, *OUT, "php");
 
@@ -144,16 +149,23 @@ sub preprocess (@) { #{{{
 
 		waitpid $pid, 0;
 		$SIG{PIPE}="DEFAULT";
-		if ($sigpipe) {
-			return  "[[".gettext("sparkline failed to run php")."]]";
+		if ($sigpipe || ! defined $png) {
+			error gettext("failed to run php");
 		}
 
-		writefile($fn, $config{destdir}, $png, 1);
+		if (! $params{preview}) {
+			writefile($fn, $config{destdir}, $png, 1);
+		}
+		else {
+			# can't write the file, so embed it in a data uri
+			eval q{use MIME::Base64};
+		        error($@) if $@;
+			return "<img src=\"data:image/png;base64,".
+				encode_base64($png)."\" />";
+		}
 	}
 
-	return '<img src="'.
-		IkiWiki::abs2rel($fn, IkiWiki::dirname($params{destpage})).
-		'" alt="graph" />';
-} # }}}
+	return '<img src="'.urlto($fn, $params{destpage}).'" alt="graph" />';
+}
 
 1