- Issue created by @thalemn
- Status changed to Needs review
about 1 year ago 11:17pm 5 October 2023 - 🇳🇿New Zealand fraserthompson Dunedin, New Zealand
I also experienced this issue after upgrading to Drupal 10 and the above patch fixed it.
Given that this is a requirement for Drupal 10 and Drupal 9 is deprecated in a few weeks, I think this should be higher priority.
- last update
about 1 year ago Patch Failed to Apply - 🇳🇿New Zealand fraserthompson Dunedin, New Zealand
Actually, while the above patch avoids the error it doesn't replicate the old behaviour and introduces an issue where empty rows are displayed.
This version fixes that.
- Status changed to Needs work
about 1 year ago 10:47am 12 October 2023 - 🇨🇭Switzerland Lukas von Blarer
The patch in #3 works for me. The indentation inside the new if statement could be improved though.
- Status changed to Needs review
about 1 year ago 11:36am 13 October 2023 - last update
about 1 year ago 36 pass - 🇮🇳India mrinalini9 New Delhi
Rerolled patch #3 and also addressed #4, please review it.
Thanks!
- 🇮🇳India arisen Goa
Reviewed the patch in #5 and tested on Drupal 10 installation.
Patch applies cleanly.Checking patch templates/commerce-wishlist-user-form.html.twig... Applied patch templates/commerce-wishlist-user-form.html.twig cleanly.
The Wishlist page works fine after applying the patch. Attached screenshots.
RTBC for me. - 🇺🇸United States rhovland Oregon
Can confirm #5 works on Drupal 10.1.
Strange this wasn't caught when marking the project D10 compatible considering this error even shows up in drupal-check and similar depreciation checking tools.
- 🇨🇲Cameroon stephane888
I suggest using filter,
{% for key, item in form.items if key|first != '#' %} => {% for key, item in form.items|filter((item, key) => key|first != '#') %} - Status changed to RTBC
11 months ago 8:19pm 29 December 2023 - 🇨🇦Canada SKAUGHT
Confirm #5 works on Drupal 10.2
----
have also manually tested #8 -- indeed, this will work too. I'm happy to mark Reviewed as is.. - 🇺🇦Ukraine marchuk.vitaliy Rivne, UA
I can also confirm that the patch in comment #5 fixes the issue. What is suggested in comment #8 also works fine, but the solution in comment #5 is simpler to understand what is going on there.
+1 to RTBC -
jsacksick →
committed 90ae9a4a on 8.x-3.x
Issue #3384115 by thalemn, fraserthompson, mrinalini9: Twig 3.0 issue.
-
jsacksick →
committed 90ae9a4a on 8.x-3.x
- Status changed to Fixed
8 months ago 9:27am 19 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.