Fixed in CVS

Please attach your fixes as a patch, not the complete code.

I found two bugs in the groupmanagement based on ldap.



first:

if the group has only one member, the memberlist doesn't work cause 
the return value of "function listUsers($dn)" in Horde/Group/ldap.php 
is a string not an array



second:

i couldn't add new member to an existing group, because of a mistake 
in "function updateGroup($group)" in Horde/Group/ldap.php.

The array $entry hast two subarrays: $entry['memberuid'] and 
$entry['memberUid']. In my case the $entry['memberUid'] is worthless 
because ldap inserts all members listed in $entry['memberuid'] and 
ignore the value with the uid of the new member in $entry['memberUid'].



I created a workaround with comments for it too (attachement), but in 
both cases there must be better solutions.



Deleting of members of the group doesn't work, too, but i had no time 
to fix it.

i apologize for my bad english.