Commit 5fb38bd7 authored by Matt Selsky's avatar Matt Selsky Committed by Frédéric Buclin

Bug 735196: Support "bugs.php.net" URLs in "See also" field

r=gerv a=LpSolit
parent 30a1fcbc
......@@ -14,6 +14,7 @@ use constant MORE_SUB_CLASSES => qw(
Bugzilla::Extension::MoreBugUrl::Rietveld
Bugzilla::Extension::MoreBugUrl::RT
Bugzilla::Extension::MoreBugUrl::GetSatisfaction
Bugzilla::Extension::MoreBugUrl::PHP
);
# We need to update bug_see_also table because both
......
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
#
# This Source Code Form is "Incompatible With Secondary Licenses", as
# defined by the Mozilla Public License, v. 2.0.
package Bugzilla::Extension::MoreBugUrl::PHP;
use strict;
use base qw(Bugzilla::BugUrl);
###############################
#### Methods ####
###############################
sub should_handle {
my ($class, $uri) = @_;
# PHP Bug URLs have only one form:
# https://bugs.php.net/bug.php?id=1234
return ($uri->authority =~ /^bugs.php.net$/i
and $uri->path =~ m|/bug.php$|
and $uri->query_param('id') =~ /^\d+$/) ? 1 : 0;
}
sub _check_value {
my $class = shift;
my $uri = $class->SUPER::_check_value(@_);
# PHP Bug URLs redirect to HTTPS, so just use the HTTPS scheme.
$uri->scheme('https');
# And remove any # part if there is one.
$uri->fragment(undef);
return $uri;
}
1;
......@@ -10,3 +10,4 @@
<li>An issue in a Rietveld installation.</li>
<li>A ticket in an RT installation.</li>
<li>A topic on getsatisfaction.com.</li>
<li>A b[% %]ug on b[% %]ugs.php.net.</li>
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