Skip to content

Fix RecursiveIteratorIterator template type #228

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Fix RecursiveIteratorIterator template type #228

wants to merge 1 commit into from

Conversation

simPod
Copy link
Contributor

@simPod simPod commented Jun 9, 2020

image
@ondrejmirtes
Copy link
Member

Unions aren’t supported yet. What real issues does the current stub cause?
@simPod
Copy link
Contributor Author

simPod commented Jun 9, 2020

Ah. It did not catch passing Traversable into RecursiveIteratorIterator.

https://phpstan.org/r/5e98305e-2a97-45bc-8447-3142a2e801fd
@ondrejmirtes
Copy link
Member

Can you open an issue (feature request) and show on 3v4l.org that it fails in PHP? Thanks.
@simPod
Copy link
Contributor Author

simPod commented Jun 9, 2020

Not sure what FR should I want 🤔

PHP failing example https://3v4l.org/evDe2
@ondrejmirtes
Copy link
Member

Just a normal "expected/actual output" kind of feature request.
@simPod simPod mentioned this pull request Jun 9, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@simPod @ondrejmirtes