[horde] CVS HEAD: Problem with LDAP Groups

[Mike Peachey]

Mike Peachey wrote:
> Jan Schneider wrote:
>> Zitat von Mike Peachey <mike.peachey at jennic.com>:
>>
>>> Jan Schneider wrote:
>>>> Zitat von Mike Peachey <mike.peachey at jennic.com>:
>>>>
>>>>> Jan Schneider wrote:
>>>>>> Zitat von Mike Peachey <mike.peachey at jennic.com>:
>>>>>> Try this patch:
>>>>>> http://cvs.horde.org/diff.php?f=framework%2FGroup%2FGroup%2Fldap.php&r1=1.18.2.7&r2=1.18.2.8&t=unified 
>>>>>> Jan.
>>>>>>
>>>>> Should I replace the whole file with one of the more recent versions?
>>>> You could try, but I wouldn't recommend it.
>>> So should I manually make those two line changes to my ldap.php?
>>>
>>> (As attempting to apply 
>>> http://cvs.horde.org/diff.php?f=framework%2FGroup%2FGroup%2Fldap.php&r1=1.18.2.7&r2=1.18.2.8&t=unified 
>>> fails because my version is 1.18.2.5
>> Yes, unless only the first chunk fails, which is fine.
> 
> No, there's a lot of failure due to the HEAD merge at 1.18.2.6 - makes 5 
> very different to 7, so will just change those two lines manually (so 
> long as you recommend not including the changes through the HEAD merge 
> as well).
> 
> Thank you.

Once worked out where to make the changes, and wondered for a while 
about the difference between strtolower() and String::lower() - I 
applied the two lines and it now works!

Thank you again!

Roll on v1.1 release!
-- 
Kind Regards,

__________________________________________________

Mike Peachey, IT
Tel: +44 114 281 2655
Fax: +44 114 281 2951
Jennic Ltd, Furnival Street, Sheffield, S1 4QT, UK
Comp Reg No: 3191371 - Registered In England
http://www.jennic.com
__________________________________________________