Bug 891236 - Allow Redmine See Also URLs

r=justdave, a=justdave


git-svn-id: svn://10.0.0.236/trunk@264978 18797224-902f-48f8-a5cc-f745e15eee43

mozilla/webtools/bugzilla/.bzrrev

@@ -1 +1 @@
-8708
+8709

mozilla/webtools/bugzilla/extensions/MoreBugUrl/Extension.pm

@@ -18,6 +18,7 @@ use constant MORE_SUB_CLASSES => qw(
     Bugzilla::Extension::MoreBugUrl::RT
     Bugzilla::Extension::MoreBugUrl::GetSatisfaction
     Bugzilla::Extension::MoreBugUrl::PHP
+    Bugzilla::Extension::MoreBugUrl::Redmine
 );
 
 # We need to update bug_see_also table because both

mozilla/webtools/bugzilla/extensions/MoreBugUrl/lib/Redmine.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::Redmine;
+use strict;
+use base qw(Bugzilla::BugUrl);
+
+###############################
+####        Methods        ####
+###############################
+
+sub should_handle {
+    my ($class, $uri) = @_;
+    return ($uri->path =~ m|/issues/\d+$|) ? 1 : 0;
+}
+
+sub _check_value {
+    my $class = shift;
+
+    my $uri = $class->SUPER::_check_value(@_);
+
+    # Redmine URLs have only one form:
+    #   http://demo.redmine.com/issues/111
+
+    # Make sure there are no query parameters.
+    $uri->query(undef);
+    # And remove any # part if there is one.
+    $uri->fragment(undef);
+
+    return $uri;
+}
+
+1;

mozilla/webtools/bugzilla/extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl

@@ -12,3 +12,4 @@
 <li>A ticket in an RT installation.</li>
 <li>A topic on getsatisfaction.com.</li>
 <li>A b[% %]ug on b[% %]ugs.php.net.</li>
+<li>An issue in a Redmine installation.</li>