[Phpgroupware-tracker] [Bug #2239] missing urldecode for ldap-url

[nobody]

=================== BUG #2239: LATEST MODIFICATIONS ==================
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=2239&group_id=509

Changes by: Dave Hall <address@hidden>
Date: Wed 03/19/03 at 08:14 (Australia/Melbourne)

            What     | Removed                   | Added
---------------------------------------------------------------------------
         Assigned to | skwashd                   | lkneschke


------------------ Additional Follow-up Comments ----------------------------
dunno why i didn't allocate this one earlier - sorry



=================== BUG #2239: FULL BUG SNAPSHOT ===================


Submitted by: rweilham                Project: phpGroupWare                 
Submitted on: Fri 01/17/03 at 20:38
Category:  eldaptir                   Bug Group:  0.9.14.001 release        
Severity:  5 - Major                  Priority:  High                       
Resolution:  None                     Assigned to:  lkneschke               
Status:  Open                         Component Version:  None              
Platform Version:  Linux - RedHat     Reproducibility:  Every Time          

Summary:  missing urldecode for ldap-url

Original Submission:  Whe using CN with full-name (cn=Firstname Lastname, 
ou=...) you are not able to view/edit/delete ldap-entries. The URL which is 
used to call [edit|view|delete].php is not retranslated from url to "normal" 
text.

After I have replaced the line in phpgroupware/edaptir/inc/class.ldap.inc.php 
it's working !

Old : $thisdn = $this->despace($dn);
New : $thisdn = urldecode($this->despace($dn));


Cheers 

Robert



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

-------------------------------------------------------
Date: Wed 03/19/03 at 08:14         By: skwashd
dunno why i didn't allocate this one earlier - sorry

-------------------------------------------------------
Date: Thu 01/23/03 at 21:19         By: rweilham
diff class.ldap.inc.php class.ldap.inc.php.orig
243c243
<                       $thisdn = urldecode($this->despace($dn));
---
>                       $thisdn = $this->despace($dn);

-------------------------------------------------------
Date: Thu 01/23/03 at 17:50         By: skwashd
what line number is this from?

I will apply the fix soon


CC list is empty


No files currently attached


For detailed info, follow this link:
http://savannah.gnu.org/bugs/?func=detailbug&bug_id=2239&group_id=509