Skip to content

Commit

Permalink
Bug 1128119: Add 'See Also' support for Phabricator. r=gerv, a=glob.
Browse files Browse the repository at this point in the history
  • Loading branch information
selsky authored and Gervase Markham committed Feb 17, 2015
1 parent 0d2cf0a commit 077949a
Showing 1 changed file with 41 additions and 0 deletions.
41 changes: 41 additions & 0 deletions extensions/MoreBugUrl/lib/Phabricator.pm
Original file line number Diff line number Diff line change
@@ -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;

0 comments on commit 077949a

Please sign in to comment.