Commit 0b899217 authored by Matt Selsky's avatar Matt Selsky Committed by Tiago Mello

Bug 731727: Add support for RT to MoreBugUrl extension

r=timello, a=LpSolit
parent 9a9f9a54
...@@ -12,6 +12,7 @@ use base qw(Bugzilla::Extension); ...@@ -12,6 +12,7 @@ use base qw(Bugzilla::Extension);
use constant MORE_SUB_CLASSES => qw( use constant MORE_SUB_CLASSES => qw(
Bugzilla::Extension::MoreBugUrl::ReviewBoard Bugzilla::Extension::MoreBugUrl::ReviewBoard
Bugzilla::Extension::MoreBugUrl::Rietveld Bugzilla::Extension::MoreBugUrl::Rietveld
Bugzilla::Extension::MoreBugUrl::RT
); );
# We need to update bug_see_also table because both # We need to update bug_see_also table because both
......
# 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::RT;
use strict;
use base qw(Bugzilla::BugUrl);
###############################
#### Methods ####
###############################
sub should_handle {
my ($class, $uri) = @_;
# RT URLs can look like various things:
# http://example.com/rt/Ticket/Display.html?id=1234
# https://example.com/Public/Bug/Display.html?id=1234
return ($uri->path =~ m|/Display.html$|
and $uri->query_param('id') =~ /^\d+$/) ? 1 : 0;
}
sub _check_value {
my $class = shift;
my $uri = $class->SUPER::_check_value(@_);
# And remove any # part if there is one.
$uri->fragment(undef);
return $uri;
}
1;
...@@ -8,3 +8,4 @@ ...@@ -8,3 +8,4 @@
<li>A Review Board review request.</li> <li>A Review Board review request.</li>
<li>An issue in a Rietveld installation.</li> <li>An issue in a Rietveld installation.</li>
<li>A ticket in an RT installation.</li>
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