search

zendframework / zend-ldap

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

Fixes issue #19 #20

Closed heiglandreas closed 8 years ago

heiglandreas commented 9 years ago

This PR checks whether the LDAP-connection shall use SSL or not and decides based on that which port to use if no port has been set.

This fixes issue #19

Maks3w commented 9 years ago

Need unit tests

Maks3w commented 9 years ago

Its needed a test for connect() params too.

Personally I would try to compact all the existing tests around the port feature.

heiglandreas commented 8 years ago

Hey @Maks3w How would you try to compact the existing tests around the port-feature? I might be a bit slow but I don't actually know how to approach that. Thanks for your help!