Oops. Revert commit with bogus message #1. paperbag=gerv.

This reverts commit 9e1efb5e001b8540672f42bfd7101fed4a394c0e.

1 files changed, 5 insertions, 16 deletions

diff --git a/Bugzilla/BugUrl/SourceForge.pm b/Bugzilla/BugUrl/SourceForge.pm
index 4a330f34b..acba0df28 100644
--- a/Bugzilla/BugUrl/SourceForge.pm
+++ b/Bugzilla/BugUrl/SourceForge.pm
@@ -22,18 +22,12 @@ sub should_handle {
 
     # SourceForge tracker URLs have only one form:
     #  http://sourceforge.net/tracker/?func=detail&aid=111&group_id=111&atid=111
-    # SourceForge Allura ticket URLs have several forms:
-    #  http://sourceforge.net/p/project/bugs/12345/
-    #  http://sourceforge.net/p/project/feature-requests/12345/
-    #  http://sourceforge.net/p/project/patches/12345/
-    #  http://sourceforge.net/p/project/support-requests/12345/
     return (lc($uri->authority) eq 'sourceforge.net'
-            and (($uri->path eq '/tracker/'
-                  and $uri->query_param('func') eq 'detail'
-                  and $uri->query_param('aid')
-                  and $uri->query_param('group_id')
-                  and $uri->query_param('atid'))
-                or $uri->path =~ m!^/p/[^/]+/(?:bugs|feature-requests|patches|support-requests)/\d+/?$!)) ? 1 : 0;
+            and $uri->path =~ m|/tracker/|
+            and $uri->query_param('func') eq 'detail'
+            and $uri->query_param('aid')
+            and $uri->query_param('group_id')
+            and $uri->query_param('atid')) ? 1 : 0;
 }
 
 sub _check_value {
@@ -44,11 +38,6 @@ sub _check_value {
     # Remove any # part if there is one.
     $uri->fragment(undef);
 
-    # Make sure the trailing slash is present
-    my $path = $uri->path;
-    $path =~ s!/*$!/!;
-    $uri->path($path);
-
     return $uri;
 }