fix(timestamps): avoid setting createdAt on documents that already exist but dont have createdAt

vkarpov15 · vkarpov15 · commit 4cfc4d615304 · 2021-12-26T21:10:23.000-05:00
Fix #11024
diff --git a/lib/helpers/timestamps/setupTimestamps.js b/lib/helpers/timestamps/setupTimestamps.js
@@ -51,7 +51,7 @@ module.exports = function setupTimestamps(schema, timestamps) {
       (this.ownerDocument ? this.ownerDocument() : this).constructor.base.now();
     const auto_id = this._id && this._id.auto;
 
-    if (!skipCreatedAt && createdAt && !this.get(createdAt) && this.$__isSelected(createdAt)) {
+    if (!skipCreatedAt && this.isNew && createdAt && !this.get(createdAt) && this.$__isSelected(createdAt)) {
       this.$set(createdAt, auto_id ? this._id.getTimestamp() : defaultTimestamp);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ module.exports = function setupTimestamps(schema, timestamps) {`
`51`	`51`	`(this.ownerDocument ? this.ownerDocument() : this).constructor.base.now();`
`52`	`52`	`const auto_id = this._id && this._id.auto;`
`53`	`53`
`54`		`- if (!skipCreatedAt && createdAt && !this.get(createdAt) && this.$__isSelected(createdAt)) {`
	`54`	`+ if (!skipCreatedAt && this.isNew && createdAt && !this.get(createdAt) && this.$__isSelected(createdAt)) {`
`55`	`55`	`this.$set(createdAt, auto_id ? this._id.getTimestamp() : defaultTimestamp);`
`56`	`56`	`}`
`57`	`57`