Commit 71d84962 authored by Koosha Khajeh Moogahi's avatar Koosha Khajeh Moogahi Committed by Frédéric Buclin

Bug 804343: Implement autolinkification for a list of comment ids

r/a=LpSolit
parent fe4e7df6
......@@ -236,7 +236,8 @@ sub quoteUrls {
# empty string
my $bug_word = template_var('terms')->{bug};
my $bug_re = qr/\Q$bug_word\E\s*\#?\s*(\d+)/i;
my $comment_re = qr/comment\s*\#?\s*(\d+)/i;
my $comment_word = template_var('terms')->{comment};
my $comment_re = qr/(?:\Q$comment_word\E|comment)\s*\#?\s*(\d+)/i;
$text =~ s~\b($bug_re(?:\s*,?\s*$comment_re)?|$comment_re)
~ # We have several choices. $1 here is the link, and $2-4 are set
# depending on which part matched
......@@ -261,6 +262,19 @@ sub quoteUrls {
substr($text, $offset, $length) = $match;
}
my $comments_word = template_var('terms')->{comments};
my $comments_re = qr/(?:comments|\Q$comments_word\E)\s*\#?\s*
\d+(?:\s*,\s*\#?\s*\d+)+/ix;
while ($text =~ m/($comments_re)/go) {
my $offset = $-[0];
my $length = $+[0] - $-[0];
my $match = $1;
$match =~ s|((?:#\s*)?(\d+))|<a href="$current_bugurl#c$2">$1</a>|g;
substr($text, $offset, $length) = $match;
}
# Old duplicate markers. These don't use $bug_word because they are old
# and were never customizable.
$text =~ s~(?<=^\*\*\*\ This\ bug\ has\ been\ marked\ as\ a\ duplicate\ of\ )
......
......@@ -24,6 +24,8 @@
"ABug" => "A Bug",
"bugs" => "bugs",
"Bugs" => "Bugs",
"comment" => "comment",
"comments" => "comments",
"zeroSearchResults" => "Zarro Boogs found",
"Bugzilla" => "Bugzilla"
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment