Skip to content

[Ldap] Update ldap.rst #15208

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 13, 2021
Merged

[Ldap] Update ldap.rst #15208

merged 1 commit into from
Apr 13, 2021

Conversation

jraller
Copy link
Contributor

@jraller jraller commented Apr 8, 2021

include documentation of the filter option for the query command.

@OskarStark
Copy link
Contributor

Is this code already available in Symfony 4.4?

@jraller
Copy link
Contributor Author

jraller commented Apr 12, 2021

Is this code already available in Symfony 4.4?

Using blame to look at /Adapter/ExtLdap/Query.php in the callSearchFunction line shows that there was a refactor on 2020-10-07, but that the code that was refactored had $this->options['filter'] already at that point.

I'm seeing this as far back as 3 years ago, so I'm feeling like it was available in 4.4.

@carsonbot carsonbot changed the title Update ldap.rst [Ldap] Update ldap.rst Apr 12, 2021
@OskarStark OskarStark added this to the 4.4 milestone Apr 12, 2021
@OskarStark
Copy link
Contributor

Thanks for your feedback ✅

@javiereguiluz javiereguiluz changed the base branch from 5.2 to 4.4 April 13, 2021 06:55
@javiereguiluz javiereguiluz requested a review from xabbuh as a code owner April 13, 2021 06:55
@javiereguiluz javiereguiluz merged commit 16f1d78 into symfony:4.4 Apr 13, 2021
@javiereguiluz
Copy link
Member

Jason, thanks a lot for adding these missing docs and for taking the time to investigate where should we merge this (we merged it in 4.4 and up). Also, congrats on your first Symfony Docs contribution 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants