diff options
author | Matt Selsky <selsky@columbia.edu> | 2012-06-17 14:19:09 +0200 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2012-06-17 14:19:09 +0200 |
commit | 4c236f418a7122cecd22c36c679a7b9faf5469f7 (patch) | |
tree | 93326ee30db6f08ef84a99f74fdb8f6fea3e9d8e /Bugzilla/BugUrl/JIRA.pm | |
parent | 35bf9943a87018e6cfa2d49f83f77c22205ddfc0 (diff) | |
download | bugzilla-4c236f418a7122cecd22c36c679a7b9faf5469f7.tar.gz bugzilla-4c236f418a7122cecd22c36c679a7b9faf5469f7.tar.xz |
Bug 759030: Clean up Bugzilla::BugUrl modules
r=timello a=LpSolit
Diffstat (limited to 'Bugzilla/BugUrl/JIRA.pm')
-rw-r--r-- | Bugzilla/BugUrl/JIRA.pm | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/Bugzilla/BugUrl/JIRA.pm b/Bugzilla/BugUrl/JIRA.pm index 8a7b90c3e..f5f7ee5fa 100644 --- a/Bugzilla/BugUrl/JIRA.pm +++ b/Bugzilla/BugUrl/JIRA.pm @@ -9,15 +9,16 @@ package Bugzilla::BugUrl::JIRA; use strict; use base qw(Bugzilla::BugUrl); -use Bugzilla::Error; -use Bugzilla::Util; - ############################### #### Methods #### ############################### sub should_handle { my ($class, $uri) = @_; + + # JIRA URLs have only one basic form (but the jira is optional): + # https://issues.apache.org/jira/browse/KEY-1234 + # http://issues.example.com/browse/KEY-1234 return ($uri->path =~ m|/browse/[A-Z][A-Z]+-\d+$|) ? 1 : 0; } @@ -26,10 +27,6 @@ sub _check_value { my $uri = $class->SUPER::_check_value(@_); - # JIRA URLs have only one basic form (but the jira is optional): - # https://issues.apache.org/jira/browse/KEY-1234 - # http://issues.example.com/browse/KEY-1234 - # Make sure there are no query parameters. $uri->query(undef); # And remove any # part if there is one. |