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

Improve filter_* array handling, support FILTER_REQUIRE_ARRAY #2318

Merged
merged 5 commits into from
Apr 5, 2023

Conversation

herndlm
Copy link
Contributor

@herndlm herndlm commented Apr 3, 2023

Refs: phpstan/phpstan#9134

fyi @GMQS

See https://www.php.net/manual/en/filter.filters.flags.php and https://3v4l.org/p8jA9 and

Support for more specialised arrays like constant arrays or lists as inputs could be improved still, but this is already complicated enough and I'm not sure if it's worth it. Could be added later still I suppose.

@herndlm
Copy link
Contributor Author

herndlm commented Apr 3, 2023

I assume you're busy with other things, but since it's in the same scope - support for filter_input() is still open via #2010 and should be ready :)

@ondrejmirtes ondrejmirtes merged commit 1c49ccb into phpstan:1.10.x Apr 5, 2023
@ondrejmirtes
Copy link
Member

Thank you!

@herndlm herndlm deleted the filter-function-array branch April 5, 2023 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants