From 4e5dcf363dd1ffa63f0d7a190fa61891061ddea2 Mon Sep 17 00:00:00 2001 From: Matt Selsky Date: Mon, 7 May 2012 11:57:31 -0300 Subject: Bug 571740: Add support for getsatisfaction for the See Also field r=timello, a=LpSolit --- extensions/MoreBugUrl/Extension.pm | 1 + extensions/MoreBugUrl/lib/GetSatisfaction.pm | 37 ++++++++++++++++++++++ .../user-error-bug_url_invalid_tracker.html.tmpl | 1 + 3 files changed, 39 insertions(+) create mode 100644 extensions/MoreBugUrl/lib/GetSatisfaction.pm (limited to 'extensions') diff --git a/extensions/MoreBugUrl/Extension.pm b/extensions/MoreBugUrl/Extension.pm index 1ec2fcf16..03a5ab608 100644 --- a/extensions/MoreBugUrl/Extension.pm +++ b/extensions/MoreBugUrl/Extension.pm @@ -13,6 +13,7 @@ use constant MORE_SUB_CLASSES => qw( Bugzilla::Extension::MoreBugUrl::ReviewBoard Bugzilla::Extension::MoreBugUrl::Rietveld Bugzilla::Extension::MoreBugUrl::RT + Bugzilla::Extension::MoreBugUrl::GetSatisfaction ); # We need to update bug_see_also table because both diff --git a/extensions/MoreBugUrl/lib/GetSatisfaction.pm b/extensions/MoreBugUrl/lib/GetSatisfaction.pm new file mode 100644 index 000000000..ff63b10e1 --- /dev/null +++ b/extensions/MoreBugUrl/lib/GetSatisfaction.pm @@ -0,0 +1,37 @@ +# 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::GetSatisfaction; +use strict; +use base qw(Bugzilla::BugUrl); + +############################### +#### Methods #### +############################### + +sub should_handle { + my ($class, $uri) = @_; + + # GetSatisfaction URLs only have one form: + # http(s)://getsatisfaction.com/PROJECT_NAME/topics/TOPIC_NAME + return ($uri->authority =~ /^getsatisfaction.com$/i + and $uri->path =~ m|^/[^/]+/topics/[^/]+$|) ? 1 : 0; +} + +sub _check_value { + my ($class, $uri) = @_; + + $uri = $class->SUPER::_check_value($uri); + + # GetSatisfaction HTTP URLs redirect to HTTPS, so just use the HTTPS + # scheme. + $uri->scheme('https'); + + return $uri; +} + +1; diff --git a/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl b/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl index 979a9d5dd..60d662034 100644 --- a/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl +++ b/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl @@ -9,3 +9,4 @@
  • A Review Board review request.
  • An issue in a Rietveld installation.
  • A ticket in an RT installation.
  • +
  • A topic on getsatisfaction.com.
  • -- cgit v1.2.3-24-g4f1b