diff options
author | David Lawrence <dkl@mozilla.com> | 2016-03-21 16:53:10 +0100 |
---|---|---|
committer | David Lawrence <dkl@mozilla.com> | 2016-03-21 16:53:10 +0100 |
commit | 3af55bfe0bd10a85b7cd69e26a19034a6d2e78f5 (patch) | |
tree | f2358310b4344ed4b2e4ab7125cc00415db4f8d5 /Bugzilla/BugUrl | |
parent | d8e47b769c98f2fb582b8bcf8773a034e53a3ded (diff) | |
download | bugzilla-3af55bfe0bd10a85b7cd69e26a19034a6d2e78f5.tar.gz bugzilla-3af55bfe0bd10a85b7cd69e26a19034a6d2e78f5.tar.xz |
Bug 1252782 - can't add a "See Also" to a Chromium bug on bugs.chromium.org
Diffstat (limited to 'Bugzilla/BugUrl')
-rw-r--r-- | Bugzilla/BugUrl/Chromium.pm | 49 |
1 files changed, 49 insertions, 0 deletions
diff --git a/Bugzilla/BugUrl/Chromium.pm b/Bugzilla/BugUrl/Chromium.pm new file mode 100644 index 000000000..5560df24c --- /dev/null +++ b/Bugzilla/BugUrl/Chromium.pm @@ -0,0 +1,49 @@ +# 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::BugUrl::Chromium; + +use 5.10.1; +use strict; +use warnings; + +use base qw(Bugzilla::BugUrl); + +use Bugzilla::Error; +use Bugzilla::Util; + +############################### +#### Methods #### +############################### + +sub should_handle { + my ($class, $uri) = @_; + return ($uri->authority =~ /^bugs.chromium.org$/i) ? 1 : 0; +} + +sub _check_value { + my ($class, $uri) = @_; + + $uri = $class->SUPER::_check_value($uri); + + my $value = $uri->as_string; + my $project_name; + if ($uri->path =~ m|^/p/([^/]+)/issues/detail$|) { + $project_name = $1; + } else { + ThrowUserError('bug_url_invalid', { url => $value }); + } + my $bug_id = $uri->query_param('id'); + detaint_natural($bug_id); + if (!$bug_id) { + ThrowUserError('bug_url_invalid', { url => $value, reason => 'id' }); + } + + return URI->new($value); +} + +1; |