bugfix

[ikiwiki.git] / IkiWiki / Plugin / sidebar.pm

diff --git a/IkiWiki/Plugin/sidebar.pm b/IkiWiki/Plugin/sidebar.pm
index 1b302dcf9f80dde6a6f1a6ba6e5620a34e7af593..0350f2b71d7fc20695c169daa4ef967e591b22fe 100644 (file)
--- a/IkiWiki/Plugin/sidebar.pm
+++ b/IkiWiki/Plugin/sidebar.pm
@@ -23,7 +23,7 @@ sub getsetup () {
                global_sidebars => {
                        type => "boolean",
                        examples => 1,
-                       description => "show sidebar page on all pages?"
+                       description => "show sidebar page on all pages?",
                        safe => 1,
                        rebuild => 1,
                },
@@ -33,23 +33,23 @@ my %pagesidebar;
 
 sub preprocess (@) {
        my %params=@_;
-       my $content=shift;
-       shift;
-
-       if (! defined $content) {
-               error(gettext("sidebar content not specified"));
-       }
 
        my $page=$params{page};
        return "" unless $page eq $params{destpage};
-       my $file = $pagesources{$page};
-       my $type = pagetype($file);
-
-       $pagesidebar{$page}=
-               IkiWiki::htmlize($page, $page, $type,
-               IkiWiki::linkify($page, $page,
-               IkiWiki::preprocess($page, $page,
-               IkiWiki::filter($page, $page, $content))));
+       
+       if (! defined $params{content}) {
+               $pagesidebar{$page}=undef;
+       }
+       else {
+               my $file = $pagesources{$page};
+               my $type = pagetype($file);
+
+               $pagesidebar{$page}=
+                       IkiWiki::htmlize($page, $page, $type,
+                       IkiWiki::linkify($page, $page,
+                       IkiWiki::preprocess($page, $page,
+                       IkiWiki::filter($page, $page, $params{content}))));
+       }
 
        return "";
 }
@@ -60,9 +60,10 @@ my $oldcontent;
 sub sidebar_content ($) {
        my $page=shift;
        
-       return $pagesidebar{$page} if exists $pagesidebar{$page};
+       return delete $pagesidebar{$page} if defined $pagesidebar{$page};
 
-       return if defined $config{global_sidebars} && !$config{global_sidebars};
+       return if ! exists $pagesidebar{$page} && 
+               defined $config{global_sidebars} && ! $config{global_sidebars};
 
        my $sidebar_page=bestlink($page, "sidebar") || return;
        my $sidebar_file=$pagesources{$sidebar_page} || return;
@@ -96,11 +97,10 @@ sub sidebar_content ($) {
 sub pagetemplate (@) {
        my %params=@_;
 
-       my $page=$params{page};
        my $template=$params{template};
-       
-       if ($template->query(name => "sidebar")) {
-               my $content=sidebar_content($page);
+       if ($params{destpage} eq $params{page} &&
+           $template->query(name => "sidebar")) {
+               my $content=sidebar_content($params{destpage});
                if (defined $content && length $content) {
                        $template->param(sidebar => $content);
                }