Le vendredi 6 novembre 2015, 11:43:58 Ferenc Kovacs a écrit : > > on the other hand functions like ftp_connect assumes the default port when > passed NULL: > > I'm fine with changing this for future versions but my understanding is > that this BC break (along with some others) was introduced in a micro > release (5.6.11 afair), so at least for 5.6 I would prefer restoring the > old behavior and for 7.0 I'm fine with the change but please document it in > NEWS and UPGRADING so we can incorporate it in the docs. >
Ok, so I pushed a fix to treat NULL (and 0 for that matter, same as in ftp_connect) into the default ldap port. I pushed only to 5.6 as I think for 7 we should keep the error, it does not makes much sense to accept NULL as port and the documentation does not mention it. MCMic -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
