From f6142bf5bb569e67686f74cdfd959c79c7c7cb23 Mon Sep 17 00:00:00 2001
From: David Lawrence <dlawrence@mozilla.com>
Date: Thu, 27 Jan 2011 16:49:58 -0500
Subject: Bug 622679 - Autocomplete suggests inactive/disabled accounts as
 matches r/a=mkanat

---
 Bugzilla/WebService/User.pm | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/Bugzilla/WebService/User.pm b/Bugzilla/WebService/User.pm
index 3a4890433..4b11611e1 100644
--- a/Bugzilla/WebService/User.pm
+++ b/Bugzilla/WebService/User.pm
@@ -193,8 +193,9 @@ sub get {
     if ($params->{'maxusermatches'}) {
         $limit = $params->{'maxusermatches'} + 1;
     }
+    my $exclude_disabled = $params->{'include_disabled'} ? 0 : 1;
     foreach my $match_string (@{ $params->{'match'} || [] }) {
-        my $matched = Bugzilla::User::match($match_string, $limit);
+        my $matched = Bugzilla::User::match($match_string, $limit, $exclude_disabled);
         foreach my $user (@$matched) {
             if (!$unique_users{$user->id}) {
                 push(@user_objects, $user);
@@ -531,6 +532,12 @@ C<groups> is an array of names of groups that a user can be in.
 If these are specified, they limit the return value to users who are
 in I<any> of the groups specified.
 
+=item C<include_disabled> (boolean)
+
+By default, when using the C<match> parameter, disabled users are excluded
+from the returned results. Setting C<include_disabled> to C<true> will include 
+any users that are set to disabled in the returned results.
+
 =back
 
 =item B<Returns> 
@@ -609,6 +616,9 @@ function.
 
 =item C<group_ids> and C<groups> were added in Bugzilla B<4.0>.
 
+=item C<include_disabled> added in Bugzilla B<4.0>. Default behavior 
+for C<match> has changed to only returning enabled accounts.
+
 =back
 
 =back
-- 
cgit v1.2.3-24-g4f1b