diff options
author | Frank Becker <Frank@Frank-Becker.de> | 2012-07-31 00:28:36 -0400 |
---|---|---|
committer | Dave Lawrence <dlawrence@mozilla.com> | 2012-07-31 00:28:36 -0400 |
commit | 75f87ced8b8ac2d4b3cf025dc3cff5bac596bb70 (patch) | |
tree | 246901e823a1e433ee982db4e104315cb3e77b37 /Bugzilla | |
parent | 4712659894b891e28d17a5475babe4c656f2060e (diff) | |
download | bugs-75f87ced8b8ac2d4b3cf025dc3cff5bac596bb70.tar bugs-75f87ced8b8ac2d4b3cf025dc3cff5bac596bb70.tar.gz bugs-75f87ced8b8ac2d4b3cf025dc3cff5bac596bb70.tar.bz2 bugs-75f87ced8b8ac2d4b3cf025dc3cff5bac596bb70.tar.xz bugs-75f87ced8b8ac2d4b3cf025dc3cff5bac596bb70.zip |
Bug 775056 - Add prefix to filter() and filter_wants()
r=dkl, a=LpSolit
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/WebService/Util.pm | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/Bugzilla/WebService/Util.pm b/Bugzilla/WebService/Util.pm index ac57549f2..83b925015 100644 --- a/Bugzilla/WebService/Util.pm +++ b/Bugzilla/WebService/Util.pm @@ -22,22 +22,24 @@ our @EXPORT_OK = qw( params_to_objects ); -sub filter ($$) { - my ($params, $hash) = @_; +sub filter ($$;$) { + my ($params, $hash, $prefix) = @_; my %newhash = %$hash; foreach my $key (keys %$hash) { - delete $newhash{$key} if !filter_wants($params, $key); + delete $newhash{$key} if !filter_wants($params, $key, $prefix); } return \%newhash; } -sub filter_wants ($$) { - my ($params, $field) = @_; +sub filter_wants ($$;$) { + my ($params, $field, $prefix) = @_; my %include = map { $_ => 1 } @{ $params->{'include_fields'} || [] }; my %exclude = map { $_ => 1 } @{ $params->{'exclude_fields'} || [] }; + $field = "${prefix}.${field}" if $prefix; + if (defined $params->{include_fields}) { return 0 if !$include{$field}; } @@ -150,6 +152,13 @@ of WebService methods. Given a hash (the second argument to this subroutine), this will remove any keys that are I<not> in C<include_fields> and then remove any keys that I<are> in C<exclude_fields>. +An optional third option can be passed that prefixes the field name to allow +filtering of data two or more levels deep. + +For example, if you want to filter out the C<id> key/value in components returned +by Product.get, you would use the value C<component.id> in your C<exclude_fields> +list. + =head2 filter_wants Returns C<1> if a filter would preserve the specified field when passing |