zendframework / zend-ldap

Ldap component from Zend Framework
BSD 3-Clause "New" or "Revised" License
29 stars 29 forks source link

Added a ldap mod replace #73

Closed glimac closed 6 years ago

glimac commented 7 years ago

I would like to add my contribution to zend-ldap by adding an updateAttributes () method.

heiglandreas commented 7 years ago

Hey @glimac. Thanks for your PR.

As I'm currently on holiday I will not be able to review the PR completely. But from a first glance I'Ve seen that there are no tests. Would you mind fixing that?

And you might want to have a look at #68 that implements the same functionality…

glimac commented 7 years ago

Hi @heiglandreas, thank you for your appreciation and performed the tests for this functionality.