From 85f865b5d98e0122934d11e3f3eb6703e4f4c620 Mon Sep 17 00:00:00 2001 From: intrigeri Date: Wed, 31 Dec 2008 22:27:12 +0100 Subject: [PATCH] po: added postscan hook, to make pages depend on the pages linking to them ... so that nicepagetitle hook's effects, such as translation status displayed in links, are updated when the linked page changes. The replacement of 'my %backlinks' with 'our %backlinks' in Render.pm made this work: previously, every postscan hook was called with an almost empty %backlinks, which defeated all my attempts to implement this feature. This feature hits performance a bit. Its cost was quite small in my real-world use-cases (a few percents bigger refresh time), but could be bigger in worst cases. Time will tell. NB: this hack could also be used by my meta branch. It may even be a ikiwiki optional feature. Signed-off-by: intrigeri --- IkiWiki/Plugin/po.pm | 11 +++++++++++ IkiWiki/Render.pm | 2 +- doc/plugins/po.mdwn | 4 ---- 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/IkiWiki/Plugin/po.pm b/IkiWiki/Plugin/po.pm index 33bc2b057..c529828d6 100644 --- a/IkiWiki/Plugin/po.pm +++ b/IkiWiki/Plugin/po.pm @@ -35,6 +35,7 @@ sub import { #{{{ hook(type => "filter", id => "po", call => \&filter); hook(type => "htmlize", id => "po", call => \&htmlize); hook(type => "pagetemplate", id => "po", call => \&pagetemplate, last => 1); + hook(type => "postscan", id => "po", call => \&postscan); hook(type => "rename", id => "po", call => \&renamepages); hook(type => "delete", id => "po", call => \&mydelete); hook(type => "change", id => "po", call => \&change); @@ -340,6 +341,16 @@ sub pagetemplate (@) { #{{{ } } # }}} +sub postscan (@) { + my %params = @_; + my $page = $params{page}; + + # backlinks involve back-dependencies, so that nicepagetitle effects, + # such as translation status displayed in links, are updated + use IkiWiki::Render; + map add_depends($page, $_), keys %{$IkiWiki::backlinks{$page}}; +} + # Add the renamed page translations to the list of to-be-renamed pages. # Save information about master page rename, so that: # - our delete hook can ignore the translations not renamed already diff --git a/IkiWiki/Render.pm b/IkiWiki/Render.pm index adae9f0d5..0ce972ac7 100644 --- a/IkiWiki/Render.pm +++ b/IkiWiki/Render.pm @@ -7,7 +7,7 @@ use strict; use IkiWiki; use Encode; -my %backlinks; +our %backlinks=(); my $backlinks_calculated=0; sub calculate_backlinks () { diff --git a/doc/plugins/po.mdwn b/doc/plugins/po.mdwn index 4ce8ad7d9..7d8da987e 100644 --- a/doc/plugins/po.mdwn +++ b/doc/plugins/po.mdwn @@ -511,10 +511,6 @@ intrigeri's `meta` branch, the generated links' text is based on the page titles set with the [[meta|plugins/meta]] plugin. This has to be merged upstream, though. -### Backlinks - -They are not updated when the source page changes (e.g. meta title). - Page formats ------------ -- 2.32.0.93.g670b81a890