changed shouldUpdateRecord condition since it makes more sense

Nozbe · Mar 2, 2023 · 120d397 · 120d397
1 parent 6470faf
commit 120d397
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/sync/impl/helpers.js b/src/sync/impl/helpers.js
@@ -67,7 +67,9 @@ export function requiresUpdate<T: Model>(
   shouldUpdateRecord?: SyncShouldUpdateRecord,
 ): boolean {
   if (shouldUpdateRecord) {
-    return shouldUpdateRecord(collection.table, local, dirtyRemote)
+    if (!shouldUpdateRecord(collection.table, local, dirtyRemote)) {
+      return false;
+    }
   }
 
   if (local._status !== 'synced') {