- Issue created by @Anybody
- First commit to issue fork.
- 🇩🇪Germany Grevil
Failing test is related to the deprecated "createUser" call in EstimatorTest.php:
Calling createUser() with $values as the first parameter is deprecated in drupal:10.1.0 and will be removed from drupal:11.0.0. Use createUser(array $permissions = [], $name = NULL, $admin = FALSE, array $values = []) instead. See https://www.drupal.org/node/3330762 →
I'll create a follow-up issue for this.
- 🇩🇪Germany Grevil
Ok, no need to create a follow-up issue. We should fix this in 📌 Automated Drupal 11 compatibility fixes for commerce_cart_estimate Needs review .
- 🇩🇪Germany Anybody Porta Westfalica
Great work once again @grevil! I left some comments!
As we're using the same defaults as before, this needs no update path, I think. Will be just fine like that imho.
- 🇩🇪Germany Grevil
Tests succeed again, as 🐛 Tests: Fix incorrect "createUser" call Active was just merged.
- 🇩🇪Germany Anybody Porta Westfalica
Current state as patch for composer-patches attached.
- Status changed to RTBC
25 days ago 10:56am 11 March 2025 -
jsacksick →
committed d72e55e3 on 2.x authored by
grevil →
Issue #3487146: Add settings for the output
-
jsacksick →
committed d72e55e3 on 2.x authored by
grevil →
Automatically closed - issue fixed for 2 weeks with no activity.