Sync - HPOS: Update checksum fields to consider (#38713)

* Change checksum fields for HPOS tables * Changelog * Version bump * Version bump * Version bump
Automattic · Aug 6, 2024 · c9d75a1 · c9d75a1
1 parent c189085
commit c9d75a1
Show file tree

Hide file tree

Showing 28 changed files with 94 additions and 28 deletions.
diff --git a/projects/packages/sync/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/packages/sync/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,4 @@
+Significance: minor
+Type: changed
+
+Sync: Change fields for HPOS checksums
diff --git a/projects/packages/sync/composer.json b/projects/packages/sync/composer.json
@@ -59,7 +59,7 @@
 			"link-template": "https://github.com/Automattic/jetpack-sync/compare/v${old}...v${new}"
 		},
 		"branch-alias": {
-			"dev-trunk": "3.4.x-dev"
+			"dev-trunk": "3.5.x-dev"
 		},
 		"dependencies": {
 			"test-only": [

diff --git a/projects/packages/sync/src/class-package-version.php b/projects/packages/sync/src/class-package-version.php
@@ -12,7 +12,7 @@
  */
 class Package_Version {
 
-	const PACKAGE_VERSION = '3.4.1';
+	const PACKAGE_VERSION = '3.5.0-alpha';
 
 	const PACKAGE_SLUG = 'sync';
 

diff --git a/projects/packages/sync/src/replicastore/class-table-checksum.php b/projects/packages/sync/src/replicastore/class-table-checksum.php
@@ -312,7 +312,8 @@ protected function get_default_tables() {
 				'table'                     => "{$wpdb->prefix}wc_orders",
 				'range_field'               => 'id',
 				'key_fields'                => array( 'id' ),
-				'checksum_text_fields'      => array( 'type', 'status', 'payment_method_title' ),
+				'checksum_fields'           => array( 'date_updated_gmt', 'total_amount' ),
+				'checksum_text_fields'      => array( 'type', 'status' ),
 				'filter_values'             => array(
 					'type' => array(
 						'operator' => 'IN',
@@ -336,7 +337,8 @@ protected function get_default_tables() {
 				'table'                     => "{$wpdb->prefix}wc_order_operational_data",
 				'range_field'               => 'order_id',
 				'key_fields'                => array( 'order_id' ),
-				'checksum_text_fields'      => array( 'order_key', 'cart_hash' ),
+				'checksum_fields'           => array( 'date_paid_gmt', 'date_completed_gmt' ),
+				'checksum_text_fields'      => array( 'order_key' ),
 				'parent_table'              => 'wc_orders',
 				'parent_join_field'         => 'id',
 				'table_join_field'          => 'order_id',

diff --git a/...ins/automattic-for-agencies-client/changelog/update-sync-hpos-checksum-fields-to-consider b/...ins/automattic-for-agencies-client/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/automattic-for-agencies-client/composer.lock b/projects/plugins/automattic-for-agencies-client/composer.lock
diff --git a/projects/plugins/backup/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/backup/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/backup/composer.lock b/projects/plugins/backup/composer.lock
diff --git a/projects/plugins/boost/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/boost/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/boost/composer.lock b/projects/plugins/boost/composer.lock
diff --git a/projects/plugins/jetpack/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/jetpack/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: other
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/jetpack/composer.lock b/projects/plugins/jetpack/composer.lock
diff --git a/projects/plugins/migration/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/migration/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/migration/composer.lock b/projects/plugins/migration/composer.lock
diff --git a/projects/plugins/mu-wpcom-plugin/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/mu-wpcom-plugin/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/mu-wpcom-plugin/composer.lock b/projects/plugins/mu-wpcom-plugin/composer.lock
diff --git a/projects/plugins/protect/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/protect/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/protect/composer.lock b/projects/plugins/protect/composer.lock
diff --git a/projects/plugins/search/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/search/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/search/composer.lock b/projects/plugins/search/composer.lock
diff --git a/projects/plugins/social/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/social/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/social/composer.lock b/projects/plugins/social/composer.lock
diff --git a/projects/plugins/starter-plugin/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/starter-plugin/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/starter-plugin/composer.lock b/projects/plugins/starter-plugin/composer.lock
diff --git a/projects/plugins/videopress/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/videopress/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/videopress/composer.lock b/projects/plugins/videopress/composer.lock
diff --git a/projects/plugins/wpcomsh/changelog/update-sync-hpos-checksum-fields-to-consider b/projects/plugins/wpcomsh/changelog/update-sync-hpos-checksum-fields-to-consider
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/wpcomsh/composer.lock b/projects/plugins/wpcomsh/composer.lock