Bug #8216

Make parameter of $entry->update() optional

Details

Request #8216	Make parameter of $entry->update() optional
Submitted	2006-07-14 10:10 UTC
From	b dot hallinger at skyforcesystems dot de
Assigned	beni
Status	Closed
Package	Net_LDAP
PHP Version	Irrelevant
Roadmaps	(Not assigned)

Comments

[2006-07-14 10:10 UTC] b dot hallinger at skyforcesystems dot de

Description:
------------
In the current CVS the method $entry->update() needs a Net_LDAP object as parameter:
function update(&ldap){

The Entry is usually generated through other Net_LDAP classes and knows its ldap object by the attribute $entry->_ldap.
So there is no need tu usually pass the ldap object.

My suggestion is to make the ldap parameter optional, so people still are able to provide a ldapobject if they want but aren't forced to.

At
http://beni.skyforcesystems.de/patches/net_ldap/Entry.php_1.21_update.patch
is a patch located that should apply correct to version 1.21 of Entry.php.
It will make the parameter optional and the documentation of the method more specific
Additionally it fixes a @param documentation at replace().