patchphpGroupWare - Patches: patch #2992, ldap fixes

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #2992: ldap fixes

Submitter:  Philipp Kamps <fipsfuchs>
Submitted:  Thu 29 Apr 2004 03:37:00 PM UTC
   
 
Category:  API - phpGWapi Priority:  7 - High
Item Group:  None Status:  Done
Privacy:  Public Assigned to:  skwashd
Open/Closed:  Closed

Fri 30 Apr 2004 11:29:30 AM UTC, comment #2: 

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.

Caeies <Caeies>
Group administrator
Fri 30 Apr 2004 11:23:24 AM UTC, comment #1: 

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

Caeies <Caeies>
Group administrator
Thu 29 Apr 2004 03:37:00 PM UTC, original submission:  

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

Philipp Kamps <fipsfuchs>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #7368:  ldap.patch added by fipsfuchs (2KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 4 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2004-05-01 skwashd StatusNone Done
    Open/ClosedOpen Closed
2004-04-29 fipsfuchs Attached File- Added ldap.patch, #3173
    Carbon-Copy- Added benoit --DOT-- hamet --AT-- int-evry --DOT-- fr

Back to the top

Powered by Savane 3.13-758e.
Corresponding source code