Commit 5ba4b1af authored by Frédéric Buclin's avatar Frédéric Buclin

Bug 607966: Use of qw(...) as parentheses is deprecated since Perl 5.13.5

r=gerv a=LpSolit
parent a6041007
...@@ -1481,7 +1481,7 @@ sub _check_dependencies { ...@@ -1481,7 +1481,7 @@ sub _check_dependencies {
my %deps_in = (dependson => $depends_on || '', blocked => $blocks || ''); my %deps_in = (dependson => $depends_on || '', blocked => $blocks || '');
foreach my $type qw(dependson blocked) { foreach my $type (qw(dependson blocked)) {
my @bug_ids = ref($deps_in{$type}) my @bug_ids = ref($deps_in{$type})
? @{$deps_in{$type}} ? @{$deps_in{$type}}
: split(/[\s,]+/, $deps_in{$type}); : split(/[\s,]+/, $deps_in{$type});
......
...@@ -159,9 +159,7 @@ sub clean_search_url { ...@@ -159,9 +159,7 @@ sub clean_search_url {
foreach my $num (1,2,3) { foreach my $num (1,2,3) {
# If there's no value in the email field, delete the related fields. # If there's no value in the email field, delete the related fields.
if (!$self->param("email$num")) { if (!$self->param("email$num")) {
foreach my $field qw(type assigned_to reporter qa_contact foreach my $field (qw(type assigned_to reporter qa_contact cc longdesc)) {
cc longdesc)
{
$self->delete("email$field$num"); $self->delete("email$field$num");
} }
} }
......
...@@ -1393,7 +1393,7 @@ sub _special_parse_email { ...@@ -1393,7 +1393,7 @@ sub _special_parse_email {
$type = "anyexact" if $type eq "exact"; $type = "anyexact" if $type eq "exact";
my @or_charts; my @or_charts;
foreach my $field qw(assigned_to reporter cc qa_contact) { foreach my $field (qw(assigned_to reporter cc qa_contact)) {
if ($params->{"email$field$id"}) { if ($params->{"email$field$id"}) {
push(@or_charts, $field, $type, $email); push(@or_charts, $field, $type, $email);
} }
......
...@@ -931,7 +931,7 @@ sub _attachment_to_hash { ...@@ -931,7 +931,7 @@ sub _attachment_to_hash {
# creator/attacher require an extra lookup, so we only send them if # creator/attacher require an extra lookup, so we only send them if
# the filter wants them. # the filter wants them.
foreach my $field qw(creator attacher) { foreach my $field (qw(creator attacher)) {
if (filter_wants $filters, $field) { if (filter_wants $filters, $field) {
$item->{$field} = $self->type('string', $attach->attacher->login); $item->{$field} = $self->type('string', $attach->attacher->login);
} }
......
...@@ -200,7 +200,7 @@ elsif ($action eq "delete") { ...@@ -200,7 +200,7 @@ elsif ($action eq "delete") {
$series->remove_from_db(); $series->remove_from_db();
# Remove (sub)categories which no longer have any series. # Remove (sub)categories which no longer have any series.
foreach my $cat qw(category subcategory) { foreach my $cat (qw(category subcategory)) {
my $is_used = $dbh->selectrow_array("SELECT COUNT(*) FROM series WHERE $cat = ?", my $is_used = $dbh->selectrow_array("SELECT COUNT(*) FROM series WHERE $cat = ?",
undef, $series->{"${cat}_id"}); undef, $series->{"${cat}_id"});
if (!$is_used) { if (!$is_used) {
......
...@@ -112,7 +112,7 @@ if (my ($category_action) = grep { $_ =~ /^categoryAction-(?:\w+)$/ } $cgi->para ...@@ -112,7 +112,7 @@ if (my ($category_action) = grep { $_ =~ /^categoryAction-(?:\w+)$/ } $cgi->para
my $type = {}; my $type = {};
$type->{$_} = $cgi->param($_) foreach $cgi->param(); $type->{$_} = $cgi->param($_) foreach $cgi->param();
# Make sure boolean fields are defined, else they fall back to 1. # Make sure boolean fields are defined, else they fall back to 1.
foreach my $boolean qw(is_active is_requestable is_requesteeble is_multiplicable) { foreach my $boolean (qw(is_active is_requestable is_requesteeble is_multiplicable)) {
$type->{$boolean} ||= 0; $type->{$boolean} ||= 0;
} }
......
...@@ -380,7 +380,7 @@ sub _create_field_values { ...@@ -380,7 +380,7 @@ sub _create_field_values {
$values{'keywords'} = create_keyword($number)->name; $values{'keywords'} = create_keyword($number)->name;
foreach my $field qw(assigned_to qa_contact reporter cc) { foreach my $field (qw(assigned_to qa_contact reporter cc)) {
$values{$field} = create_user("$number-$field")->login; $values{$field} = create_user("$number-$field")->login;
} }
...@@ -504,7 +504,7 @@ sub _create_flags { ...@@ -504,7 +504,7 @@ sub _create_flags {
my $flagtypes = _create_flagtypes($number); my $flagtypes = _create_flagtypes($number);
my %flags; my %flags;
foreach my $type qw(a b) { foreach my $type (qw(a b)) {
$flags{$type} = _get_flag_values(@_, $flagtypes->{$type}); $flags{$type} = _get_flag_values(@_, $flagtypes->{$type});
} }
return \%flags; return \%flags;
...@@ -615,9 +615,9 @@ sub _create_one_bug { ...@@ -615,9 +615,9 @@ sub _create_one_bug {
# These are necessary for the changedfrom tests. # These are necessary for the changedfrom tests.
my $extra_values = $self->_extra_bug_create_values->{$number}; my $extra_values = $self->_extra_bug_create_values->{$number};
foreach my $field qw(comments remaining_time percentage_complete foreach my $field (qw(comments remaining_time percentage_complete
keyword_objects everconfirmed dependson blocked keyword_objects everconfirmed dependson blocked
groups_in classification) groups_in classification))
{ {
$extra_values->{$field} = $bug->$field; $extra_values->{$field} = $bug->$field;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment