Skip to content
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

Closed
SignpostMarv opened this issue Feb 20, 2023 · 6 comments
Closed

php 8 interface method used in ErrorBaseline #9352

SignpostMarv opened this issue Feb 20, 2023 · 6 comments

Comments

@SignpostMarv
Copy link
Contributor

#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

@psalm-github-bot
Copy link

Hey @SignpostMarv, can you reproduce the issue on https://psalm.dev ?

@SignpostMarv
Copy link
Contributor Author

this impacts psalm 5.7.0 and 5.7.1

@othercorey
Copy link
Contributor

@SignpostMarv Did the patch fix your crash?

@SignpostMarv
Copy link
Contributor Author

@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.

@othercorey
Copy link
Contributor

I've created a separate issue to trach the architectural issue vs this crash from a BC break.

@weirdan
Copy link
Collaborator

weirdan commented Feb 24, 2023

The crash is fixed and the rest is tracked in #9370

@weirdan weirdan closed this as completed Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants