@dhyasama/totem-models 6.22.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

@@ -757,6 +757,49 @@ module.exports = function(mongoose, config) {
 
   };
 
+  Person.statics.getByIdAsGlobal = function getByIdAsGlobal(id, options, cb) {
+
+    var self = this;
+
+    // Save original id so we can put it back when we're done
+    var orginalCustomerId = config.CUSTOMER_ID;
+
+    // Switch to global
+    config.CUSTOMER_ID = "GLOBAL_PROCESS";
+    console.log('config.CUSTOMER_ID pre query', config.CUSTOMER_ID);
+
+    try {
+
+      self.findById(id, function(err, result) {
+
+        // Put original id back
+        config.CUSTOMER_ID = orginalCustomerId;
+        console.log('config.CUSTOMER_ID post query', config.CUSTOMER_ID);
+
+        if (err) { return cb(err, null); }
+
+        return cb(null, result);
+
+      });
+
+
+    }
+    catch(err) {
+
+      // Put original id back
+      config.CUSTOMER_ID = orginalCustomerId;
+      console.log('config.CUSTOMER_ID post query', config.CUSTOMER_ID);
+
+      console.log('ERROR WITH PERSON.GETBYIDGLOBAL');
+      console.log('config.CUSTOMER_ID', config.CUSTOMER_ID);
+      console.log(err);
+
+      return cb(err, null);
+
+    }
+
+  };
+
   Person.statics.getByIdRaw = function (id, cb) {
 
     // directly from mongo driver to avoid mongoose middlewares
@@ -1064,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.22.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);

package/test/Person.js

@@ -162,6 +162,21 @@ describe('Person', function() {
     person5.name.first = 'Tim';
     person5.name.last = 'Reynolds';
     person5.slug = 'tim-reynolds';
+    person5.sources = [{
+      person: new mongoose.Types.ObjectId(),
+      customer: new mongoose.Types.ObjectId()
+    }];
+
+
+    // sources: [{
+    //   _id: false,
+    //   person: { type: Schema.ObjectId, ref: 'Person' },
+    //   customer: { type: Schema.ObjectId, ref: 'Organization' },
+    //   interactions: {
+    //     calendar: { type: Number, required: false, default: 0 },
+    //     email: { type: Number, required: false, default: 0 }
+    //   }
+    // }],
 
     Person.upsert(person2, 'test-user', function(err, person2) {
       Person.upsert(person3, 'test-user', function(err, person3) {
@@ -845,6 +860,17 @@ describe('Person', function() {
 
   });
 
+  it('finds a person as global', function (done) {
+
+    Person.getByIdAsGlobal(person5._id, {}, function (err, result) {
+      should.not.exist(err);
+      should.exist(result);
+      result.sources.length.should.equal(1);
+      done();
+    });
+
+  });
+
   it('finds the primary email for a person', function(done) {
 
     Person.getById(person.id, function(err, person) {