summaryrefslogtreecommitdiffstats
path: root/Bugzilla/WebService
diff options
context:
space:
mode:
authorDave Lawrence <dlawrence@mozilla.com>2013-01-09 23:53:09 +0100
committerDave Lawrence <dlawrence@mozilla.com>2013-01-09 23:53:09 +0100
commit5c5d1c59f0a0d16032662e9d61a7c859715efb18 (patch)
treeb759c9477da8c56e7789038c64336f3b9d965342 /Bugzilla/WebService
parent864425625db466dbbbc346e41d2acddbce101ecf (diff)
downloadbugzilla-5c5d1c59f0a0d16032662e9d61a7c859715efb18.tar.gz
bugzilla-5c5d1c59f0a0d16032662e9d61a7c859715efb18.tar.xz
Fixed filter_wants in Bugzilla::WebService::Bug to work properly with exludes/includes. Founds by automated webservice tests
Diffstat (limited to 'Bugzilla/WebService')
-rw-r--r--Bugzilla/WebService/Util.pm5
1 files changed, 2 insertions, 3 deletions
diff --git a/Bugzilla/WebService/Util.pm b/Bugzilla/WebService/Util.pm
index feefd47af..dbf670bf7 100644
--- a/Bugzilla/WebService/Util.pm
+++ b/Bugzilla/WebService/Util.pm
@@ -49,15 +49,14 @@ sub filter_wants ($$;$) {
my ($params, $field, $prefix) = @_;
my %include = map { $_ => 1 } @{ $params->{'include_fields'} || [] };
my %exclude = map { $_ => 1 } @{ $params->{'exclude_fields'} || [] };
- my $field_temp;
$field = "${prefix}.${field}" if $prefix;
if (defined $params->{include_fields}) {
- return 0 if !$include{$field_temp};
+ return 0 if !$include{$field};
}
if (defined $params->{exclude_fields}) {
- return 0 if $exclude{$field_temp};
+ return 0 if $exclude{$field};
}
return 1;