diff --git a/mozilla/webtools/bugzilla/.bzrrev b/mozilla/webtools/bugzilla/.bzrrev index 5c242e8a1e7..865760f643c 100644 --- a/mozilla/webtools/bugzilla/.bzrrev +++ b/mozilla/webtools/bugzilla/.bzrrev @@ -1 +1 @@ -9217 \ No newline at end of file +9218 \ No newline at end of file diff --git a/mozilla/webtools/bugzilla/.gitrev b/mozilla/webtools/bugzilla/.gitrev index 1c9586df946..07059ca63a2 100644 --- a/mozilla/webtools/bugzilla/.gitrev +++ b/mozilla/webtools/bugzilla/.gitrev @@ -1 +1 @@ -82674d427a9b6f210347279dbb74da631f3d57cb \ No newline at end of file +9e1efb5e001b8540672f42bfd7101fed4a394c0e \ No newline at end of file diff --git a/mozilla/webtools/bugzilla/Bugzilla/BugUrl/SourceForge.pm b/mozilla/webtools/bugzilla/Bugzilla/BugUrl/SourceForge.pm index acba0df28b7..4a330f34b35 100644 --- a/mozilla/webtools/bugzilla/Bugzilla/BugUrl/SourceForge.pm +++ b/mozilla/webtools/bugzilla/Bugzilla/BugUrl/SourceForge.pm @@ -22,12 +22,18 @@ 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 =~ 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; + 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; } sub _check_value { @@ -38,6 +44,11 @@ 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; } diff --git a/mozilla/webtools/bugzilla/Bugzilla/Install.pm b/mozilla/webtools/bugzilla/Bugzilla/Install.pm index 5a2266e36fb..8d07eecc367 100644 --- a/mozilla/webtools/bugzilla/Bugzilla/Install.pm +++ b/mozilla/webtools/bugzilla/Bugzilla/Install.pm @@ -135,11 +135,13 @@ use constant SYSTEM_GROUPS => ( }, { name => 'bz_canusewhineatothers', - description => 'Can configure whine reports for other users', + description => 'Can configure queries and schedules for periodic' + . ' reports to be run and sent via email to other users and groups', }, { name => 'bz_canusewhines', - description => 'User can configure whine reports for self', + description => 'Can configure queries and schedules for periodic' + . ' reports to be run and sent via email to themselves', # inherited_by means that users in the groups listed below are # automatically members of bz_canusewhines. inherited_by => ['editbugs', 'bz_canusewhineatothers'], diff --git a/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm b/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm index ab02fe41db3..540845fb21e 100644 --- a/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm +++ b/mozilla/webtools/bugzilla/Bugzilla/Install/DB.pm @@ -730,6 +730,17 @@ sub update_table_definitions { $dbh->bz_add_column('longdescs', 'is_markdown', {TYPE => 'BOOLEAN', NOTNULL => 1, DEFAULT => 'FALSE'}); + $dbh->do('UPDATE groups SET description = ? WHERE name = ? and description = ?', + undef, + "Can configure queries and schedules for periodic reports to be run and sent via email to other users and groups", + "bz_canusewhineatothers", + "Can configure whine reports for other users"); + $dbh->do('UPDATE groups SET description = ? WHERE name = ? and description = ?', + undef, + "Can configure queries and schedules for periodic reports to be run and sent via email to themselves", + "bz_canusewhines", + "User can configure whine reports for self"); + ################################################################ # New --TABLE-- changes should go *** A B O V E *** this point # ################################################################ diff --git a/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl index 78d8823f5cf..015f18525a1 100644 --- a/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl +++ b/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl @@ -291,7 +291,7 @@
  • An issue in a JIRA installation.
  • A ticket in a Trac installation.
  • A b[% %]ug in a MantisBT installation.
  • -
  • A b[% %]ug on sourceforge.net.
  • +
  • A b[% %]ug or ticket on sourceforge.net.
  • An issue/pull request on github.com.
  • [% Hook.process('bug_url_invalid_tracker') %]