From: Max Kanat-Alexander <mkanat@bugzilla.org>
Date: Mon, 14 Feb 2011 07:44:02 +0000 (-0800)
Subject: Bug 633422: Fix the documentation for User.get's include_disabled parameter
X-Git-Tag: bugzilla-3.4.11~5
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1baa76d47c1c6c2f3ad9cd67f3756865b603d8c1;p=thirdparty%2Fbugzilla.git

Bug 633422: Fix the documentation for User.get's include_disabled parameter
and make User.get check that its required parameters are passed.
r=LpSolit, a=mkanat
---

diff --git a/Bugzilla/WebService/User.pm b/Bugzilla/WebService/User.pm
index 790a9da7c2..5ff6631ac2 100644
--- a/Bugzilla/WebService/User.pm
+++ b/Bugzilla/WebService/User.pm
@@ -133,6 +133,11 @@ sub create {
 sub get {
     my ($self, $params) = validate(@_, 'names', 'ids');
 
+    defined($params->{names}) || defined($params->{ids})
+        || defined($params->{match})
+        || ThrowCodeError('params_required', 
+               { function => 'User.get', params => ['ids', 'names', 'match'] });
+
     my @user_objects;
     @user_objects = map { Bugzilla::User->check($_) } @{ $params->{names} }
                     if $params->{names};