diff options
author | Byron Jones <bjones@mozilla.com> | 2014-02-06 07:44:21 +0100 |
---|---|---|
committer | Byron Jones <bjones@mozilla.com> | 2014-02-06 07:44:21 +0100 |
commit | a03987f1d0d7fe615b0c632cf2079b4c0a791a4a (patch) | |
tree | 1ebc84df86f365bb10e1ca9e4d0cbe714f76d803 /extensions/TagNewUsers | |
parent | e944ebab1ef01993751c75d5fd25ec6b590cb858 (diff) | |
download | bugzilla-a03987f1d0d7fe615b0c632cf2079b4c0a791a4a.tar.gz bugzilla-a03987f1d0d7fe615b0c632cf2079b4c0a791a4a.tar.xz |
Bug 967607: User.get shouldn't load data that requires extra database queries unless required
Diffstat (limited to 'extensions/TagNewUsers')
-rw-r--r-- | extensions/TagNewUsers/Extension.pm | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/extensions/TagNewUsers/Extension.pm b/extensions/TagNewUsers/Extension.pm index 13a517406..599504225 100644 --- a/extensions/TagNewUsers/Extension.pm +++ b/extensions/TagNewUsers/Extension.pm @@ -11,6 +11,7 @@ use base qw(Bugzilla::Extension); use Bugzilla::Field; use Bugzilla::User; use Bugzilla::Install::Util qw(indicate_progress); +use Bugzilla::WebService::Util qw(filter_wants); use Date::Parse; use Scalar::Util qw(blessed); @@ -245,6 +246,8 @@ sub webservice_user_get { my ($self, $args) = @_; my ($webservice, $params, $users) = @$args{qw(webservice params users)}; + return unless filter_wants($params, 'is_new'); + foreach my $user (@$users) { # Most of the time the hash values are XMLRPC::Data objects my $email = blessed $user->{'email'} ? $user->{'email'}->value : $user->{'email'}; |