diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2016-01-12 12:54:31 +0100 |
|---|---|---|
| committer | Gervase Markham <gerv@mozilla.org> | 2016-01-15 12:35:40 +0100 |
| commit | a0c3ade6d450ee0403612199a96db5bb8a577006 (patch) | |
| tree | ea0aa5c4c583ba147e83de2096a74094717046d7 | |
| parent | 35429919e6438ee05a5bc9413952cc30fed9e6b6 (diff) | |
| download | bugzilla-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.pm | 11 |
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); |