@dhyasama/totem-models 8.23.0 → 8.24.0

package/lib/Deal.js

@@ -188,10 +188,6 @@ module.exports = function(mongoose, config) {
   // Get all deals belonging to a customer
   Deal.statics.getForCustomer = function (customerId, options, cb) {
 
-    // todo - fix deep population
-    // not sure why chained deepPopulate isn't working (it throws an error)
-    // it works to deepPopulate each result instance in parallel
-
     let self = this;
 
     if (!cb) { throw new Error('cb is required'); }
@@ -213,13 +209,8 @@ module.exports = function(mongoose, config) {
       else if(options.updated.gte) terms['history'] = { $elemMatch: { timestamp: { $gte: options.updated.gte } } };
     };
 
-    self
-      .find(terms)
-      .populate({ path: 'documents', match: { customer: customerId } })
-      .populate({ path: 'messages', match: { customer: customerId } })
-      .populate('organization', 'name description logoUrl website websiteAliases chairs contact')
-      .populate('referrer.person', 'name avatarUrl title')
-      .exec(cb);
+    let query = self.find(terms);
+    query.exec(cb);
 
   };
 

package/lib/Message.js

@@ -75,7 +75,7 @@ module.exports = function(mongoose, config) {
 
   };
 
-  Message.statics.deleteDocument = function(docId, cb) {
+  Message.statics.deleteDocument = function(docId, customerId, cb) {
 
     // Delete the doc itself along with any references
 
@@ -88,7 +88,7 @@ module.exports = function(mongoose, config) {
     };
 
     async.series([
-      Document.delete.bind(Document, docId),
+      Document.delete.bind(Document, { CUSTOMER_ID: customerId }, docId),
       removeReferences
     ], function(err, results) {
       return cb(err, null);

package/package.json

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

package/test/Deal.js

@@ -237,21 +237,6 @@ describe('Deal', function() {
 
   });
 
-  it('gets new deals for a customer', function(done) {
-
-    Deal.getForCustomer(customer._id, { since: timestamps[0] }, function(err, result) {
-
-      should.not.exist(err);
-      should.exist(result);
-
-      result.length.should.be.greaterThan(0);
-
-      return done();
-
-    });
-
-  });
-
   it('deletes a deal', function(done) {
     Deal.delete(dealId, { CUSTOMER_ID: customer._id }, function(err, result) {
       should.not.exist(err);