Skip to content

added regression test - #1946

Merged
ondrejmirtes merged 3 commits into
phpstan:1.9.xfrom
staabm:bug7913
Nov 1, 2022
Merged

added regression test#1946
ondrejmirtes merged 3 commits into
phpstan:1.9.xfrom
staabm:bug7913

Conversation

@staabm

@staabm staabm commented Nov 1, 2022

Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried wrapping it in a method/function, but using const like in the repro example, only works in top level code, it seems

@ondrejmirtes

Copy link
Copy Markdown
Member

I'd like a NodeScopeResolverTest with assertType before and after empty().

@ondrejmirtes

Copy link
Copy Markdown
Member

Rebase please :) (always make sure you're on the latest commit before starting the work).

@ondrejmirtes
ondrejmirtes merged commit 7b5a77e into phpstan:1.9.x Nov 1, 2022
@ondrejmirtes

Copy link
Copy Markdown
Member

Thank you!

@staabm
staabm deleted the bug7913 branch November 1, 2022 09:52
ondrejmirtes added a commit that referenced this pull request Sep 11, 2026
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014LVEGd9G9w8j64EZQ7rysC
ondrejmirtes added a commit that referenced this pull request Sep 11, 2026
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014LVEGd9G9w8j64EZQ7rysC
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.

False positive "Empty array passed to foreach" only for constant arrays in 1.8.3

2 participants