Commit 7a87be59 authored by Matt Selsky's avatar Matt Selsky Committed by Frédéric Buclin

Bug 704999: Add support for GitHub for the 'See Also' field

r=timello a=LpSolit
parent 9a5a196d
...@@ -57,6 +57,7 @@ use constant SUB_CLASSES => qw( ...@@ -57,6 +57,7 @@ use constant SUB_CLASSES => qw(
Bugzilla::BugUrl::Trac Bugzilla::BugUrl::Trac
Bugzilla::BugUrl::MantisBT Bugzilla::BugUrl::MantisBT
Bugzilla::BugUrl::SourceForge Bugzilla::BugUrl::SourceForge
Bugzilla::BugUrl::GitHub
); );
############################### ###############################
......
# 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::BugUrl::GitHub;
use strict;
use base qw(Bugzilla::BugUrl);
###############################
#### Methods ####
###############################
sub should_handle {
my ($class, $uri) = @_;
# GitHub issue URLs have only one form:
# https://github.com/USER_OR_TEAM_OR_ORGANIZATION_NAME/REPOSITORY_NAME/issues/111
return ($uri->authority =~ /^github.com$/i
and $uri->path =~ m|^/[^/]+/[^/]+/issues/\d+$|) ? 1 : 0;
}
sub _check_value {
my ($class, $uri) = @_;
$uri = $class->SUPER::_check_value($uri);
# GitHub HTTP URLs redirect to HTTPS, so just use the HTTPS scheme.
$uri->scheme('https');
return $uri;
}
1;
...@@ -254,6 +254,7 @@ ...@@ -254,6 +254,7 @@
<li>A ticket in a Trac installation.</li> <li>A ticket in a Trac installation.</li>
<li>A b[% %]ug in a MantisBT installation.</li> <li>A b[% %]ug in a MantisBT installation.</li>
<li>A b[% %]ug on sourceforge.net.</li> <li>A b[% %]ug on sourceforge.net.</li>
<li>An issue on github.com.</li>
[% Hook.process('bug_url_invalid_tracker') %] [% Hook.process('bug_url_invalid_tracker') %]
</ul> </ul>
[% ELSIF reason == 'id' %] [% ELSIF reason == 'id' %]
......
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