diff options
author | Matt Selsky <selsky@columbia.edu> | 2015-02-17 18:24:09 +0100 |
---|---|---|
committer | Gervase Markham <gerv@gerv.net> | 2015-02-17 18:24:09 +0100 |
commit | 077949ad0423d2446f2dff331fb5c44ee87c449a (patch) | |
tree | 7ab2e402761acb604e3ad4e87e81454eea7bb990 | |
parent | 0d2cf0ac8b6b752794067b7ef6b2234cced0361e (diff) | |
download | bugzilla-077949ad0423d2446f2dff331fb5c44ee87c449a.tar.gz bugzilla-077949ad0423d2446f2dff331fb5c44ee87c449a.tar.xz |
Bug 1128119: Add 'See Also' support for Phabricator. r=gerv, a=glob.
-rw-r--r-- | extensions/MoreBugUrl/lib/Phabricator.pm | 41 |
1 files changed, 41 insertions, 0 deletions
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; |