X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/2794d7ef5abc4fa8fc2eb42d5c85ada197df0767..6a2abf8352a7a1516345f408b6356bd1ace68f19:/IkiWiki/Plugin/inline.pm

diff --git a/IkiWiki/Plugin/inline.pm b/IkiWiki/Plugin/inline.pm
index d8f2ca0d8..70d5be235 100644
--- a/IkiWiki/Plugin/inline.pm
+++ b/IkiWiki/Plugin/inline.pm
@@ -5,6 +5,7 @@ package IkiWiki::Plugin::inline;
 use warnings;
 use strict;
 use IkiWiki;
+use URI;
 
 sub import { #{{{
 	IkiWiki::hook(type => "preprocess", id => "inline", 
@@ -21,29 +22,31 @@ sub import { #{{{
 package IkiWiki;
 
 my %toping;
-my $processing_inline=0;
+
+sub yesno ($) { #{{{
+	my $val=shift;
+	return (defined $val && lc($val) eq "yes");
+} #}}}
 
 sub preprocess_inline (@) { #{{{
 	my %params=@_;
-
+	
 	if (! exists $params{pages}) {
 		return "";
 	}
-	if (! exists $params{archive}) {
-		$params{archive}="no";
-	}
-	if (! exists $params{show} && $params{archive} eq "no") {
+	my $raw=yesno($params{raw});
+	my $archive=yesno($params{archive});
+	my $rss=exists $params{rss} ? yesno($params{rss}) : 1;
+	if (! exists $params{show} && ! $archive) {
 		$params{show}=10;
 	}
-	if (! exists $params{rss}) {
-		$params{rss}="yes";
+	my $desc;
+	if (exists $params{description}) {
+		$desc = $params{description} 
+	} else {
+		$desc = $config{wikiname};
 	}
-
-	# Avoid nested inlines, to avoid loops etc.
-	if ($processing_inline) {
-		return "";
-	}
-	$processing_inline=1;
+	my $actions=yesno($params{actions});
 
 	my @list;
 	foreach my $page (keys %pagesources) {
@@ -71,7 +74,7 @@ sub preprocess_inline (@) { #{{{
 		}
 		$ret.=$formtemplate->output;
 	}
-	elsif ($config{rss} && $params{rss} eq "yes") {
+	elsif ($config{rss} && $rss) {
 		# Add a rss link button.
 		my $linktemplate=template("rsslink.tmpl", blind_cache => 1);
 		$linktemplate->param(rssurl => rsspage(basename($params{page})));
@@ -79,43 +82,72 @@ sub preprocess_inline (@) { #{{{
 	}
 	
 	my $template=template(
-		(($params{archive} eq "no")
-			? "inlinepage.tmpl"
-			: "inlinepagetitle.tmpl"),
+		($archive ? "inlinepagetitle.tmpl" : "inlinepage.tmpl"),
 		blind_cache => 1,
-	);
+	) unless $raw;
 	
 	foreach my $page (@list) {
-		# Don't use htmllink because this way the title is separate
-		# and can be overridden by other plugins.
-		my $link=htmlpage(bestlink($params{page}, $page));
-		$link=abs2rel($link, dirname($params{page}));
-		$template->param(pageurl => $link);
-		$template->param(title => pagetitle(basename($page)));
-		$template->param(content => get_inline_content($page, $params{page}))
-			if $params{archive} eq "no";
-		$template->param(ctime => displaytime($pagectime{$page}));
+		if (! $raw) {
+			# Get the content before populating the template,
+			# since getting the content uses the same template
+			# if inlines are nested.
+			# TODO: if $archive=1, the only reason to do this
+			# is to let the meta plugin get page title info; so stop
+			# calling this next line then once the meta plugin can
+			# store that accross runs (also tags plugin).
+			my $content=get_inline_content($page, $params{destpage});
+			# Don't use htmllink because this way the title is separate
+			# and can be overridden by other plugins.
+			my $link=htmlpage(bestlink($params{page}, $page));
+			$link=abs2rel($link, dirname($params{destpage}));
+			$template->param(pageurl => $link);
+			$template->param(title => pagetitle(basename($page)));
+			$template->param(content => $content);
+			$template->param(ctime => displaytime($pagectime{$page}));
 
-		run_hooks(pagetemplate => sub {
-			shift->(page => $page, destpage => $params{page},
-				template => $template,);
-		});
+			if ($actions) {
+				my $file = $pagesources{$page};
+				my $type = pagetype($file);
+				if ($config{discussion}) {
+					$template->param(have_actions => 1);
+					$template->param(discussionlink => htmllink($page, $page, "Discussion", 1, 1));
+				}
+				if (length $config{cgiurl} && defined $type) {
+					$template->param(have_actions => 1);
+					$template->param(editurl => cgiurl(do => "edit", page => $page));
+				}
+			}
+
+			run_hooks(pagetemplate => sub {
+				shift->(page => $page, destpage => $params{page},
+					template => $template,);
+			});
 
-		$ret.=$template->output;
-		$template->clear_params;
+			$ret.=$template->output;
+			$template->clear_params;
+		}
+		else {
+			my $file=$pagesources{$page};
+			my $type=pagetype($file);
+			if (defined $type) {
+				$ret.="\n".
+				      linkify($page, $params{page},
+				      preprocess($page, $params{page},
+				      filter($page,
+				      readfile(srcfile($file)))));
+			}
+		}
 	}
 	
 	# TODO: should really add this to renderedfiles and call
 	# check_overwrite, but currently renderedfiles
 	# only supports listing one file per page.
-	if ($config{rss} && $params{rss} eq "yes") {
+	if ($config{rss} && $rss) {
 		writefile(rsspage($params{page}), $config{destdir},
-			genrss($params{page}, @list));
+			genrss($desc, $params{page}, @list));
 		$toping{$params{page}}=1 unless $config{rebuild};
 	}
 	
-	$processing_inline=0;
-
 	return $ret;
 } #}}}
 
@@ -126,7 +158,11 @@ sub get_inline_content ($$) { #{{{
 	my $file=$pagesources{$page};
 	my $type=pagetype($file);
 	if (defined $type) {
-		return htmlize($type, preprocess($page, $destpage, linkify($page, $destpage, readfile(srcfile($file)))));
+		return htmlize($page, $type,
+		       linkify($page, $destpage,
+		       preprocess($page, $destpage,
+		       filter($page,
+		       readfile(srcfile($file))))));
 	}
 	else {
 		return "";
@@ -137,7 +173,11 @@ sub date_822 ($) { #{{{
 	my $time=shift;
 
 	eval q{use POSIX};
-	return POSIX::strftime("%a, %d %b %Y %H:%M:%S %z", localtime($time));
+	my $lc_time= POSIX::setlocale(&POSIX::LC_TIME);
+	POSIX::setlocale(&POSIX::LC_TIME, "C");
+	my $ret=POSIX::strftime("%a, %d %b %Y %H:%M:%S %z", localtime($time));
+	POSIX::setlocale(&POSIX::LC_TIME, $lc_time);
+	return $ret;
 } #}}}
 
 sub absolute_urls ($$) { #{{{
@@ -158,21 +198,24 @@ sub rsspage ($) { #{{{
 	return $page.".rss";
 } #}}}
 
-sub genrss ($@) { #{{{
+sub genrss ($$@) { #{{{
+	my $desc=shift;
 	my $page=shift;
 	my @pages=@_;
 	
-	my $url="$config{url}/".htmlpage($page);
+	my $url=URI->new(encode_utf8($config{url}."/".htmlpage($page)));
 	
-	my $itemtemplate=template("rssitem.tmpl", blind_cache => 1, 
-		die_on_bad_params => 0);
+	my $itemtemplate=template("rssitem.tmpl", blind_cache => 1);
 	my $content="";
 	foreach my $p (@pages) {
 		next unless exists $renderedfiles{$p};
 
+		my $u=URI->new(encode_utf8("$config{url}/$renderedfiles{$p}"));
+
 		$itemtemplate->param(
 			title => pagetitle(basename($p)),
-			url => "$config{url}/$renderedfiles{$p}",
+			url => $u,
+			permalink => $u,
 			pubdate => date_822($pagectime{$p}),
 			content => absolute_urls(get_inline_content($p, $page), $url),
 		);
@@ -180,6 +223,7 @@ sub genrss ($@) { #{{{
 			shift->(page => $p, destpage => $page,
 				template => $itemtemplate);
 		});
+
 		$content.=$itemtemplate->output;
 		$itemtemplate->clear_params;
 	}
@@ -190,8 +234,8 @@ sub genrss ($@) { #{{{
 		wikiname => $config{wikiname},
 		pageurl => $url,
 		content => $content,
+		rssdesc => $desc,
 	);
-	
 	run_hooks(pagetemplate => sub {
 		shift->(page => $page, destpage => $page,
 			template => $template);
@@ -223,7 +267,7 @@ sub pingurl (@) { #{{{
 			}
 			my $r=$res->value;
 			if (! exists $r->{flerror} || $r->{flerror}) {
-				debug("Ping rejected: ".$r->{message});
+				debug("Ping rejected: ".(exists $r->{message} ? $r->{message} : "[unknown reason]"));
 			}
 		}
 	}