[Bps-public-commit] RT-Extension-LDAPImport branch, master, updated. 0.20_01-46-ge7131a4

[Kevin Falcone]

The branch, master has been updated
       via  e7131a4d2460ea6697ecb1e5669ed0e6aed5594f (commit)
      from  2cb90e9da0ca7d01a465c57e872790a0af7c239e (commit)

Summary of changes:
 lib/RT/Extension/LDAPImport.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

- Log -----------------------------------------------------------------
commit e7131a4d2460ea6697ecb1e5669ed0e6aed5594f
Author: Kevin Falcone <falcone at bestpractical.com>
Date:   Wed Aug 4 17:57:58 2010 -0400

    Don't do a recursive user search
    
    We don't handle nested groups, and if LDAP is flat and only lists
    users in the parent group, we'll try and fail to remove the recursively
    found users from the RT group, rather than removing the nested group.

diff --git a/lib/RT/Extension/LDAPImport.pm b/lib/RT/Extension/LDAPImport.pm
index 7baa95e..8ed9fd1 100644
--- a/lib/RT/Extension/LDAPImport.pm
+++ b/lib/RT/Extension/LDAPImport.pm
@@ -695,7 +695,7 @@ sub add_group_members {
 
     my $rt_group_members = {};
     if ($args{group}) {
-        my $user_members = $group->UserMembersObj;
+        my $user_members = $group->UserMembersObj( Recursively => 0);
         while ( my $member = $user_members->Next ) {
             $rt_group_members->{$member->Name}++;
         }

-----------------------------------------------------------------------