Do not trigger 'pre_update' logic for HPOS option when value remains unchanged #45604
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Submission Review Guidelines:
Changes proposed in this Pull Request:
We have some
pre_update_option
logic attached to the HPOS settingwoocommerce_custom_orders_table_enabled
(which enables HPOS) but this logic is executed regardless of whether the setting is actually changing value or not.This can result in a confusing experience when a fatal error due to datastores being out of sync is triggered despite the datastore not being changed.
Closes #45225.
How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
If datastores are not in sync, run
wp wc cot sync
first if you want to change the current datastore.trunk
, so optionally check that too.wp wc cot sync
to sync orders.Changelog entry
Significance
Type
Message
Comment