fix: id won't override _id

Author: IslandRhythms

lib/document.js

@@ -1020,6 +1020,9 @@ Document.prototype.overwrite = function overwrite(obj) {
  */
 
 Document.prototype.$set = function $set(path, val, type, options) {
+  if (path._id && path.id) {
+    delete path.id;
+  }
   if (utils.isPOJO(type)) {
     options = type;
     type = undefined;

test/document.test.js

@@ -12413,14 +12413,29 @@ describe('document', function() {
       _id: {
         type: Number
       }
-    }, { _id: false, id: false });
+    });
     const Test = db.model('Test', testSchema);
 
     const x = new Test({ _id: 1, id: 2});
     await x.save();
     const fromDb = await Test.findById(x._id).lean();
     assert.equal(fromDb._id, 1);
   });
+  it('should ignore `id` if settings with an object that contains `_id` and `id` (gh-13672)', async function() {
+    const testSchema = new Schema({
+      _id: {
+        type: Number
+      }
+    });
+    const Test = db.model('Test', testSchema);
+
+    const x = new Test();
+    x.set('_id', { _id: 1, id: 2 });
+    await x.save();
+
+    const fromDb = await Test.findById(x._id).lean();
+    assert.equal(fromDb._id, 1);
+  })
 });
 
 describe('Check if instance function that is supplied in schema option is availabe', function() {