@dhyasama/totem-models 7.40.1 → 7.42.0

package/lib/LimitedPartner.js

@@ -345,7 +345,7 @@ module.exports = function(mongoose, config) {
       return personId.toString() == idOfPersonToAdd.toString();
     });
 
-    if (!match) this.people.push(idOfPersonToAdd);
+    if (!match) this.people = this.people.concat([idOfPersonToAdd]);
 
   };
 
@@ -381,7 +381,7 @@ module.exports = function(mongoose, config) {
       .exec(cb);
   };
 
-  LimitedPartner.statics.getById = function(id, role, cb) {
+  LimitedPartner.statics.getById = function(customerId, lpId, role, cb) {
 
     var self = this;
     var query;
@@ -408,11 +408,11 @@ module.exports = function(mongoose, config) {
     };
 
     if (CUSTOMER_ID == 'GLOBAL_PROCESS') {
-      query = self.findById(id, self.getReadFilterKeys(role))
+      query = self.findById(lpId, self.getReadFilterKeys(role))
       getLp();
     }
     else {
-      query = self.findOne({ '_id': id, 'transactions.fund': { $in : self.customerFunds }}, self.getReadFilterKeys(role));
+      query = self.findOne({ '_id': lpId, 'customer': customerId, 'transactions.fund': { $in : self.customerFunds }}, self.getReadFilterKeys(role));
       getLp();
     }
 
@@ -444,7 +444,7 @@ module.exports = function(mongoose, config) {
 
   };
 
-  LimitedPartner.statics.getByName = function(lpName, role, cb) {
+  LimitedPartner.statics.getByName = function(customerId, lpName, role, cb) {
 
     var self = this;
     var query;
@@ -459,7 +459,7 @@ module.exports = function(mongoose, config) {
       getLps();
     }
     else {
-      query = self.find({'name': lpName, 'transactions.fund': { $in : self.customerFunds }}, self.getReadFilterKeys(role));
+      query = self.find({'customer': customerId, 'name': lpName, 'transactions.fund': { $in : self.customerFunds }}, self.getReadFilterKeys(role));
       getLps();
     }
 

package/lib/List.js

@@ -95,7 +95,7 @@ module.exports = function(mongoose, config) {
       if (err) return cb(err, null);
       if (!result) return cb(null, null);
 
-      result[collection].push(itemId);
+      result[collection] = result.collection.concat([itemId]);
 
       result.save(cb);
 

package/lib/Organization.js

@@ -570,7 +570,7 @@ module.exports = function(mongoose, config) {
       // Add website to website aliases
       // There are unique constraints on website and website aliases
       // This enables us to enforce uniqueness across both and across documents
-      if (obj.website && obj.websiteAliases) obj.websiteAliases.push(obj.website);
+      if (obj.website && obj.websiteAliases) obj.websiteAliases = obj.websiteAliases.concat([obj.website]);
 
       if (obj.websiteAliases) {
         obj.websiteAliases = _.uniq(obj.websiteAliases);
@@ -1068,7 +1068,7 @@ module.exports = function(mongoose, config) {
     if (dealMatch) { dealMatch = deal; }
 
     // Add the deal
-    else { self.deals.push(deal); }
+    else self.deals = self.deals.concat([deal]);
 
   };
 
@@ -1089,7 +1089,7 @@ module.exports = function(mongoose, config) {
     });
 
     // If not, add it
-    if (!match) self.people.push(personToAdd);
+    if (!match) self.people = self.people.concat([personToAdd]);
 
   };
 
@@ -1117,7 +1117,7 @@ module.exports = function(mongoose, config) {
       return orgId.toString() == id.toString();
     });
 
-    if (!match) { self.related.push(id); }
+    if (!match) self.related = self.related.concat([id]);
 
   };
 
@@ -1202,7 +1202,7 @@ module.exports = function(mongoose, config) {
 
     var addDealToOrg = function addDealToOrg(previous, callback) {
 
-      previous.org.deals.push(previous.deal);
+      previous.org.deals = previous.org.deals.concat([previous.deal]);
 
       self.upsert(previous.org, previous.user.username, function(err, result) {
 

package/lib/Person.js

@@ -233,7 +233,7 @@ module.exports = function(mongoose, config) {
   };
 
   Person.methods.relatePerson = function(id) {
-    this.related.push(id);
+    this.related = this.related.concat([id]);
   };
 
   Person.methods.addChairPos = function(chairObj, orgId) {
@@ -245,11 +245,11 @@ module.exports = function(mongoose, config) {
 
     switch(chairObj.num) {
       case 'chair1':
-        this.chairs.first.push(chair);
+        this.chairs.first = this.chairs.first.concat([chair]);
         break;
 
       case 'chair2':
-        this.chairs.second.push(chair);
+        this.chairs.second = this.chairs.second.concat([chair]);
         break;
     }
 
@@ -276,12 +276,12 @@ module.exports = function(mongoose, config) {
     }
 
     if(chairObj.val && chairObj.val == "") {
-      this.previous.push({
+      this.previous = this.previous.concat([{
         field: field,
         value: {
           company: orgId
         }
-      });
+      }]);
     }
 
   };
@@ -459,7 +459,7 @@ module.exports = function(mongoose, config) {
 
         if (validator.isEmail(data.rawQuery)) {
           if (data.emails.indexOf(data.rawQuery) == -1) {
-            data.emails.push(data.rawQuery);
+            data.emails = data.emails.concat([data.rawQuery]);
           }
         }
 
@@ -474,7 +474,7 @@ module.exports = function(mongoose, config) {
 
         if (pn.isValid()) {
           if (data.phones.indexOf(data.rawQuery) == -1) {
-            data.phones.push(data.rawQuery);
+            data.phones = data.phones.concat([data.rawQuery]);
           }
         }
 
@@ -1068,11 +1068,11 @@ module.exports = function(mongoose, config) {
 
       switch(chairPos) {
         case 'first':
-          person.chairs.first.push(chairObj);
+          person.chairs.first = person.chairs.first.concat([chairObj]);
           break;
 
         case 'second':
-          person.chairs.second.push(chairObj);
+          person.chairs.second = person.chairs.second.concat([chairObj]);
           break;
       }
 

package/lib/Round.js

@@ -315,7 +315,7 @@ module.exports = function(mongoose, config) {
     });
 
     // If not, add it
-    if (!match) vehicle.investments.push(investment);
+    if (!match) vehicle.investments = vehicle.investments.concat([investment]);
     // note there is no need to compute vehicle cost as it is done during upsert
 
   };
@@ -342,9 +342,7 @@ module.exports = function(mongoose, config) {
     });
 
     // If not, add it
-    if (!match) self.people.push({
-      person: person
-    });
+    if (!match) self.people = self.people.concat([{person: person}]);
 
   };
 
@@ -392,7 +390,7 @@ module.exports = function(mongoose, config) {
         realized: data.realized,
         valuationDate: data.valuationDate
       };
-      self.vehicles.push(vehicle);
+      self.vehicles = self.vehicles.concat([vehicle]);
     }
 
     return vehicle;

package/package.json

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