pagecount: switched to use_pagespec
authorJoey Hess <joey@gnu.kitenet.net>
Thu, 8 Oct 2009 22:41:08 +0000 (18:41 -0400)
committerJoey Hess <joey@gnu.kitenet.net>
Thu, 8 Oct 2009 22:47:25 +0000 (18:47 -0400)
IkiWiki/Plugin/pagecount.pm

index 419f2d535ec6fbe5cb32a8a09173bfd4378bc827..40474b2a1c4b1e05bfac394075339a2d37f7c82a 100644 (file)
@@ -20,20 +20,20 @@ sub getsetup () {
 
 sub preprocess (@) {
        my %params=@_;
-       $params{pages}="*" unless defined $params{pages};
+       my $pages=defined $params{pages} ? $params{pages} : "*";
        
-       # Needs to update count whenever a page is added or removed, so
-       # register a presence dependency.
-       add_depends($params{page}, $params{pages}, deptype("presence"));
-       
-       my @pages;
-       if ($params{pages} eq "*") {
-               @pages=keys %pagesources;
-       }
-       else {
-               @pages=pagespec_match_list([keys %pagesources], $params{pages}, location => $params{page});
+       # Just get a list of all the pages, and count the items in it.
+       # Use a presence dependency to only update when pages are added
+       # or removed.
+
+       if ($pages eq '*') {
+               # optimisation to avoid needing to try matching every page
+               add_depends($params{page}, $pages, deptype("presence"));
+               return scalar keys %pagesources;
        }
-       return $#pages+1;
+
+       return scalar use_pagespec($params{page}, $pages,
+               deptype => deptype("presence"));
 }
 
 1