@dhyasama/totem-models 9.9.1 → 9.10.0

package/lib/Fund.js

@@ -155,6 +155,47 @@ module.exports = function(mongoose, config) {
 
   };
 
+  Fund.statics.modify = function(filter, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+    // at this time, the only known and approved use of this method is for adding and removing roles in totem web
+
+    if (!cb) throw new Error('cb is required');
+    if (!filter) { return cb(new Error('filter 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.findOneAndUpdate(filter, update, { upsert: false, new: true }, cb);
+
+  };
+
+  Fund.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
+    // at this time, the only known and approved use of this method is for the org edit form in totem web
+
+    if (!cb) throw new Error('cb is required');
+    if (!id) { return cb(new Error('id is required'), null); }
+    if (!mongoose.Types.ObjectId.isValid(id)) { return cb(new Error('id is not a valid ObjectId'), 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);
+
+  };
+
   Fund.set('usePushEach', true);
   Fund.set('autoIndex', false);
   Fund.set('usePushEach', true);

package/lib/LimitedPartner.js

@@ -637,17 +637,38 @@ module.exports = function(mongoose, config) {
 
   };
 
+  LimitedPartner.statics.modify = function(filter, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+    // at this time, the only known and approved use of this method is for adding and removing roles in totem web
+
+    if (!cb) throw new Error('cb is required');
+    if (!filter) { return cb(new Error('filter 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.findOneAndUpdate(filter, update, { upsert: false, new: true }, cb);
+
+  };
+
   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
+    // at this time, the only known and approved use of this method is for the org edit form in totem web
 
-    if (!cb) { throw new Error('cb is required'); }
+    if (!cb) throw new Error('cb is required');
     if (!id) { return cb(new Error('id is required'), null); }
     if (!mongoose.Types.ObjectId.isValid(id)) { return cb(new Error('id is not a valid ObjectId'), null); }
-    if (!update) { return cb(new Error('options is required'), null); }
+    if (!update) { return cb(new Error('update is required'), null); }
 
-    const self = this;
+    var self = this;
 
     // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
     // options runValidators defaults false which is ok since we have upsert false

package/lib/Organization.js

@@ -1649,6 +1649,7 @@ module.exports = function(mongoose, config) {
 
     if (!cb) throw new Error('cb is required');
     if (!id) { return cb(new Error('id is required'), null); }
+    if (!mongoose.Types.ObjectId.isValid(id)) { return cb(new Error('id is not a valid ObjectId'), null); }
     if (!update) { return cb(new Error('update is required'), null); }
 
     var self = this;

package/lib/Person.js

@@ -594,16 +594,38 @@ module.exports = function(mongoose, config) {
     });
   };
 
+  Person.statics.modify = function(filter, update, cb) {
+
+    // VERY IMPORTANT NOTE
+    // findByIdAndUpdate and findOneAndUpdate do not trigger pre-save hook so that code will not run here
+    // at this time, the only known and approved use of this method is for adding and removing roles in totem web
+
+    if (!cb) throw new Error('cb is required');
+    if (!filter) { return cb(new Error('filter 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.findOneAndUpdate(filter, update, { upsert: false, new: true }, cb);
+
+  };
+
   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
+    // at this time, the only known and approved use of this method is for the org edit form in totem web
 
     if (!cb) throw new Error('cb is required');
     if (!id) { return cb(new Error('id is required'), null); }
+    if (!mongoose.Types.ObjectId.isValid(id)) { return cb(new Error('id is not a valid ObjectId'), null); }
     if (!update) { return cb(new Error('update is required'), null); }
 
-    const self = this;
+    var self = this;
 
     // https://mongoosejs.com/docs/api.html#model_Model.findOneAndUpdate
     // options runValidators defaults false which is ok since we have upsert false

package/package.json

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