From 7a87be5913af9ac8b82552a6835fc07163dc520b Mon Sep 17 00:00:00 2001
From: Matt Selsky <selsky@columbia.edu>
Date: Wed, 18 Apr 2012 14:37:58 +0200
Subject: Bug 704999: Add support for GitHub for the 'See Also' field r=timello
 a=LpSolit

---
 Bugzilla/BugUrl.pm                              |  1 +
 Bugzilla/BugUrl/GitHub.pm                       | 36 +++++++++++++++++++++++++
 template/en/default/global/user-error.html.tmpl |  1 +
 3 files changed, 38 insertions(+)
 create mode 100644 Bugzilla/BugUrl/GitHub.pm

diff --git a/Bugzilla/BugUrl.pm b/Bugzilla/BugUrl.pm
index 888b11398..c48a33378 100644
--- a/Bugzilla/BugUrl.pm
+++ b/Bugzilla/BugUrl.pm
@@ -57,6 +57,7 @@ use constant SUB_CLASSES => qw(
     Bugzilla::BugUrl::Trac
     Bugzilla::BugUrl::MantisBT
     Bugzilla::BugUrl::SourceForge
+    Bugzilla::BugUrl::GitHub
 );
 
 ###############################
diff --git a/Bugzilla/BugUrl/GitHub.pm b/Bugzilla/BugUrl/GitHub.pm
new file mode 100644
index 000000000..63be65bed
--- /dev/null
+++ b/Bugzilla/BugUrl/GitHub.pm
@@ -0,0 +1,36 @@
+# 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::GitHub;
+use strict;
+use base qw(Bugzilla::BugUrl);
+
+###############################
+####        Methods        ####
+###############################
+
+sub should_handle {
+    my ($class, $uri) = @_;
+
+    # GitHub issue URLs have only one form:
+    #  https://github.com/USER_OR_TEAM_OR_ORGANIZATION_NAME/REPOSITORY_NAME/issues/111
+    return ($uri->authority =~ /^github.com$/i
+            and $uri->path =~ m|^/[^/]+/[^/]+/issues/\d+$|) ? 1 : 0;
+}
+
+sub _check_value {
+    my ($class, $uri) = @_;
+
+    $uri = $class->SUPER::_check_value($uri);
+
+    # GitHub HTTP URLs redirect to HTTPS, so just use the HTTPS scheme.
+    $uri->scheme('https');
+
+    return $uri;
+}
+
+1;
diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl
index 3b2eca6d4..72d0241a0 100644
--- a/template/en/default/global/user-error.html.tmpl
+++ b/template/en/default/global/user-error.html.tmpl
@@ -254,6 +254,7 @@
         <li>A ticket in a Trac installation.</li>
         <li>A b[% %]ug in a MantisBT installation.</li>
         <li>A b[% %]ug on sourceforge.net.</li>
+        <li>An issue on github.com.</li>
         [% Hook.process('bug_url_invalid_tracker') %]
       </ul>
     [% ELSIF reason == 'id' %]
-- 
cgit v1.2.3-24-g4f1b