- Issue created by @andypost
- 🇫🇷France andypost
1) /builds/issue/drupal-3488365/vendor/symfony/deprecation-contracts/function.php:25 Since twig/twig 3.15: Instantiating "Twig\Node\Node" directly is deprecated; the class will become abstract in 4.0.
- 🇫🇷France andypost
Added the most breaking changes to summary
- https://github.com/twigphp/Twig/pull/4335
- https://github.com/twigphp/Twig/pull/4398 - 🇬🇧United Kingdom longwave UK
@andypost given we are so close to 11.1/10.4 it might be worth just skipping the new deprecations and then fixing them individually in followups like we did for the last round in Twig?
- 🇫🇷France andypost
Filed plan issue 🌱 Fix twig 3.15 deprecations Active
PS: looks like few tests still needs tuning
- 🇫🇷France andypost
On other hand there's only 3 places to fix in core and tests are green https://git.drupalcode.org/project/drupal/-/merge_requests/10247
- 🇫🇷France andypost
Fixed deprecations as they are regular expressions, thanks @longwave
- 🇬🇧United Kingdom longwave UK
Looks good to me, I think we should just commit the fixes here given it is only changing Node to Nodes - not worth deferring that to a followup.
Ideally we will get this into both 11.1.0 and 10.4.0.
- 🇳🇱Netherlands bbrala Netherlands
I agree, we should commit the fixes too if possible. Don't want the same as we did with 3.14.
- 🇳🇱Netherlands bbrala Netherlands
The code looks good. The split between node and nodes did confuse me a little when seeing the namespace still there, but that does make sense.
Id opt to fix with deprecation fixes. Since, as longwace argues, its pretty small change.
+1 on rtbc
- 🇬🇧United Kingdom longwave UK
Committed and pushed to 11.x, 11.1.x, 10.5.x and 10.4.x. Thanks!
-
longwave →
committed c10262e1 on 10.4.x
Issue #3488365 by andypost: Upgrade twig/twig to 3.15.0
-
longwave →
committed c10262e1 on 10.4.x
-
longwave →
committed 9bff4f2d on 10.5.x
Issue #3488365 by andypost: Upgrade twig/twig to 3.15.0
-
longwave →
committed 9bff4f2d on 10.5.x
-
longwave →
committed 9c1da89f on 11.1.x
Issue #3488365 by andypost: Upgrade twig/twig to 3.15.0
-
longwave →
committed 9c1da89f on 11.1.x
-
longwave →
committed aa5bd859 on 11.x
Issue #3488365 by andypost: Upgrade twig/twig to 3.15.0
-
longwave →
committed aa5bd859 on 11.x
-
longwave →
committed 7f749579 on 10.4.x
Revert "Issue #3488365 by andypost: Upgrade twig/twig to 3.15.0" This...
-
longwave →
committed 7f749579 on 10.4.x
-
longwave →
committed 316004d2 on 10.5.x
Revert "Issue #3488365 by andypost: Upgrade twig/twig to 3.15.0" This...
-
longwave →
committed 316004d2 on 10.5.x
- 🇬🇧United Kingdom longwave UK
This broke various tests on 10.4/10.5:
2x: Since twig/twig 3.15: Using the "deprecated", "deprecating_package", and "alternative" options is deprecated, pass a "deprecation_info" one instead.
- 🇳🇱Netherlands spokje
Fail in tests seems like a random
BlockCacheTest
fail. RTBC-ing -
longwave →
committed a7400832 on 10.4.x
Issue #3488365 by andypost, longwave: Upgrade twig/twig to 3.15.0
-
longwave →
committed a7400832 on 10.4.x
-
longwave →
committed 860fe111 on 10.5.x
Issue #3488365 by andypost, longwave: Upgrade twig/twig to 3.15.0
-
longwave →
committed 860fe111 on 10.5.x
- 🇬🇧United Kingdom longwave UK
Committed and pushed to 10.5.x and 10.4.x, thanks!
- 🇫🇷France andypost
There's 3.16 release but looks to downgrades detected https://github.com/twigphp/Twig/blob/v3.16.0/CHANGELOG
Added it for 10.4/10.5 in 📌 Update Composer dependencies for 10.4.0-RC1 Active
Automatically closed - issue fixed for 2 weeks with no activity.