Skip to content
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

Address changes from doctrine/collections 1.8.0 #2467

Merged
merged 1 commit into from
Sep 29, 2022

Conversation

greg0ire
Copy link
Member

Collection::contains() and Collection::indexOf() use templating to infer if they are going to return false or null respectively, in case $element does not even match the type of the collection.

Collection::contains() and Collection::indexOf() use templating to infer
if they are going to return false or null respectively, in case $element
does not even match the type of the collection.
@greg0ire greg0ire force-pushed the address-collections-18 branch from aea54df to 3d24c3d Compare September 29, 2022 19:11
@greg0ire greg0ire marked this pull request as ready for review September 29, 2022 19:14
@greg0ire greg0ire requested review from alcaeus and malarzm September 29, 2022 19:14
@greg0ire greg0ire merged commit ebb5c10 into doctrine:2.4.x Sep 29, 2022
@greg0ire greg0ire deleted the address-collections-18 branch September 29, 2022 21:03
@malarzm malarzm added the Task label Sep 29, 2022
@malarzm malarzm added this to the 2.4.3 milestone Sep 29, 2022
@malarzm
Copy link
Member

malarzm commented Sep 29, 2022

Thanks @greg0ire! :)

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

Successfully merging this pull request may close these issues.

3 participants