We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
This causes Psalm to crash ("could not get storage class for X") when those definitions are then referenced in other files.
This can't be reproduced on psalm.dev as multiple files are needed to trigger the error. Here's a minimal reproduction:
<?php // one.php if (true) { class One {} } else { class One {} } class Two {}
<?php // two.php new Two();
In composer.json:
composer.json
{ "autoload": { "files": ["one.php", "two.php"] } }
I get that Psalm doesn't like the conditional duplicate class definition of One, but it doesn't seem like that should prevent subsequent definitions from being parsed.
One
The text was updated successfully, but these errors were encountered:
6ad5e1c
Fix #4264 - prevent crash when analysing file with duplicate classes
ed26bb3
8375890
Fix vimeo#4264 - prevent crash when analysing file with duplicate cla…
a4b6cb8
…sses
No branches or pull requests
This causes Psalm to crash ("could not get storage class for X") when those definitions are then referenced in other files.
This can't be reproduced on psalm.dev as multiple files are needed to trigger the error. Here's a minimal reproduction:
In
composer.json
:I get that Psalm doesn't like the conditional duplicate class definition of
One
, but it doesn't seem like that should prevent subsequent definitions from being parsed.The text was updated successfully, but these errors were encountered: