diff options
| author | Matt Selsky <selsky@columbia.edu> | 2012-04-18 14:37:58 +0200 |
|---|---|---|
| committer | Frédéric Buclin <LpSolit@gmail.com> | 2012-04-18 14:37:58 +0200 |
| commit | 7a87be5913af9ac8b82552a6835fc07163dc520b (patch) | |
| tree | c13b3c3b5e2ef0bb3f5d2dd536dcbdf2bc3b36e8 | |
| parent | 9a5a196d79ed8692c45595c4e3d42e34571bc3a7 (diff) | |
| download | bugzilla-7a87be5913af9ac8b82552a6835fc07163dc520b.tar.gz bugzilla-7a87be5913af9ac8b82552a6835fc07163dc520b.tar.xz | |
Bug 704999: Add support for GitHub for the 'See Also' field
r=timello a=LpSolit
| -rw-r--r-- | Bugzilla/BugUrl.pm | 1 | ||||
| -rw-r--r-- | Bugzilla/BugUrl/GitHub.pm | 36 | ||||
| -rw-r--r-- | template/en/default/global/user-error.html.tmpl | 1 |
3 files changed, 38 insertions, 0 deletions
diff --git a/Bugzilla/BugUrl.pm b/Bugzilla/BugUrl.pm index 888b11398..c48a33378 100644 --- a/Bugzilla/BugUrl.pm +++ b/Bugzilla/BugUrl.pm @@ -57,6 +57,7 @@ use constant SUB_CLASSES => qw( Bugzilla::BugUrl::Trac Bugzilla::BugUrl::MantisBT Bugzilla::BugUrl::SourceForge + Bugzilla::BugUrl::GitHub ); ############################### diff --git a/Bugzilla/BugUrl/GitHub.pm b/Bugzilla/BugUrl/GitHub.pm new file mode 100644 index 000000000..63be65bed --- /dev/null +++ b/Bugzilla/BugUrl/GitHub.pm @@ -0,0 +1,36 @@ +# 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; diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl index 3b2eca6d4..72d0241a0 100644 --- a/template/en/default/global/user-error.html.tmpl +++ b/template/en/default/global/user-error.html.tmpl @@ -254,6 +254,7 @@ <li>A ticket in a Trac installation.</li> <li>A b[% %]ug in a MantisBT installation.</li> <li>A b[% %]ug on sourceforge.net.</li> + <li>An issue on github.com.</li> [% Hook.process('bug_url_invalid_tracker') %] </ul> [% ELSIF reason == 'id' %] |