fix(document): support skipping timestamps on save() with `save({ tim…

…estamps: false })` Fix #7357
Automattic · Jan 12, 2019 · 17819f1 · 17819f1
1 parent 3d48a22
commit 17819f1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/lib/model.js b/lib/model.js
@@ -465,8 +465,11 @@ Model.prototype.save = function(options, fn) {
       return cb(parallelSave);
     }
 
+    this.$__.saveOptions = options;
+
     this.$__save(options, error => {
       this.$__.saving = undefined;
+      delete this.$__.saveOptions;
 
       if (error) {
         this.$__handleReject(error);

diff --git a/lib/schema.js b/lib/schema.js
@@ -918,6 +918,10 @@ Schema.prototype.setupTimestamp = function(timestamps) {
   this.add(schemaAdditions);
 
   this.pre('save', function(next) {
+    if (get(this, '$__.saveOptions.timestamps') === false) {
+      return next();
+    }
+
     const defaultTimestamp = (this.ownerDocument ? this.ownerDocument() : this).
       constructor.base.now();
     const auto_id = this._id && this._id.auto;