Skip to content

Fix PHP 8.5 deprecation in html_cva() when a recipe value is null - #5014

Merged
fabpot merged 1 commit into
twigphp:3.xfrom
goba:3.x
Oct 25, 2026
Merged

fabpot merged 1 commit into
twigphp:3.xfrom
goba:3.x

Conversation

@goba

@goba goba commented Oct 24, 2026

Copy link
Copy Markdown

See #5013

Comment thread extra/html-extra/Tests/CvaTest.php Outdated
@goba

goba commented Oct 25, 2026

Copy link
Copy Markdown
Author

Done, thanks!

That said I believe for PHP 8.5 the run uses PHPUnit 11.3, and html-extra's phpunit.xml.dist doesn't register the bridge. Locally doing the same the test only reports the deprecation and still passes without the fix.

@fabpot

fabpot commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

Done, thanks!

That said I believe for PHP 8.5 the run uses PHPUnit 11.3, and html-extra's phpunit.xml.dist doesn't register the bridge. Locally doing the same the test only reports the deprecation and still passes without the fix.

I've done #5015 and #5016 to address this.

@fabpot

fabpot commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

Thank you @goba.

@fabpot
fabpot merged commit 2159ada into twigphp:3.x Oct 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants