nodester 0.7.16 → 0.7.17

package/lib/models/mixins.js

@@ -218,7 +218,13 @@ async function _updateOne(
 ) {
 	try {
 		const include = opts.include ?? [];
-		let instance = await this.findOne({ where, include });
+		const hasValidWhere = where && Object.keys(where).length > 0 && Object.values(where).every(val => val !== undefined);
+		let instance = null;
+
+		if (hasValidWhere) {
+			instance = await this.findOne({ where, include });
+		}
+
 		let isNewRecord = false;
 
 		if (!instance) {
@@ -285,7 +291,10 @@ async function _updateOne(
 						}
 						return associatedModel.updateOne(
 							where,
-							singleData,
+							{
+								...singleData,
+								[foreignKey]: instance.id
+							},
 							associationUpdateOpts
 						)
 					});
@@ -305,12 +314,18 @@ async function _updateOne(
 					}
 					else {
 						// Note: for now we are only able to work with a model with single PrimaryKey:
-						const where = {
-							[pkField]: includeData[pkField]
+						const where = {}
+						if (includeData[pkField] !== undefined && includeData[pkField] !== null) {
+							where[pkField] = includeData[pkField];
+						} else {
+							where[foreignKey] = instance.id;
 						}
 						fullInstanceData[association] = await associatedModel.updateOne(
 							where,
-							includeData,
+							{
+								...includeData,
+								[foreignKey]: instance.id
+							},
 							associationUpdateOpts
 						);
 					}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodester",
-  "version": "0.7.16",
+  "version": "0.7.17",
   "description": "A versatile REST framework for Node.js",
   "directories": {
     "docs": "docs",