Merge branch 'master' into debian-jessie-backports

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

diff --git a/IkiWiki/Plugin/brokenlinks.pm b/IkiWiki/Plugin/brokenlinks.pm
index f8d44892b79f6e0f30612e08d949b2ebf69a35d2..2318298fe4726c31017c250cf5b57c96b8866640 100644 (file)
--- a/IkiWiki/Plugin/brokenlinks.pm
+++ b/IkiWiki/Plugin/brokenlinks.pm
@@ -23,36 +23,27 @@ sub preprocess (@) {
        my %params=@_;
        $params{pages}="*" unless defined $params{pages};
        
        my %params=@_;
        $params{pages}="*" unless defined $params{pages};
        

-       # Needs to update whenever a page is added or removed, so
-       # register a dependency.
-       add_depends($params{page}, $params{pages});
-       
-       my %broken;
-       my $discussion=gettext("Discussion");
-       foreach my $page (pagespec_match_list([keys %links],
-                       $params{pages}, location => $params{page})) {
-               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;
        my @broken;

-       foreach my $link (keys %broken) {
-               my $page=$broken{$link}->[0];
+       foreach my $link (keys %IkiWiki::brokenlinks) {
+               next if $link =~ /.*\/\Q$config{discussionpage}\E/i && $config{discussion};
+
+               my @pages=pagespec_match_list($params{page}, $params{pages},
+                       list => $IkiWiki::brokenlinks{$link},
+                       # needs to update when links on a page change
+                       deptype => deptype("links")
+               );
+               next unless @pages;
+
+               my $page=$IkiWiki::brokenlinks{$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)
                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}}));
+                       } sort @pages)
+               );

        }
        
        }
        

-       return gettext("There are no broken links!") unless %broken;
+       return gettext("There are no broken links!") unless @broken;

        return "<ul>\n"
                .join("\n",
                        map {
        return "<ul>\n"
                .join("\n",
                        map {