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
php 8 interface method used in ErrorBaseline #9352
Comments
Hey @SignpostMarv, can you reproduce the issue on https://psalm.dev ? |
this impacts psalm 5.7.0 and 5.7.1 |
@SignpostMarv Did the patch fix your crash? |
@othercorey that appears to fix the crash, yes- so that covers the bug introduced by #9184, but the issue introduced by #8140 that you noted in your comment on #9354 (re: the php-versioned stubs) hasn't been fixed so the bug can't be closed yet. |
I've created a separate issue to trach the architectural issue vs this crash from a BC break. |
The crash is fixed and the rest is tracked in #9370 |
#9184 introduced a call to
DOMParentNode::append()
, which doesn't exist on 7.4#8140 added the method to the stub, but didn't flag it as only being present from 8.x onwards, so it's falsely flagged as being fine
expected behaviour: set baseline should behave on php 7.4
actual behaviour: set baseline fails due to undefined method on
DOMElement
The text was updated successfully, but these errors were encountered: