move feedpages application up
authorJoey Hess <joey@kodama.kitenet.net>
Thu, 11 Dec 2008 18:50:51 +0000 (13:50 -0500)
committerJoey Hess <joey@kodama.kitenet.net>
Thu, 11 Dec 2008 18:58:40 +0000 (13:58 -0500)
I wanted this nearer to the top, but decided to put it after the
add_depends. Reasoning: It's possible with a combinaton of feedpages and
show options to make @list and @feedlist contain completly differing sets
of pages. We want to add_depends all pages in both sets. We could combine
the two lists and add_depends that, but it's slightly more efficient to
defer reducing @feedlist, and add_depends whichever list is longer.

IkiWiki/Plugin/inline.pm

index 7fe5a4dcf5c2f0543bef836067820133b2014405..0c8f503847aeb54cc0f08b68316673561c7ae936 100644 (file)
@@ -232,6 +232,10 @@ sub preprocess_inline (@) { #{{{
        # that if they are removed or otherwise changed, the inline will be
        # sure to be updated.
        add_depends($params{page}, join(" or ", $#list >= $#feedlist ? @list : @feedlist));
+       
+       if ($feeds && exists $params{feedpages}) {
+               @feedlist=grep { pagespec_match($_, $params{feedpages}, location => $params{page}) } @feedlist;
+       }
 
        my $feednum="";
 
@@ -364,10 +368,6 @@ sub preprocess_inline (@) { #{{{
        }
        
        if ($feeds) {
-               if (exists $params{feedpages}) {
-                       @feedlist=grep { pagespec_match($_, $params{feedpages}, location => $params{page}) } @feedlist;
-               }
-       
                if ($rss) {
                        my $rssp=rsspage($params{destpage}).$feednum;
                        will_render($params{destpage}, $rssp);