Because Mongo is terrible, don't use the new defaulting behavior when…

… the database isn't SQL.
vapor · Feb 17, 2024 · 1c1508f · 1c1508f
1 parent 73c1966
commit 1c1508f
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/Sources/FluentKit/Model/Model+CRUD.swift b/Sources/FluentKit/Model/Model+CRUD.swift
@@ -1,4 +1,5 @@
 import NIOCore
+import protocol SQLKit.SQLDatabase
 
 extension Model {
     public func save(on database: Database) -> EventLoopFuture<Void> {
@@ -22,7 +23,7 @@ extension Model {
             self.anyID.generate()
             let promise = database.eventLoop.makePromise(of: DatabaseOutput.self)
             Self.query(on: database)
-                .set(self.collectInput(withDefaultedValues: true))
+                .set(self.collectInput(withDefaultedValues: database is SQLDatabase))
                 .action(.create)
                 .run { promise.succeed($0) }
                 .cascadeFailure(to: promise)
@@ -36,7 +37,7 @@ extension Model {
             }
         } else {
             return Self.query(on: database)
-                .set(self.collectInput(withDefaultedValues: true))
+                .set(self.collectInput(withDefaultedValues: database is SQLDatabase))
                 .action(.create)
                 .run()
                 .flatMapThrowing {
@@ -179,7 +180,7 @@ extension Collection where Element: FluentKit.Model {
             }.create(model, on: database)
         }, on: database.eventLoop).flatMap {
             Element.query(on: database)
-                .set(self.map { $0.collectInput(withDefaultedValues: true) })
+                .set(self.map { $0.collectInput(withDefaultedValues: database is SQLDatabase) })
                 .create()
         }.map {
             for model in self {