Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
There is an update dependencies conflict.
We need to run commerce_product_update_8208 after commerce_order_update_8207
Comment | File | Size | Author |
---|---|---|---|
#4 | 3104218-4-fix-update-from-213.patch | 1020 bytes | bojanz |
| |||
#2 | 3104218-2.patch | 967 bytes | facine |
Comments
Comment #2
facine CreditAttribution: facine as a volunteer and at Cambrico commentedComment #3
bojanz CreditAttribution: bojanz at Centarro commentedDid this patch make the update process succeed on your end?
I am assuming that the error happens because installing the submodule clears the caches before the field definitions are updated, but this patch only makes the product updates happen before, the order's order_items field is still converted after the submodule is installed. So my theory might be wrong.
Here's a review:
1) We need to account for installs that use commerce_order without commerce_product, remove no dependencies if commerce_product is not installed.
2) Let's depend on commerce_product_update_8209 instead of commerce_product_update_8208 since that completes the base field conversion process (by fixing the Views).
3) The code comment is wrong, it is ensuring that commerce_order_update_8207 is executed after, not before the product update.
Comment #4
bojanz CreditAttribution: bojanz at Centarro commentedHere's an updated patch.
Do you still have a backup you could test this one? Just in case the change to commerce_product_update_8209 broke anything.
Comment #5
facine CreditAttribution: facine as a volunteer and at Cambrico commented#4 works fine, thanks!
Comment #6
bojanz CreditAttribution: bojanz at Centarro commentedApparently I'm still asleep today.
Committed in https://www.drupal.org/commitlog/commit/11974/ff92243a51f29ce0f7f66e9116... with the wrong commit message.
Comment #8
idefix6 CreditAttribution: idefix6 commentedI upgraded now to the current dev version with:
composer require 'drupal/commerce:2.x-dev'
I see:
But if I try to execute:
./vendor/bin/drush cache-rebuild --uri drupal8.xxxx.de
I get the error:
the site is completely broken and does not work anymore.
Do you have an idea how to fix this, that the site can continue to work?
Comment #9
facine CreditAttribution: facine as a volunteer and at Cambrico commentedI think you need to execute:
./vendor/bin/drush updatedb --uri drupal8.xxxx.de
Comment #10
idefix6 CreditAttribution: idefix6 commentedYes, but I only get:
So it seems that I have to enforce this update in a way I do not know.
Can you maybe help here, please?
Comment #11
facine CreditAttribution: facine as a volunteer and at Cambrico commentedTry with http://your.site/update.php
Comment #12
idefix6 CreditAttribution: idefix6 commentedI get in the logfile:
Comment #13
idefix6 CreditAttribution: idefix6 commentedWith version 2.17 and the new drupal 8.8.3 it works now.
Maybe drupal 8.8.3 has a bug in core that blocked this (at least for me).
Thanks a lot!
Comment #14
Honza Pobořil CreditAttribution: Honza Pobořil as a volunteer commentedUpdate did not worked by
drupal update:execute
but by update.php worked (after backup restore).