From d532205e6899e2fdbd0f892147a7e859b0e3bda1 Mon Sep 17 00:00:00 2001 From: Matt Selsky Date: Tue, 1 Jan 2013 23:59:13 +0100 Subject: [PATCH 001/240] Bug 816870: All extensions must require Perl 5.10.1 to pass tests r/a=LpSolit --- extensions/BmpConvert/Config.pm | 2 ++ extensions/BmpConvert/Extension.pm | 2 ++ extensions/Example/Config.pm | 2 ++ extensions/Example/Extension.pm | 2 ++ extensions/Example/lib/Auth/Login.pm | 2 ++ extensions/Example/lib/Auth/Verify.pm | 2 ++ extensions/Example/lib/Config.pm | 2 ++ extensions/Example/lib/Util.pm | 2 ++ extensions/Example/lib/WebService.pm | 2 ++ extensions/MoreBugUrl/Config.pm | 2 ++ extensions/MoreBugUrl/Extension.pm | 2 ++ extensions/MoreBugUrl/lib/GetSatisfaction.pm | 2 ++ extensions/MoreBugUrl/lib/PHP.pm | 2 ++ extensions/MoreBugUrl/lib/RT.pm | 2 ++ extensions/MoreBugUrl/lib/ReviewBoard.pm | 2 ++ extensions/MoreBugUrl/lib/Rietveld.pm | 2 ++ extensions/OldBugMove/Config.pm | 2 ++ extensions/OldBugMove/Extension.pm | 2 ++ extensions/OldBugMove/lib/Params.pm | 1 + extensions/Voting/Config.pm | 2 ++ extensions/Voting/Extension.pm | 2 ++ template/en/default/extensions/config.pm.tmpl | 2 ++ template/en/default/extensions/extension.pm.tmpl | 2 ++ template/en/default/extensions/util.pm.tmpl | 2 ++ 24 files changed, 47 insertions(+) diff --git a/extensions/BmpConvert/Config.pm b/extensions/BmpConvert/Config.pm index 42fb3ceab..91808bc4b 100644 --- a/extensions/BmpConvert/Config.pm +++ b/extensions/BmpConvert/Config.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::BmpConvert; + +use 5.10.1; use strict; use constant NAME => 'BmpConvert'; use constant REQUIRED_MODULES => [ diff --git a/extensions/BmpConvert/Extension.pm b/extensions/BmpConvert/Extension.pm index b30bab604..eb634f4a1 100644 --- a/extensions/BmpConvert/Extension.pm +++ b/extensions/BmpConvert/Extension.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::BmpConvert; + +use 5.10.1; use strict; use parent qw(Bugzilla::Extension); diff --git a/extensions/Example/Config.pm b/extensions/Example/Config.pm index b40ed9906..13635dc7f 100644 --- a/extensions/Example/Config.pm +++ b/extensions/Example/Config.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Example; + +use 5.10.1; use strict; use constant NAME => 'Example'; use constant REQUIRED_MODULES => [ diff --git a/extensions/Example/Extension.pm b/extensions/Example/Extension.pm index 7fc9072dc..c0015803e 100644 --- a/extensions/Example/Extension.pm +++ b/extensions/Example/Extension.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Example; + +use 5.10.1; use strict; use parent qw(Bugzilla::Extension); diff --git a/extensions/Example/lib/Auth/Login.pm b/extensions/Example/lib/Auth/Login.pm index 03ca2481d..c1cfe62cc 100644 --- a/extensions/Example/lib/Auth/Login.pm +++ b/extensions/Example/lib/Auth/Login.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Example::Auth::Login; + +use 5.10.1; use strict; use parent qw(Bugzilla::Auth::Login); use constant user_can_create_account => 0; diff --git a/extensions/Example/lib/Auth/Verify.pm b/extensions/Example/lib/Auth/Verify.pm index 0a0b40232..df5059a05 100644 --- a/extensions/Example/lib/Auth/Verify.pm +++ b/extensions/Example/lib/Auth/Verify.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Example::Auth::Verify; + +use 5.10.1; use strict; use parent qw(Bugzilla::Auth::Verify); use Bugzilla::Constants; diff --git a/extensions/Example/lib/Config.pm b/extensions/Example/lib/Config.pm index b0497a783..fac0046af 100644 --- a/extensions/Example/lib/Config.pm +++ b/extensions/Example/lib/Config.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Example::Config; + +use 5.10.1; use strict; use warnings; diff --git a/extensions/Example/lib/Util.pm b/extensions/Example/lib/Util.pm index b4ed3e0a4..ccc349c9c 100644 --- a/extensions/Example/lib/Util.pm +++ b/extensions/Example/lib/Util.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Example::Util; + +use 5.10.1; use strict; use warnings; diff --git a/extensions/Example/lib/WebService.pm b/extensions/Example/lib/WebService.pm index e5e14af53..c8c120616 100644 --- a/extensions/Example/lib/WebService.pm +++ b/extensions/Example/lib/WebService.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Example::WebService; + +use 5.10.1; use strict; use warnings; use parent qw(Bugzilla::WebService); diff --git a/extensions/MoreBugUrl/Config.pm b/extensions/MoreBugUrl/Config.pm index b5af9c00e..1e62306c4 100644 --- a/extensions/MoreBugUrl/Config.pm +++ b/extensions/MoreBugUrl/Config.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::MoreBugUrl; + +use 5.10.1; use strict; use constant NAME => 'MoreBugUrl'; diff --git a/extensions/MoreBugUrl/Extension.pm b/extensions/MoreBugUrl/Extension.pm index b251e70a8..5c71fa598 100644 --- a/extensions/MoreBugUrl/Extension.pm +++ b/extensions/MoreBugUrl/Extension.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::MoreBugUrl; + +use 5.10.1; use strict; use parent qw(Bugzilla::Extension); diff --git a/extensions/MoreBugUrl/lib/GetSatisfaction.pm b/extensions/MoreBugUrl/lib/GetSatisfaction.pm index 5adb897bc..4c077b1a6 100644 --- a/extensions/MoreBugUrl/lib/GetSatisfaction.pm +++ b/extensions/MoreBugUrl/lib/GetSatisfaction.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::MoreBugUrl::GetSatisfaction; + +use 5.10.1; use strict; use parent qw(Bugzilla::BugUrl); diff --git a/extensions/MoreBugUrl/lib/PHP.pm b/extensions/MoreBugUrl/lib/PHP.pm index 38795b5e6..1521cae1a 100644 --- a/extensions/MoreBugUrl/lib/PHP.pm +++ b/extensions/MoreBugUrl/lib/PHP.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::MoreBugUrl::PHP; + +use 5.10.1; use strict; use parent qw(Bugzilla::BugUrl); diff --git a/extensions/MoreBugUrl/lib/RT.pm b/extensions/MoreBugUrl/lib/RT.pm index 132118089..724456828 100644 --- a/extensions/MoreBugUrl/lib/RT.pm +++ b/extensions/MoreBugUrl/lib/RT.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::MoreBugUrl::RT; + +use 5.10.1; use strict; use parent qw(Bugzilla::BugUrl); diff --git a/extensions/MoreBugUrl/lib/ReviewBoard.pm b/extensions/MoreBugUrl/lib/ReviewBoard.pm index 0c4d83fab..08128353f 100644 --- a/extensions/MoreBugUrl/lib/ReviewBoard.pm +++ b/extensions/MoreBugUrl/lib/ReviewBoard.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::MoreBugUrl::ReviewBoard; + +use 5.10.1; use strict; use parent qw(Bugzilla::BugUrl); diff --git a/extensions/MoreBugUrl/lib/Rietveld.pm b/extensions/MoreBugUrl/lib/Rietveld.pm index f9e894032..a8bccba08 100644 --- a/extensions/MoreBugUrl/lib/Rietveld.pm +++ b/extensions/MoreBugUrl/lib/Rietveld.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::MoreBugUrl::Rietveld; + +use 5.10.1; use strict; use parent qw(Bugzilla::BugUrl); diff --git a/extensions/OldBugMove/Config.pm b/extensions/OldBugMove/Config.pm index 95648610e..a42567fee 100644 --- a/extensions/OldBugMove/Config.pm +++ b/extensions/OldBugMove/Config.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::OldBugMove; + +use 5.10.1; use strict; use constant NAME => 'OldBugMove'; __PACKAGE__->NAME; diff --git a/extensions/OldBugMove/Extension.pm b/extensions/OldBugMove/Extension.pm index 913553790..b58ffe883 100644 --- a/extensions/OldBugMove/Extension.pm +++ b/extensions/OldBugMove/Extension.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::OldBugMove; + +use 5.10.1; use strict; use parent qw(Bugzilla::Extension); use Bugzilla::Constants; diff --git a/extensions/OldBugMove/lib/Params.pm b/extensions/OldBugMove/lib/Params.pm index dbb1eb21d..c922e1faf 100644 --- a/extensions/OldBugMove/lib/Params.pm +++ b/extensions/OldBugMove/lib/Params.pm @@ -7,6 +7,7 @@ package Bugzilla::Extension::OldBugMove::Params; +use 5.10.1; use strict; use Bugzilla::Config::Common; diff --git a/extensions/Voting/Config.pm b/extensions/Voting/Config.pm index 4fefe957a..812ede60f 100644 --- a/extensions/Voting/Config.pm +++ b/extensions/Voting/Config.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Voting; + +use 5.10.1; use strict; use constant NAME => 'Voting'; diff --git a/extensions/Voting/Extension.pm b/extensions/Voting/Extension.pm index 981751a34..387d44ee0 100644 --- a/extensions/Voting/Extension.pm +++ b/extensions/Voting/Extension.pm @@ -6,6 +6,8 @@ # defined by the Mozilla Public License, v. 2.0. package Bugzilla::Extension::Voting; + +use 5.10.1; use strict; use parent qw(Bugzilla::Extension); diff --git a/template/en/default/extensions/config.pm.tmpl b/template/en/default/extensions/config.pm.tmpl index cb7195767..07ac83a41 100644 --- a/template/en/default/extensions/config.pm.tmpl +++ b/template/en/default/extensions/config.pm.tmpl @@ -13,6 +13,8 @@ [% PROCESS extensions/license.txt.tmpl %] package B[% %]ugzilla::Extension::[% name %]; + +use 5.10.1; use strict; use constant NAME => '[% name %]'; diff --git a/template/en/default/extensions/extension.pm.tmpl b/template/en/default/extensions/extension.pm.tmpl index 993bceb18..ebeb73719 100644 --- a/template/en/default/extensions/extension.pm.tmpl +++ b/template/en/default/extensions/extension.pm.tmpl @@ -13,6 +13,8 @@ [% PROCESS extensions/license.txt.tmpl %] package B[% %]ugzilla::Extension::[% name %]; + +use 5.10.1; use strict; use parent qw(B[% %]ugzilla::Extension); diff --git a/template/en/default/extensions/util.pm.tmpl b/template/en/default/extensions/util.pm.tmpl index c693c8a01..3493007f4 100644 --- a/template/en/default/extensions/util.pm.tmpl +++ b/template/en/default/extensions/util.pm.tmpl @@ -13,6 +13,8 @@ [% PROCESS extensions/license.txt.tmpl %] package B[% %]ugzilla::Extension::[% name %]::Util; + +use 5.10.1; use strict; use parent qw(Exporter); our @EXPORT = qw( From df37671cc2878484c5868300af60e46f732aa0ab Mon Sep 17 00:00:00 2001 From: Sunil Joshi Date: Wed, 2 Jan 2013 00:22:58 +0100 Subject: [PATCH 002/240] Bug 401918: describekeywords.cgi with no keywords simply displays an empty page instead of throwing an error r/a=LpSolit --- describekeywords.cgi | 3 +++ template/en/default/global/user-error.html.tmpl | 5 +++++ 2 files changed, 8 insertions(+) diff --git a/describekeywords.cgi b/describekeywords.cgi index ef0634960..4976fada1 100755 --- a/describekeywords.cgi +++ b/describekeywords.cgi @@ -24,6 +24,9 @@ my $vars = {}; Bugzilla->switch_to_shadow_db; $vars->{'keywords'} = Bugzilla::Keyword->get_all_with_bug_count(); +if (!@{$vars->{keywords}}) { + ThrowUserError("no_keywords"); +} $vars->{'caneditkeywords'} = $user->in_group("editkeywords"); print $cgi->header(); diff --git a/template/en/default/global/user-error.html.tmpl b/template/en/default/global/user-error.html.tmpl index ce80a5eef..03c26c5aa 100644 --- a/template/en/default/global/user-error.html.tmpl +++ b/template/en/default/global/user-error.html.tmpl @@ -1282,6 +1282,11 @@ [% title = "No Initial $terms.Bug Status" %] No [% terms.bug %] status is available on [% terms.bug %] creation. Please report the problem to [% Param("maintainer") %]. + + [% ELSIF error == "no_keywords" %] + [% title = "No Keywords" %] + [% admindocslinks = {'keywords.html' => 'Setting up a keyword'} %] + No keywords have been defined for this installation. [% ELSIF error == "no_new_quips" %] [% title = "No New Quips" %] From 62f2be6159ae5d990c41714bb3dfd35ec0336d12 Mon Sep 17 00:00:00 2001 From: Hugo Seabrook Date: Wed, 2 Jan 2013 00:25:33 +0100 Subject: [PATCH 003/240] Bug 415658: Allow apostrophes in email addresses r/a=LpSolit --- Bugzilla/Config/Auth.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Bugzilla/Config/Auth.pm b/Bugzilla/Config/Auth.pm index 289536bbd..699881122 100644 --- a/Bugzilla/Config/Auth.pm +++ b/Bugzilla/Config/Auth.pm @@ -82,7 +82,7 @@ sub get_param_list { { name => 'emailregexp', type => 't', - default => q:^[\\w\\.\\+\\-=]+@[\\w\\.\\-]+\\.[\\w\\-]+$:, + default => q:^[\\w\\.\\+\\-=']+@[\\w\\.\\-]+\\.[\\w\\-]+$:, checker => \&check_regexp }, From 99c572c88d359ee907b822089bcca9446e89c046 Mon Sep 17 00:00:00 2001 From: Koosha Khajeh Moogahi Date: Wed, 2 Jan 2013 00:40:17 +0100 Subject: [PATCH 004/240] Bug 353351: The name of the component in enter_bug.cgi should not be case sensitive r=glob a=LpSolit --- template/en/default/bug/create/create.html.tmpl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/template/en/default/bug/create/create.html.tmpl b/template/en/default/bug/create/create.html.tmpl index 62b2d6a60..132385ef5 100644 --- a/template/en/default/bug/create/create.html.tmpl +++ b/template/en/default/bug/create/create.html.tmpl @@ -212,7 +212,7 @@ TUI_hide_default('attachment_text_field'); [% NEXT IF NOT c.is_active %]