Rename oldData to previousValues

Author: simolus3

packages/powersync_core/lib/src/crud.dart

@@ -91,7 +91,7 @@ class CrudEntry {
   ///
   /// This is only tracked for tables for which this has been enabled by setting
   /// the [Table.trackPreviousValues].
-  final Map<String, dynamic>? oldData;
+  final Map<String, dynamic>? previousValues;
 
   CrudEntry(
     this.clientId,
@@ -100,7 +100,7 @@ class CrudEntry {
     this.id,
     this.transactionId,
     this.opData, {
-    this.oldData,
+    this.previousValues,
     this.metadata,
   });
 
@@ -113,7 +113,7 @@ class CrudEntry {
       data['id'] as String,
       row['tx_id'] as int,
       data['data'] as Map<String, Object?>?,
-      oldData: data['old'] as Map<String, Object?>?,
+      previousValues: data['old'] as Map<String, Object?>?,
       metadata: data['metadata'] as String?,
     );
   }
@@ -128,7 +128,7 @@ class CrudEntry {
       'tx_id': transactionId,
       'data': opData,
       'metadata': metadata,
-      'old': oldData,
+      'old': previousValues,
     };
   }
 

packages/powersync_core/lib/src/schema.dart

@@ -35,7 +35,8 @@ final class TrackPreviousValuesOptions {
   /// A filter of column names for which updates should be tracked.
   ///
   /// When set to a non-null value, columns not included in this list will not
-  /// appear in [CrudEntry.oldData]. By default, all columns are included.
+  /// appear in [CrudEntry.previousValues]. By default, all columns are
+  /// included.
   final List<String>? columnFilter;
 
   /// Whether to only include old values when they were changed by an update,
@@ -64,7 +65,7 @@ class Table {
   /// through [CrudEntry.metadata].
   final bool trackMetadata;
 
-  /// Whether to track old values of columns for [CrudEntry.oldData].
+  /// Whether to track old values of columns for [CrudEntry.previousValues].
   ///
   /// See [TrackPreviousValuesOptions] for details.
   final TrackPreviousValuesOptions? trackPreviousValues;

packages/powersync_core/test/crud_test.dart

@@ -304,7 +304,8 @@ void main() {
       await powersync.execute('UPDATE lists SET name = ?;', ['new name']);
 
       final batch = await powersync.getNextCrudTransaction();
-      expect(batch!.crud[0].oldData, {'name': 'entry', 'content': 'content'});
+      expect(batch!.crud[0].previousValues,
+          {'name': 'entry', 'content': 'content'});
     });
 
     test('include old values with column filter', () async {
@@ -325,7 +326,7 @@ void main() {
           ['new name', 'new content']);
 
       final batch = await powersync.getNextCrudTransaction();
-      expect(batch!.crud[0].oldData, {'name': 'name'});
+      expect(batch!.crud[0].previousValues, {'name': 'name'});
     });
 
     test('include old values when changed', () async {
@@ -345,7 +346,7 @@ void main() {
       await powersync.execute('UPDATE lists SET name = ?', ['new name']);
 
       final batch = await powersync.getNextCrudTransaction();
-      expect(batch!.crud[0].oldData, {'name': 'name'});
+      expect(batch!.crud[0].previousValues, {'name': 'name'});
     });
 
     test('ignore empty update', () async {