@dhyasama/totem-models 6.23.0 → 6.24.0

package/lib/Deal.js

@@ -287,6 +287,25 @@ module.exports = function(mongoose, config) {
       });
   };
 
+  Deal.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   Deal.statics.upsert = function upsert(deal, user, cb) {
 
     // Deal is dumb and doesn't validate customer stages or duplicate deals. That responsibility is with the org.

package/lib/Document.js

@@ -133,6 +133,25 @@ module.exports = function(mongoose, config) {
     }).exec(cb);
   };
 
+  Document.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   Document.statics.upsert = function upsert(doc, cb) {
 
     if (!doc) { return cb(new Error('doc is required'), null); }

package/lib/Flag.js

@@ -66,6 +66,25 @@ module.exports = function(mongoose, config) {
     }).exec(cb);
   };
 
+  Flag.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   Flag.statics.upsert = function(flag, cb) {
     flag.save(cb);
   };

package/lib/Interaction.js

@@ -194,6 +194,25 @@ module.exports = function(mongoose, config) {
 
   };
 
+  Interaction.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   Interaction.statics.upsert = function upsert(interaction, cb) {
 
     if (!interaction) { return cb(new Error('Interaction is required'), null); }

package/lib/LimitedPartner.js

@@ -726,6 +726,25 @@ module.exports = function(mongoose, config) {
 
   };
 
+  LimitedPartner.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   LimitedPartner.statics.upsert = function(lp, username, role, cb) {
 
     if (!lp) { return cb(new Error('lp is required'), null); }

package/lib/List.js

@@ -209,6 +209,25 @@ module.exports = function(mongoose, config) {
 
   };
 
+  List.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   List.statics.remove = function(id, cb) {
     this.findOneAndRemove({
       '_id': id,

package/lib/Message.js

@@ -296,6 +296,25 @@ module.exports = function(mongoose, config) {
 
   };
 
+  Message.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   Message.statics.upsert = function(message, cb) {
 
     var timestamp = new Date();

package/lib/News.js

@@ -88,6 +88,25 @@ module.exports = function(mongoose, config) {
 
   };
 
+  News.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   News.statics.upsert = function (news, cb) {
     news.markModified('extracted');
     news.save(cb);

package/lib/Note.js

@@ -74,6 +74,25 @@ module.exports = function(mongoose, config) {
 
   };
 
+  Note.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   Note.statics.upsert = function(note, cb) {
     note.save(cb);
   };

package/lib/Person.js

@@ -1107,6 +1107,25 @@ module.exports = function(mongoose, config) {
     customerCompanyIds = ids;
   };
 
+  Person.statics.modifyById = function(id, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
+
+    var self = this;
+
+    // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
+    // options runValidators defaults false which is ok since we have upsert false
+    // new returns the updated document
+
+    self.findByIdAndUpdate(id, update, { upsert: false, new: true }, cb);
+
+  };
+
   Person.statics.upsert = function(currentPerson, username, cb) {
 
     if (!currentPerson) { return cb(new Error('Current Person is required'), null); }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dhyasama/totem-models",
-  "version": "6.23.0",
+  "version": "6.24.0",
   "author": "Jason Reynolds",
   "license": "UNLICENSED",
   "description": "Models for Totem platform",

package/test/Deal.js

@@ -92,7 +92,13 @@ describe('Deal', function() {
     var deal = new Deal({
       customer: customer,
       organization: org,
-      stage: 'New'
+      stage: 'New',
+      source: {
+        person: new mongoose.Types.ObjectId()
+      },
+      referrer: {
+        person: new mongoose.Types.ObjectId()
+      }
     });
 
     Deal.upsert(deal, account, function(err, result) {
@@ -159,6 +165,33 @@ describe('Deal', function() {
 
   });
 
+  it('modifies a deal by id', function (done) {
+
+    Deal.getById(dealId, function(err, deal) {
+
+      should.not.exist(err);
+      should.exist(deal);
+      deal._id.toString().should.equal(dealId.toString());
+
+      var newSource = new mongoose.Types.ObjectId();
+
+      var update = {
+        $set: {
+          'source.person': newSource
+        }
+      };
+
+      Deal.modifyById(dealId, update, function (err, result) {
+        should.not.exist(err);
+        should.exist(result);
+        result.source.person.toString().should.equal(newSource.toString());
+        done();
+      });
+
+    });
+
+  });
+
   it('gets all deals for a customer', function(done) {
 
     var d = new Deal({

package/test/Document.js

@@ -325,6 +325,33 @@ describe('Document', function() {
     });
   });
 
+  it('modifies a doc by id', function (done) {
+
+    Document.getById(documentId, function(err, doc) {
+
+      should.not.exist(err);
+      should.exist(doc);
+      doc._id.toString().should.equal(documentId.toString());
+
+      var newCreatedBy = new mongoose.Types.ObjectId();
+
+      var update = {
+        $set: {
+          'createdBy': newCreatedBy
+        }
+      };
+
+      Document.modifyById(documentId, update, function (err, result) {
+        should.not.exist(err);
+        should.exist(result);
+        result.createdBy.toString().should.equal(newCreatedBy.toString());
+        done();
+      });
+
+    });
+
+  });
+
   it('deletes a document', function(done) {
     Document.delete(documentId, function(err, result) {
       should.not.exist(err);

package/test/Flag.js

@@ -236,6 +236,33 @@ describe('Flag', function() {
     });
   });
 
+  it('modifies a flag by id', function (done) {
+
+    Flag.getById(flagId, function(err, flag) {
+
+      should.not.exist(err);
+      should.exist(flag);
+      flag._id.toString().should.equal(flagId.toString());
+
+      var newCreatedBy = new mongoose.Types.ObjectId();
+
+      var update = {
+        $set: {
+          'createdBy': newCreatedBy
+        }
+      };
+
+      Flag.modifyById(flagId, update, function (err, result) {
+        should.not.exist(err);
+        should.exist(result);
+        result.createdBy.toString().should.equal(newCreatedBy.toString());
+        done();
+      });
+
+    });
+
+  });
+
   it('deletes a flag', function(done) {
     Flag.delete(flagId, function(err, result) {
       should.not.exist(err);

package/test/List.js

@@ -262,6 +262,41 @@ describe('List', function() {
 
   });
 
+  it('modifies a list by id', function (done) {
+
+    List.getById(myAwesomeList.id, function(err, list) {
+
+      should.not.exist(err);
+      should.exist(list);
+      list._id.toString().should.equal(myAwesomeList.id.toString());
+
+      var newCreatedBy = new mongoose.Types.ObjectId();
+      var newPerson = new mongoose.Types.ObjectId();
+      var newOrg = new mongoose.Types.ObjectId();
+
+      var update = {
+        $set: {
+          'createdBy': newCreatedBy,
+          'people': [newPerson],
+          'organizations': [newOrg]
+        }
+      };
+
+      List.modifyById(myAwesomeList.id, update, function (err, result) {
+        should.not.exist(err);
+        should.exist(result);
+        result.createdBy.toString().should.equal(newCreatedBy.toString());
+        result.people.length.should.equal(1);
+        result.people[0].toString().should.equal(newPerson.toString());
+        result.organizations.length.should.equal(1);
+        result.organizations[0].toString().should.equal(newOrg.toString());
+        done();
+      });
+
+    });
+
+  });
+
   it('deletes a list', function(done) {
 
     List.remove(myAwesomeList.id, function(err, result) {

package/test/Message.js

@@ -328,4 +328,23 @@ describe('Message', function() {
 
   });
 
+  it('modifies a message by id', function (done) {
+
+    var newOrg = new mongoose.Types.ObjectId();
+
+    var update = {
+      $set: {
+        'organization': newOrg
+      }
+    };
+
+    Message.modifyById(messageOne.id, update, function (err, result) {
+      should.not.exist(err);
+      should.exist(result);
+      result.organization.toString().should.equal(newOrg.toString());
+      done();
+    });
+
+  });
+
 });

package/test/Note.js

@@ -236,6 +236,33 @@ describe('Note', function() {
     });
   });
 
+  it('modifies a note by id', function (done) {
+
+    Note.getById(noteId, function(err, note) {
+
+      should.not.exist(err);
+      should.exist(note);
+      note._id.toString().should.equal(noteId.toString());
+
+      var newCreatedBy = new mongoose.Types.ObjectId();
+
+      var update = {
+        $set: {
+          'createdBy': newCreatedBy
+        }
+      };
+
+      Note.modifyById(noteId, update, function (err, result) {
+        should.not.exist(err);
+        should.exist(result);
+        result.createdBy.toString().should.equal(newCreatedBy.toString());
+        done();
+      });
+
+    });
+
+  });
+
   it('deletes a note', function(done) {
     Note.delete(noteId, function(err, result) {
       should.not.exist(err);