From: harishcm <harishcm@web>
Date: Sat, 21 Nov 2009 10:18:00 +0000 (-0500)
Subject: Patch added
X-Git-Url: https://sipb.mit.edu/gitweb.cgi/ikiwiki.git/commitdiff_plain/dc15be03d69604fcae20ce49d684d3146ff068a6?hp=62dd681b3aa406a9bb6c1174eeaaacc8fe6b94ff;ds=sidebyside

Patch added
---

diff --git a/doc/bugs/Building_a_sidebar_does_not_regenerate_the_subpages.mdwn b/doc/bugs/Building_a_sidebar_does_not_regenerate_the_subpages.mdwn
index 596719a8b..93aafc2c0 100644
--- a/doc/bugs/Building_a_sidebar_does_not_regenerate_the_subpages.mdwn
+++ b/doc/bugs/Building_a_sidebar_does_not_regenerate_the_subpages.mdwn
@@ -7,3 +7,54 @@ If sandbox/page.mdwn has been generated and sandbox/sidebar.mdwn is created, the
 	# currently requires a wiki rebuild.
 	add_depends($page, $sidebar_page);
 
+----
+Below is a patch for the bug. It's inspired by trying to solve [[todo/Post-compilation inclusion of the sidebar]].
+What do you think about it? I have a concern though. If a sidebar is removed, it takes two refreshes to update 
+the affected pages. Is this a feature or a bug? --[[harishcm]]
+
+    --- sidebar.pm.orig	2009-11-21 17:40:02.000000000 +0800
+    +++ sidebar_bugfix.pm	2009-11-21 18:16:11.000000000 +0800
+    @@ -10,6 +10,7 @@
+     
+     sub import {
+     	hook(type => "getsetup", id => "sidebar", call => \&getsetup);
+    +	hook(type => "needsbuild", id => "sidebar", call => \&needsbuild);
+     	hook(type => "pagetemplate", id => "sidebar", call => \&pagetemplate);
+     }
+     
+    @@ -21,6 +22,22 @@
+     		},
+     }
+     
+    +sub needsbuild (@) {
+    +	my $needsbuild=shift;
+    +
+    +	# Determine every pages' sidebar page
+    +	foreach my $page (keys %pagesources) {
+    +		my $sidebar_page='';
+    +		$sidebar_page=bestlink($page, "sidebar");
+    +		
+    +		# If a page's sidebar has changed, force rebuild
+    +		if (!exists $pagestate{$page}{sidebar}{sidebar_page} || $pagestate{$page}{sidebar}{sidebar_page} ne $sidebar_page) {
+    +			$pagestate{$page}{sidebar}{sidebar_page} = $sidebar_page;
+    +			push @$needsbuild, $pagesources{$page};
+    +		}
+    +	}
+    +}
+    +
+     sub sidebar_content ($) {
+     	my $page=shift;
+     	
+    @@ -29,9 +46,9 @@
+     	my $sidebar_type=pagetype($sidebar_file);
+     	
+     	if (defined $sidebar_type) {
+    -		# FIXME: This isn't quite right; it won't take into account
+    -		# adding a new sidebar page. So adding such a page
+    -		# currently requires a wiki rebuild.
+    +		# Record current sidebar page for rechecking bestlink 
+    +		# during wiki refresh. Also add depends on the sidebar page.
+    +		$pagestate{$page}{sidebar}{sidebar_page}=$sidebar_page;
+     		add_depends($page, $sidebar_page);
+     
+     		my $content=readfile(srcfile($sidebar_file));