X-Git-Url: https://sipb.mit.edu/gitweb.cgi/ikiwiki.git/blobdiff_plain/54d5308cd83c67e7e9c32450c776ef0dac63549f..2078f706d656a12f0f9495e0bb348ee4b0282f42:/IkiWiki/Plugin/brokenlinks.pm

diff --git a/IkiWiki/Plugin/brokenlinks.pm b/IkiWiki/Plugin/brokenlinks.pm
index 9485da398..0295a8fe6 100644
--- a/IkiWiki/Plugin/brokenlinks.pm
+++ b/IkiWiki/Plugin/brokenlinks.pm
@@ -4,9 +4,10 @@ package IkiWiki::Plugin::brokenlinks;
 
 use warnings;
 use strict;
+use IkiWiki 2.00;
 
 sub import { #{{{
-	IkiWiki::register_plugin("preprocess", "brokenlinks", \&preprocess);
+	hook(type => "preprocess", id => "brokenlinks", call => \&preprocess);
 } # }}}
 
 sub preprocess (@) { #{{{
@@ -15,25 +16,42 @@ sub preprocess (@) { #{{{
 	
 	# 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);
+	my %broken;
+	foreach my $page (keys %links) {
+		if (pagespec_match($page, $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,
-					IkiWiki::htmllink($page, $link, 1).
-					" in ".
-					IkiWiki::htmllink($params{page}, $page, 1);
+				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