diff options
| author | Frédéric Buclin <LpSolit@netscape.net> | 2017-04-24 15:36:57 +0200 |
|---|---|---|
| committer | Frédéric Buclin <LpSolit@netscape.net> | 2017-04-24 15:36:57 +0200 |
| commit | d695b1d38447edae0558829766726d9d99489689 (patch) | |
| tree | 8fdfe0d94d994ef376f1b79d32f8dd48b84c6b12 /extensions/MoreBugUrl | |
| parent | fc55ae56a457aa489da0ad72fac598f40d0df8c3 (diff) | |
| download | bugs-d695b1d38447edae0558829766726d9d99489689.tar bugs-d695b1d38447edae0558829766726d9d99489689.tar.gz bugs-d695b1d38447edae0558829766726d9d99489689.tar.bz2 bugs-d695b1d38447edae0558829766726d9d99489689.tar.xz bugs-d695b1d38447edae0558829766726d9d99489689.zip | |
Backport upstream bug 1128119: Add 'See Also' support for Phabricator
Diffstat (limited to 'extensions/MoreBugUrl')
4 files changed, 43 insertions, 0 deletions
diff --git a/extensions/MoreBugUrl/Extension.pm b/extensions/MoreBugUrl/Extension.pm index 18507f8d1..0a4223e19 100644 --- a/extensions/MoreBugUrl/Extension.pm +++ b/extensions/MoreBugUrl/Extension.pm @@ -22,6 +22,7 @@ use constant MORE_SUB_CLASSES => qw( Bugzilla::Extension::MoreBugUrl::PHP Bugzilla::Extension::MoreBugUrl::Redmine Bugzilla::Extension::MoreBugUrl::Savane + Bugzilla::Extension::MoreBugUrl::Phabricator ); # We need to update bug_see_also table because both diff --git a/extensions/MoreBugUrl/disabled b/extensions/MoreBugUrl/disabled deleted file mode 100644 index e69de29bb..000000000 --- a/extensions/MoreBugUrl/disabled +++ /dev/null diff --git a/extensions/MoreBugUrl/lib/Phabricator.pm b/extensions/MoreBugUrl/lib/Phabricator.pm new file mode 100644 index 000000000..818d9af8f --- /dev/null +++ b/extensions/MoreBugUrl/lib/Phabricator.pm @@ -0,0 +1,41 @@ +# 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::Phabricator; + +use 5.10.1; +use strict; +use warnings; + +use parent qw(Bugzilla::BugUrl); + +############################### +#### Methods #### +############################### + +sub should_handle { + my ($class, $uri) = @_; + return ($uri->path =~ m|^/T\d+$|) ? 1 : 0; +} + +sub _check_value { + my $class = shift; + + my $uri = $class->SUPER::_check_value(@_); + + # Phabricator URLs have only one form: + # http://example.com/T111 + + # Make sure there are no query parameters. + $uri->query(undef); + # And remove any # part if there is one. + $uri->fragment(undef); + + return $uri; +} + +1; diff --git a/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl b/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl index 7e544ef21..2ac6f89a5 100644 --- a/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl +++ b/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl @@ -14,3 +14,4 @@ <li>A b[% %]ug on b[% %]ugs.php.net.</li> <li>An issue in a Redmine installation.</li> <li>A b[% %]ug in a Savane installation.</li> +<li>A task in a Phabricator installation.</li> |