@ruiapp/rapid-core 0.10.5 → 0.10.6

package/dist/index.js

@@ -2600,8 +2600,8 @@ async function findEntities(server, dataAccessor, options) {
             }
         }
     });
-    // if `keepNonPropertyFields` is true and `properties` are not specified, then select relation columns automatically.
-    if (options.keepNonPropertyFields && (!options.properties || !options.properties.length)) {
+    // if `keepNonPropertyFields` is true, then select relation columns automatically.
+    if (options.keepNonPropertyFields) {
         const oneRelationPropertiesWithNoLinkTable = getEntityPropertiesIncludingBase(server, model).filter((property) => property.relation === "one" && !property.linkTableName);
         oneRelationPropertiesWithNoLinkTable.forEach((property) => {
             if (property.targetIdColumnName) {
@@ -2675,12 +2675,12 @@ async function findEntities(server, dataAccessor, options) {
                                 return lodash.find(relationLinks, (link) => {
                                     return link[relationProperty.selfIdColumnName] == row["id"] && link[relationProperty.targetIdColumnName] == targetEntity["id"];
                                 });
-                            }).map((targetEntity) => mapDbRowToEntity(server, relationModel, targetEntity, options.keepNonPropertyFields));
+                            });
                         }
                         else {
                             row[relationProperty.code] = lodash.filter(relationLinks, (link) => {
                                 return link[relationProperty.selfIdColumnName] == row["id"];
-                            }).map((link) => mapDbRowToEntity(server, relationModel, link.targetEntity, options.keepNonPropertyFields));
+                            }).map((link) => link.targetEntity);
                         }
                     });
                 }
@@ -2708,20 +2708,20 @@ async function findEntities(server, dataAccessor, options) {
                         selectRelationOptions: relationOptions[relationProperty.code],
                     });
                 }
-                const targetModel = server.getModel({
+                server.getModel({
                     singularCode: relationProperty.targetSingularCode,
                 });
                 rows.forEach((row) => {
                     if (isManyRelation) {
                         row[relationProperty.code] = lodash.filter(relatedEntities, (relatedEntity) => {
                             return relatedEntity[relationProperty.selfIdColumnName] == row.id;
-                        }).map((item) => mapDbRowToEntity(server, targetModel, item, options.keepNonPropertyFields));
+                        });
                     }
                     else {
-                        row[relationProperty.code] = mapDbRowToEntity(server, targetModel, lodash.find(relatedEntities, (relatedEntity) => {
+                        row[relationProperty.code] = lodash.find(relatedEntities, (relatedEntity) => {
                             // TODO: id property code should be configurable.
                             return relatedEntity["id"] == row[relationProperty.targetIdColumnName];
-                        }), options.keepNonPropertyFields);
+                        });
                     }
                 });
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ruiapp/rapid-core",
-  "version": "0.10.5",
+  "version": "0.10.6",
   "description": "",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/dataAccess/entityManager.ts

@@ -208,8 +208,8 @@ async function findEntities(server: IRpdServer, dataAccessor: IRpdDataAccessor,
     }
   });
 
-  // if `keepNonPropertyFields` is true and `properties` are not specified, then select relation columns automatically.
-  if (options.keepNonPropertyFields && (!options.properties || !options.properties.length)) {
+  // if `keepNonPropertyFields` is true, then select relation columns automatically.
+  if (options.keepNonPropertyFields) {
     const oneRelationPropertiesWithNoLinkTable = getEntityPropertiesIncludingBase(server, model).filter(
       (property) => property.relation === "one" && !property.linkTableName,
     );
@@ -289,11 +289,11 @@ async function findEntities(server: IRpdServer, dataAccessor: IRpdDataAccessor,
                 return find(relationLinks, (link: any) => {
                   return link[relationProperty.selfIdColumnName!] == row["id"] && link[relationProperty.targetIdColumnName!] == targetEntity["id"];
                 });
-              }).map((targetEntity) => mapDbRowToEntity(server, relationModel, targetEntity, options.keepNonPropertyFields));
+              });
             } else {
               row[relationProperty.code] = filter(relationLinks, (link: any) => {
                 return link[relationProperty.selfIdColumnName!] == row["id"];
-              }).map((link) => mapDbRowToEntity(server, relationModel, link.targetEntity, options.keepNonPropertyFields));
+              }).map((link) => link.targetEntity);
             }
           });
         }
@@ -332,17 +332,12 @@ async function findEntities(server: IRpdServer, dataAccessor: IRpdDataAccessor,
           if (isManyRelation) {
             row[relationProperty.code] = filter(relatedEntities, (relatedEntity: any) => {
               return relatedEntity[relationProperty.selfIdColumnName!] == row.id;
-            }).map((item) => mapDbRowToEntity(server, targetModel!, item, options.keepNonPropertyFields));
+            });
           } else {
-            row[relationProperty.code] = mapDbRowToEntity(
-              server,
-              targetModel!,
-              find(relatedEntities, (relatedEntity: any) => {
-                // TODO: id property code should be configurable.
-                return relatedEntity["id"] == row[relationProperty.targetIdColumnName!];
-              }),
-              options.keepNonPropertyFields,
-            );
+            row[relationProperty.code] = find(relatedEntities, (relatedEntity: any) => {
+              // TODO: id property code should be configurable.
+              return relatedEntity["id"] == row[relationProperty.targetIdColumnName!];
+            });
           }
         });
       }