[phpldapadmin] [ phpldapadmin-Patches-2806663 ] php 5.3: ldap_first_entry and ldap_next_entry changes

Patches item #2806663, was opened at 2009-06-16 01:05
Message generated for change (Comment added) made by wurley
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498548&aid=2806663&group_id=61828

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Closed
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Francisco Luis Araña Cruz (farana)
Assigned to: Nobody/Anonymous (nobody)
Summary: php 5.3: ldap_first_entry and ldap_next_entry changes

Initial Comment:
The third parameter to ldap_first_entry and ldap_next_entry functions is now handled automatically by php 5.3.

Attached patch simply removes the third parameter in calls to those functions.

----------------------------------------------------------------------

>Comment By: Deon George (wurley)
Date: 2009-06-16 10:49

Message:
I've reversed this patch, I thought this parameter was removed in PHP5, but
not until PHP 5.2.4, which means those with earlier version of PHP get an
error. (As is the case for a RHEL and possibly other supported distros).

PLA is supported on PHP 5 (from 5.x) and since it appears
ldap_next_entry() wont throw an error if there is a 3rd argument, there is
no real reason that this patch or a modification is required yet. :)

----------------------------------------------------------------------

Comment By: Deon George (wurley)
Date: 2009-06-16 09:51

Message:
Thank you - applied to CVS. Will be available in the next SANDPIT-WIP drop.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498548&aid=2806663&group_id=61828

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
______________________________________
phpLDAPadmin development mailing list.
To unsbuscribe: https://lists.sourceforge.net/lists/listinfo/phpldapadmin-devel
http://phpldapadmin.sourceforge.net/