@clairejs/server 3.28.5 → 3.28.7

package/README.md

@@ -1,5 +1,13 @@
 ## Change Log
 
+#### 3.28.7
+
+-   fix update records with hasMany field
+
+#### 3.28.6
+
+-   fix calling stripData in with null / undefined
+
 #### 3.28.5
 
 -   implement new job scheduler

package/dist/http/controller/DefaultHttpRequestHandler.js

@@ -49,7 +49,7 @@ let DefaultHttpRequestHandler = class DefaultHttpRequestHandler extends Abstract
             });
         const response = await endpoint.controller[endpoint.name](...params);
         //-- validate response value against response dto
-        if (endpoint.responseDto) {
+        if (endpoint.responseDto && response.value !== null && response.value !== undefined) {
             response.value = stripData(response.value, endpoint.responseDto);
         }
         return response;

package/dist/http/repository/ModelRepository.js

@@ -510,7 +510,7 @@ export class ModelRepository extends AbstractRepository {
             const adapter = this.db.use(model, tx);
             const allInnerRecords = await adapter.getRecords({ _eq: { [field.hasMany.column]: theRecord.id } }, { limit: field.hasMany?.single ? 1 : undefined });
             const currentIds = allInnerRecords.map((r) => r.id);
-            const newIds = innerUpdates.map((r) => r.id).filter((id) => !!id);
+            const newIds = innerUpdates.flatMap((r) => (r.id ? [r.id] : []));
             const tobeRemovedIds = currentIds.filter((id) => !newIds.includes(id));
             const tobeAdded = innerUpdates
                 .filter((r) => !r.id)
@@ -537,7 +537,10 @@ export class ModelRepository extends AbstractRepository {
             //-- return the new inners records
             const finalInnerRecords = tobeAdded
                 .map((toAdd, index) => ({ ...toAdd, ...added.records[index] }))
-                .concat(updatedToBeKept.map((r) => r.modified[0]));
+                .concat(tobeKept.map((tbk) => {
+                const matched = updatedToBeKept.find((utbk) => utbk.modified[0].id === tbk.id);
+                return { ...tbk, ...matched };
+            }));
             theRecord[field.name] = (field.hasMany?.single ? finalInnerRecords[0] : finalInnerRecords);
         }
         //-- ok clean up

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@clairejs/server",
-    "version": "3.28.5",
+    "version": "3.28.7",
     "description": "Claire server NodeJs framework written in Typescript.",
     "types": "dist/index.d.ts",
     "main": "dist/index.js",