Adapt to changed mongoose behaviour (#73)

fix: updateOne hangs if nothing was found

Author: vinerich

lib/index.js

@@ -210,7 +210,7 @@ exports.default = function (schema, opts) {
   function postUpdateOne(result, next) {
     var _this3 = this;
 
-    if (result.nModified === 0) return;
+    if (result.nModified === 0 && !result.upserted) return next();
 
     var conditions = void 0;
     if (this._originalId) conditions = { _id: { $eq: this._originalId } };else conditions = mergeQueryConditionsWithUpdate(this._conditions, this._update);
@@ -269,7 +269,7 @@ exports.default = function (schema, opts) {
   function postUpdateMany(result, next) {
     var _this5 = this;
 
-    if (result.nModified === 0) return;
+    if (result.nModified === 0 && !result.upserted) return next();
 
     var conditions = void 0;
     if (this._originalIds.length === 0) conditions = mergeQueryConditionsWithUpdate(this._conditions, this._update);else conditions = { _id: { $in: this._originalIds } };

package-lock.json

@@ -14,7 +14,7 @@
     "fast-json-patch": "^2.2.1",
     "humps": "^2.0.1",
     "lodash": "^4.17.20",
-    "mongoose": "^5.6.2"
+    "mongoose": "^5.10.7"
   },
   "devDependencies": {
     "babel-cli": "^6.26.0",

package.json

@@ -346,7 +346,7 @@ export default function (schema, opts) {
   })
 
   function postUpdateOne(result, next) {
-    if (result.nModified === 0) return
+    if (result.nModified === 0 && !result.upserted) return next()
 
     let conditions
     if (this._originalId) conditions = { _id: { $eq: this._originalId } }
@@ -388,7 +388,7 @@ export default function (schema, opts) {
   }
 
   function postUpdateMany(result, next) {
-    if (result.nModified === 0) return
+    if (result.nModified === 0 && !result.upserted) return next()
 
     let conditions
     if (this._originalIds.length === 0)