pagespec_match_list * optimisation

[git.ikiwiki.info.git] / IkiWiki / Plugin / brokenlinks.pm

diff --git a/IkiWiki/Plugin/brokenlinks.pm b/IkiWiki/Plugin/brokenlinks.pm
index 9485da398f8417f8e81503d799149a1e83be197e..da97dbc2855bc59f17efe5792bacb6c36fe8e7f7 100644 (file)
--- a/IkiWiki/Plugin/brokenlinks.pm
+++ b/IkiWiki/Plugin/brokenlinks.pm
@@ -4,36 +4,62 @@ package IkiWiki::Plugin::brokenlinks;
 
 use warnings;
 use strict;
 
 use warnings;
 use strict;

+use IkiWiki 3.00;

 
 

-sub import { #{{{
-       IkiWiki::register_plugin("preprocess", "brokenlinks", \&preprocess);
-} # }}}
+sub import {
+       hook(type => "getsetup", id => "brokenlinks", call => \&getsetup);
+       hook(type => "preprocess", id => "brokenlinks", call => \&preprocess);
+}

 
 

-sub preprocess (@) { #{{{
+sub getsetup {
+       return
+               plugin => {
+                       safe => 1,
+                       rebuild => undef,
+               },
+}
+
+sub preprocess (@) {

        my %params=@_;
        $params{pages}="*" unless defined $params{pages};
        
        # Needs to update whenever a page is added or removed, so
        # register a dependency.
        my %params=@_;
        $params{pages}="*" unless defined $params{pages};
        
        # Needs to update whenever a page is added or removed, so
        # register a dependency.

-       IkiWiki::add_depends($params{page}, $params{pages});
+       add_depends($params{page}, $params{pages});

        
        

-       my @broken;
-       foreach my $page (%IkiWiki::links) {
-               if (IkiWiki::globlist_match($page, $params{pages})) {
-                       foreach my $link (@{$IkiWiki::links{$page}}) {
-                               next if $link =~ /.*\/discussion/i;
-                               my $bestlink=IkiWiki::bestlink($page, $link);
-                               next if length $bestlink;
-                               push @broken,
-                                       IkiWiki::htmllink($page, $link, 1).
-                                       " in ".
-                                       IkiWiki::htmllink($params{page}, $page, 1);
-                       }
+       my %broken;
+       foreach my $page (pagespec_match_list([keys %links],
+                       $params{pages}, location => $params{page})) {
+               my $discussion=gettext("discussion");
+               my %seen;
+               foreach my $link (@{$links{$page}}) {
+                       next if $seen{$link};
+                       $seen{$link}=1;
+                       next if $link =~ /.*\/\Q$discussion\E/i && $config{discussion};
+                       my $bestlink=bestlink($page, $link);
+                       next if length $bestlink;
+                       push @{$broken{$link}}, $page;

                }
        }
                }
        }

+
+       my @broken;
+       foreach my $link (keys %broken) {
+               my $page=$broken{$link}->[0];
+               push @broken, sprintf(gettext("%s from %s"),
+                       htmllink($page, $params{destpage}, $link, noimageinline => 1),
+                       join(", ", map {
+                               htmllink($params{page}, $params{destpage}, $_,  noimageinline => 1)
+                       } @{$broken{$link}}));
+       }

        
        

-       return "There are no broken links!" unless @broken;
-       return "<ul>\n".join("\n", map { "<li>$_</li>" } sort @broken)."</ul>\n";
-} # }}}
+       return gettext("There are no broken links!") unless %broken;
+       return "<ul>\n"
+               .join("\n",
+                       map {
+                               "<li>$_</li>"
+                       }
+                       sort @broken)
+               ."</ul>\n";
+}

 
 1
 
 1