Merge pull request #101 from nodes-vapor/feature/fix-saveOrUpdate-beh…

…avior

Use create instead of save

.circleci/config.yml

@@ -11,6 +11,7 @@ jobs:
       - run:
           name: Install dependencies
           command: |
+            export HOMEBREW_NO_AUTO_UPDATE=1
             brew tap vapor/homebrew-tap
             brew install cmysql
             brew install ctls

.gitignore

@@ -1,13 +1,7 @@
-*.swp
 Packages
 .build
-.idea
-xcuserdata
-*.xcodeproj
-Config/secrets/
 .DS_Store
-node_modules/
-bower_components/
-.swift-version
-Package.pins
+*.xcodeproj
+DerivedData/
 Package.resolved
+.swiftpm

Sources/Sugar/Helpers/Model.swift

@@ -11,7 +11,22 @@ public extension Model {
             .unwrap(or: Abort(.notFound, reason: "\(Self.self) with id \(id) not found"))
     }
 
-    func saveOrUpdate (
+    @available(*, deprecated, message: "use `creatOrUpdate(given:withSoftDeleted:restore:on)`")
+    func saveOrUpdate(
+        given filters: [FilterOperator<Self.Database, Self>],
+        withSoftDeleted: Bool = false,
+        restore: Bool = false,
+        on db: DatabaseConnectable
+    ) throws -> Future<Self> {
+        return try createOrUpdate(
+            given: filters,
+            withSoftDeleted: withSoftDeleted,
+            restore: restore,
+            on: db
+        )
+    }
+
+    func createOrUpdate(
         given filters: [FilterOperator<Self.Database, Self>],
         withSoftDeleted: Bool = false,
         restore: Bool = false,
@@ -25,7 +40,7 @@ public extension Model {
 
         return query.first().flatMap(to: Self.self) { result in
             guard let result = result else {
-                return self.save(on: db)
+                return self.create(on: db)
             }
 
             var copy = self