Skip to content

Conversation

patrickkusebauch
Copy link
Contributor

@patrickkusebauch patrickkusebauch commented Jun 5, 2024

Closes: phpstan/phpstan#11147

Bug where I incorrectly assumed that the lists of class names and class reflections line up.

@patrickkusebauch patrickkusebauch changed the title DynamicFunctionReturnTypeExtension DynamicFunctionReturnTypeExtension bugfix Jun 5, 2024
@patrickkusebauch patrickkusebauch changed the title DynamicFunctionReturnTypeExtension bugfix get_debug_type() DynamicFunctionReturnTypeExtension bugfix Jun 5, 2024
@ondrejmirtes
Copy link
Member

Please add the failing example as a new test in AnalyserIntegrationTest (and make sure it fails first). Thanks.

@patrickkusebauch
Copy link
Contributor Author

patrickkusebauch commented Jun 5, 2024

I will, I will, I wanted to get something going and iterate on it. Need to remind myself how to run testing locally again. It has been a while.

@ondrejmirtes
Copy link
Member

Just vendor/bin/phpunit and path to the file with the test 😊

@patrickkusebauch
Copy link
Contributor Author

Thanks for the help and direction.

@patrickkusebauch

This comment was marked as resolved.

@ondrejmirtes ondrejmirtes merged commit 38f3906 into phpstan:1.11.x Jun 8, 2024
@ondrejmirtes
Copy link
Member

Thank you!

@greew greew mentioned this pull request Jun 9, 2024
@staabm staabm mentioned this pull request Jun 10, 2024
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.

Internal error: Call to a member function isFinal() on null while analysing file

2 participants