diff options
| author | Dylan William Hardison <dylan@mozilla.com> | 2015-05-29 18:22:20 +0200 |
|---|---|---|
| committer | Dylan William Hardison <dylan@hardison.net> | 2015-05-29 18:22:20 +0200 |
| commit | d0acf505c5a451b8f90c84628398b42d65dabc42 (patch) | |
| tree | 71c29cd5801d791cee84ffb28d0cfe5ee52bb33d /Bugzilla/BugUrl | |
| parent | 09f53409bf172a3d6452d992e3afce7cd7879f12 (diff) | |
| download | bugzilla-d0acf505c5a451b8f90c84628398b42d65dabc42.tar.gz bugzilla-d0acf505c5a451b8f90c84628398b42d65dabc42.tar.xz | |
Bug 1165866: "See Also" allows github issues but not github pull requests
Diffstat (limited to 'Bugzilla/BugUrl')
| -rw-r--r-- | Bugzilla/BugUrl/GitHub.pm | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/Bugzilla/BugUrl/GitHub.pm b/Bugzilla/BugUrl/GitHub.pm index 63be65bed..29f3c2ea4 100644 --- a/Bugzilla/BugUrl/GitHub.pm +++ b/Bugzilla/BugUrl/GitHub.pm @@ -18,8 +18,10 @@ sub should_handle { # 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; + # GitHub pull request URLs have only one form: + # https://github.com/USER_OR_TEAM_OR_ORGANIZATION_NAME/REPOSITORY_NAME/pull/111 + return (lc($uri->authority) eq 'github.com' + and $uri->path =~ m!^/[^/]+/[^/]+/(?:issues|pull)/\d+$!) ? 1 : 0; } sub _check_value { |