diff --git a/webtools/bugzilla/CGI.pl b/webtools/bugzilla/CGI.pl index c12ce8e5e7c..0d9762b222a 100644 --- a/webtools/bugzilla/CGI.pl +++ b/webtools/bugzilla/CGI.pl @@ -29,11 +29,9 @@ require 'globals.pl'; sub GeneratePersonInput { my ($field, $required, $def_value, $extraJavaScript) = (@_); - if (!defined $extraJavaScript) { - $extraJavaScript = ""; - } + $extraJavaScript ||= ""; if ($extraJavaScript ne "") { - $extraJavaScript = "onChange=\" $extraJavaScript \""; + $extraJavaScript = "onChange=\"$extraJavaScript\""; } return ""; } diff --git a/webtools/bugzilla/buglist.cgi b/webtools/bugzilla/buglist.cgi index ee4aa39fd3c..8d3182200dd 100755 --- a/webtools/bugzilla/buglist.cgi +++ b/webtools/bugzilla/buglist.cgi @@ -546,14 +546,13 @@ while (@row = FetchSQLData()) { next; } my $value = shift @row; - my $nowrap = ""; if ($::needquote{$c}) { $value = html_quote($value); } else { $value = "$value"; } - pnl "$value"; + pnl "$value"; } if ($dotweak) { my $value = shift @row; diff --git a/webtools/bugzilla/query.cgi b/webtools/bugzilla/query.cgi index bee45e9f410..5f8d362149b 100755 --- a/webtools/bugzilla/query.cgi +++ b/webtools/bugzilla/query.cgi @@ -57,7 +57,7 @@ if (!defined $::buffer || $::buffer eq "") { $::buffer = $::COOKIE{"DEFAULTQUERY"}; } -my %default; +use vars qw(%default); my %type; foreach my $name ("bug_status", "resolution", "assigned_to", "rep_platform", diff --git a/webtools/bugzilla/reports.cgi b/webtools/bugzilla/reports.cgi index c7a22fcd32d..21cfd780e37 100755 --- a/webtools/bugzilla/reports.cgi +++ b/webtools/bugzilla/reports.cgi @@ -261,6 +261,9 @@ FIN # start painting report # ######################### + $bugs_status{'NEW'} ||= '0'; + $bugs_status{'ASSIGNED'} ||= '0'; + $bugs_status{'REOPENED'} ||= '0'; print <$quip