fix(update): handle subdocument pre('validate') errors in update vali…

…dation Fix #7187
Automattic · Oct 11, 2019 · 936ddfb · 936ddfb
1 parent 98b3b09
commit 936ddfb
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/lib/document.js b/lib/document.js
@@ -1932,9 +1932,11 @@ Document.prototype.validate = function(options, callback) {
     options = null;
   }
 
-  return utils.promiseOrCallback(callback, cb => this.$__validate(options, function(error) {
-    cb(error);
-  }), this.constructor.events);
+  return utils.promiseOrCallback(callback, cb => {
+    this.$__validate(options, function(error) {
+      cb(error);
+    });
+  }, this.constructor.events);
 };
 
 /*!

diff --git a/lib/helpers/updateValidators.js b/lib/helpers/updateValidators.js
@@ -136,6 +136,9 @@ module.exports = function(query, schema, castedDoc, options, callback) {
                     _err.path = updates[i] + '.' + key;
                     validationErrors.push(_err);
                   }
+                } else {
+                  err.path = updates[i];
+                  validationErrors.push(err);
                 }
               }
               callback(null);