[Phpgroupware-tracker] [patch #2992] ldap fixes

[Dave Hall]

This mail is an automated notification from the patch tracker
 of the project: phpGroupWare.

/**************************************************************************/
[patch #2992] Latest Modifications:

Changes by: 
                Dave Hall <address@hidden>
'Date: 
                Sat 05/01/04 at 09:50 (Australia/Melbourne)

            What     | Removed                   | Added
---------------------------------------------------------------------------
          Resolution | None                      | Fixed
              Status | Open                      | Closed







/**************************************************************************/
[patch #2992] Full Item Snapshot:

URL: <http://savannah.gnu.org/patch/?func=detailitem&item_id=2992>
Project: phpGroupWare
Submitted by: Philipp Kamps
On: Thu 04/29/04 at 15:37

Category:  API -  phpGWapi
Priority:  7 - High
Resolution:  Fixed
Assigned to:  skwashd
Originator Email:  
Status:  Closed


Summary:  ldap fixes

Original Submission:  this fix enables the lastloginfrom and lastlogin 
functionality and fixes some a missing utf8_encode for the description ldap 
attribute

Follow-up Comments
------------------


-------------------------------------------------------
Date: Fri 04/30/04 at 11:29         By: Caeies
Reading you patch, I Heavily think that the str_replace is _NEEDED_ because in 
case of lang('') failed, it will add an * which is an illegal characters in 
utf-8 as well for LDAP entries ... So in the case that the string is not 
translated (that's was the problem for french people) The ldap_insert will fail 
...

Caeies,
I will test the ldap_auth stuff in a few moment.

-------------------------------------------------------
Date: Fri 04/30/04 at 11:23         By: Caeies
There is still some problem,

Fips, could you add an utf8_decode at least in read_repository for the [cn] 
attribute ? ( near line 95 )
$this->data['fullname']               = 
utf8_decode(utf8_decode($allValues[0]['cn'][0]));

I need to add a 2x conversion because I suspect a double encoding issue. Don't 
have enougth time to find where is exatcly the pblm (need to trace the code).

There is probably some other issues of this kind




CC List
-------

CC Address                          | Comment
------------------------------------+-----------------------------
benoit --DOT-- hamet --AT-- int-evry --DOT-- fr | 



File Attachments
-------------------

-------------------------------------------------------
Date: Thu 04/29/04 at 15:37  Name: ldap.patch  Size: 2KB   By: fipsfuchs

http://savannah.gnu.org/patch/download.php?item_id=2992&amp;item_file_id=3173






For detailed info, follow this link:
<http://savannah.gnu.org/patch/?func=detailitem&item_id=2992>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/