fix: Prevent update from clearing out implicit fields. (serverpod#3360)

Author: SandPod

packages/serverpod/lib/src/database/adapters/postgres/database_connection.dart

@@ -183,7 +183,7 @@ class DatabaseConnection {
 
     var table = rows.first.table;
 
-    var selectedColumns = (columns ?? table.columns).toSet();
+    var selectedColumns = (columns ?? table.managedColumns).toSet();
 
     if (columns != null) {
       _validateColumnsExists(selectedColumns, table.columns.toSet());

packages/serverpod/lib/src/database/concepts/table.dart

@@ -25,6 +25,10 @@ class Table<T_ID> {
   /// List of [Column] used by the table.
   List<Column> get columns => [id];
 
+  /// List of [Column] that are managed by this table.
+  /// This is mostly used to exclude columns created by implicit relations.
+  List<Column> get managedColumns => columns;
+
   /// Query prefix for [Column]s of the table.
   String get queryPrefix {
     return tableRelation?.relationQueryAlias ?? tableName;

tests/serverpod_test_client/lib/src/protocol/models_with_relations/one_to_many/implicit/book.dart

@@ -0,0 +1,102 @@
+/* AUTOMATICALLY GENERATED CODE DO NOT MODIFY */
+/*   To generate run: "serverpod generate"    */
+
+// ignore_for_file: implementation_imports
+// ignore_for_file: library_private_types_in_public_api
+// ignore_for_file: non_constant_identifier_names
+// ignore_for_file: public_member_api_docs
+// ignore_for_file: type_literal_in_constant_pattern
+// ignore_for_file: use_super_parameters
+
+// ignore_for_file: no_leading_underscores_for_library_prefixes
+import 'package:serverpod_client/serverpod_client.dart' as _i1;
+import '../../../models_with_relations/one_to_many/implicit/chapter.dart'
+    as _i2;
+
+abstract class Book implements _i1.SerializableModel {
+  Book._({
+    this.id,
+    required this.title,
+    this.chapters,
+  });
+
+  factory Book({
+    int? id,
+    required String title,
+    List<_i2.Chapter>? chapters,
+  }) = _BookImpl;
+
+  factory Book.fromJson(Map<String, dynamic> jsonSerialization) {
+    return Book(
+      id: jsonSerialization['id'] as int?,
+      title: jsonSerialization['title'] as String,
+      chapters: (jsonSerialization['chapters'] as List?)
+          ?.map((e) => _i2.Chapter.fromJson((e as Map<String, dynamic>)))
+          .toList(),
+    );
+  }
+
+  /// The database id, set if the object has been inserted into the
+  /// database or if it has been fetched from the database. Otherwise,
+  /// the id will be null.
+  int? id;
+
+  String title;
+
+  List<_i2.Chapter>? chapters;
+
+  /// Returns a shallow copy of this [Book]
+  /// with some or all fields replaced by the given arguments.
+  @_i1.useResult
+  Book copyWith({
+    int? id,
+    String? title,
+    List<_i2.Chapter>? chapters,
+  });
+  @override
+  Map<String, dynamic> toJson() {
+    return {
+      if (id != null) 'id': id,
+      'title': title,
+      if (chapters != null)
+        'chapters': chapters?.toJson(valueToJson: (v) => v.toJson()),
+    };
+  }
+
+  @override
+  String toString() {
+    return _i1.SerializationManager.encode(this);
+  }
+}
+
+class _Undefined {}
+
+class _BookImpl extends Book {
+  _BookImpl({
+    int? id,
+    required String title,
+    List<_i2.Chapter>? chapters,
+  }) : super._(
+          id: id,
+          title: title,
+          chapters: chapters,
+        );
+
+  /// Returns a shallow copy of this [Book]
+  /// with some or all fields replaced by the given arguments.
+  @_i1.useResult
+  @override
+  Book copyWith({
+    Object? id = _Undefined,
+    String? title,
+    Object? chapters = _Undefined,
+  }) {
+    return Book(
+      id: id is int? ? id : this.id,
+      title: title ?? this.title,
+      chapters: chapters is List<_i2.Chapter>?
+          ? chapters
+          : this.chapters?.map((e0) => e0.copyWith()).toList(),
+    );
+  }
+}

tests/serverpod_test_client/lib/src/protocol/models_with_relations/one_to_many/implicit/chapter.dart

@@ -0,0 +1,84 @@
+/* AUTOMATICALLY GENERATED CODE DO NOT MODIFY */
+/*   To generate run: "serverpod generate"    */
+
+// ignore_for_file: implementation_imports
+// ignore_for_file: library_private_types_in_public_api
+// ignore_for_file: non_constant_identifier_names
+// ignore_for_file: public_member_api_docs
+// ignore_for_file: type_literal_in_constant_pattern
+// ignore_for_file: use_super_parameters
+
+// ignore_for_file: no_leading_underscores_for_library_prefixes
+import 'package:serverpod_client/serverpod_client.dart' as _i1;
+
+abstract class Chapter implements _i1.SerializableModel {
+  Chapter._({
+    this.id,
+    required this.title,
+  });
+
+  factory Chapter({
+    int? id,
+    required String title,
+  }) = _ChapterImpl;
+
+  factory Chapter.fromJson(Map<String, dynamic> jsonSerialization) {
+    return Chapter(
+      id: jsonSerialization['id'] as int?,
+      title: jsonSerialization['title'] as String,
+    );
+  }
+
+  /// The database id, set if the object has been inserted into the
+  /// database or if it has been fetched from the database. Otherwise,
+  /// the id will be null.
+  int? id;
+
+  String title;
+
+  /// Returns a shallow copy of this [Chapter]
+  /// with some or all fields replaced by the given arguments.
+  @_i1.useResult
+  Chapter copyWith({
+    int? id,
+    String? title,
+  });
+  @override
+  Map<String, dynamic> toJson() {
+    return {
+      if (id != null) 'id': id,
+      'title': title,
+    };
+  }
+
+  @override
+  String toString() {
+    return _i1.SerializationManager.encode(this);
+  }
+}
+
+class _Undefined {}
+
+class _ChapterImpl extends Chapter {
+  _ChapterImpl({
+    int? id,
+    required String title,
+  }) : super._(
+          id: id,
+          title: title,
+        );
+
+  /// Returns a shallow copy of this [Chapter]
+  /// with some or all fields replaced by the given arguments.
+  @_i1.useResult
+  @override
+  Chapter copyWith({
+    Object? id = _Undefined,
+    String? title,
+  }) {
+    return Chapter(
+      id: id is int? ? id : this.id,
+      title: title ?? this.title,
+    );
+  }
+}