summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2016-01-12 12:54:31 +0100
committerGervase Markham <gerv@mozilla.org>2016-01-15 12:35:40 +0100
commita0c3ade6d450ee0403612199a96db5bb8a577006 (patch)
treeea0aa5c4c583ba147e83de2096a74094717046d7
parent35429919e6438ee05a5bc9413952cc30fed9e6b6 (diff)
downloadbugzilla-a0c3ade6d450ee0403612199a96db5bb8a577006.tar.gz
bugzilla-a0c3ade6d450ee0403612199a96db5bb8a577006.tar.xz
Bug 1229994: MoreBugUrl: Savane: support more sub-urls than just bugs. r=gerv.
-rw-r--r--extensions/MoreBugUrl/lib/Savane.pm11
1 files changed, 6 insertions, 5 deletions
diff --git a/extensions/MoreBugUrl/lib/Savane.pm b/extensions/MoreBugUrl/lib/Savane.pm
index efda1fa4f..5b35bbf7d 100644
--- a/extensions/MoreBugUrl/lib/Savane.pm
+++ b/extensions/MoreBugUrl/lib/Savane.pm
@@ -19,7 +19,12 @@ use parent qw(Bugzilla::BugUrl);
sub should_handle {
my ($class, $uri) = @_;
- return ($uri->as_string =~ m|/bugs/(index\.php)?\?\d+$|) ? 1 : 0;
+ # Savane URLs look like the following (the index.php is optional):
+ # https://savannah.gnu.org/bugs/index.php?107657
+ # https://savannah.gnu.org/patch/index.php?107657
+ # https://savannah.gnu.org/support/index.php?107657
+ # https://savannah.gnu.org/task/index.php?107657
+ return ($uri->as_string =~ m|/(bugs\|patch\|support\|task)/(index\.php)?\?\d+$|) ? 1 : 0;
}
sub _check_value {
@@ -27,10 +32,6 @@ sub _check_value {
my $uri = $class->SUPER::_check_value(@_);
- # Savane URLs have only two forms:
- # http://gna.org/bugs/index.php?12345
- # http://gna.org/bugs/?12345
-
# And remove any # part if there is one.
$uri->fragment(undef);