Too many patches when saving refs? about mongoose-patch-history HOT 12 CLOSED

  organization: {
    type: mongoose.Schema.Types.ObjectId,
    ref: 'Organization',
  },

  organization: {
    type: String,
    ref: 'Organization',
  },

const mongoose = require('mongoose');
const patchHistory = require('mongoose-patch-history');
const Promise = require('bluebird');
mongoose.Promise = Promise;

// Organization Model
const OrganizationSchema = new mongoose.Schema({
  name: {
    type: String,
  },
});
const Organization = mongoose.model('Organization', OrganizationSchema);

// Person Model
const PersonSchema = new mongoose.Schema({
  organization: {
    type: mongoose.Schema.Types.ObjectId,
    ref: 'Organization',
  },
  name: {
    type: String,
  },
});

PersonSchema.plugin(patchHistory.default, {
  mongoose,
  name: 'roomPatches',
});

const Person = mongoose.model('Person', PersonSchema);

//////////////
// The Test //
//////////////
const uri = 'mongodb://localhost:27017/mongoose_patch_history_test';

let workOrg;
let homeOrg;
let person;
mongoose.connect(uri)
  // Setup
  .then(() => Organization.create({ name: 'Home '}))
  .then((o) => (homeOrg = o))
  .then(() => Organization.create({ name: 'Work '}))
  .then((o) => (workOrg = o))
  .then(() => Person.create({ name: 'Bob', organization: homeOrg._id }))
  .then((p) => (person = p))
  // Test
  .then(() => person.patches.find({ ref: person.id }))
  .then((patches) => {
    console.log(JSON.stringify(patches,null,2));
    // [
    //   {
    //     "_id": "589d7124c2d0b1aa30dcfb14",
    //     "ref": "589d7124c2d0b1aa30dcfb13",
    //     "__v": 0,
    //     "ops": [
    //       {
    //         "op": "add",
    //         "path": "/name",
    //         "value": "Bob"
    //       },
    //       {
    //         "op": "add",
    //         "path": "/organization",
    //         "value": "589d7124c2d0b1aa30dcfb11"
    //       }
    //     ],
    //     "date": "2017-02-10T07:52:04.201Z"
    //   }
    // ]
  })
  // Switch the org
  .then(() => {
    person.organization = workOrg._id;
    return person.save();
  })
  .then(() => person.patches.find({ ref: person.id }))
  .then((patches) => {
    console.log(JSON.stringify(patches,null,2));
    // [
    //   {
    //     "_id": "589d7195101d63abfae70aee",
    //     "ref": "589d7195101d63abfae70aed",
    //     "__v": 0,
    //     "ops": [
    //       {
    //         "op": "add",
    //         "path": "/name",
    //         "value": "Bob"
    //       },
    //       {
    //         "op": "add",
    //         "path": "/organization",
    //         "value": "589d7195101d63abfae70aeb"
    //       }
    //     ],
    //     "date": "2017-02-10T07:53:57.053Z"
    //   },
    //   {
    //     "_id": "589d7195101d63abfae70aef",
    //     "ref": "589d7195101d63abfae70aed",
    //     "__v": 0,
    //     "ops": [
    //       {
    //         "op": "replace",
    //         "path": "/organization/id/11",
    //         "value": 236
    //       }
    //     ],
    //     "date": "2017-02-10T07:53:57.070Z"
    //   }
    // ]
  });

"ops": [
  {
    "op": "replace",
    "path": "/organization/id/11",
    "value": 236
  }

"ops": [
  {
    "op": "replace",
    "path": "/organization", // <--- This path
    "value": THE_WORK_ORG_ID // <--- A Mongo Object ID string
  }

  "dependencies": {
    "bluebird": "^3.4.7",
    "mongoose": "^4.8.1",
    "mongoose-patch-history": "^1.1.5"
  }

console.log(this._original, this.data());
// { name: 'Bob', organization: 589e966e0167f20340a26d44 } { name: 'Bob', organization: 589e966e0167f20340a26d45 }
const ops = jsonpatch.compare(this.isNew ? {} : this._original, this.data())
console.log(ops);
//[ { op: 'replace', path: '/organization/id/11', value: 69 } ]

console.log(typeof this._original.organization, typeof this.data().organization);
// object object
console.log(this._original.organization.constructor.name);
// ObjectID