@nestledjs/api 0.1.6 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestledjs/api",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "generators": "./generators.json",
   "type": "commonjs",
   "main": "./src/index.js",

package/src/generate-crud/files/data-access/src/lib/api-crud-data-access.service.ts__tmpl__

@@ -125,26 +125,22 @@ function generateRelationHandling(model, operation) {
   // Generate the improved iteration logic
   code += '    for (const [relationName, config] of Object.entries(relationMappings)) {\n';
   code += '      if (config.ids !== undefined && config.ids !== null) {\n';
+  code += '        const ids = Array.isArray(config.ids) ? config.ids.map(id => ({ id })) : [{ id: config.ids }];\n';
+  code += '        \n';
+  code += '        if (config.isList) {\n';
   
-  // Improved logic based on relationship type and operation
-  code += '        if (config.isVirtual) {\n';
-  code += '          // Virtual relationships (many-to-many or one-to-many reverse)\n';
-  code += '          if (config.isList) {\n';
-  code += '            // Many-to-many or one-to-many from the "many" side - use set for updates\n';
-  code += `            data[relationName] = { ${operation === 'create' ? 'connect' : 'set'}: Array.isArray(config.ids) ? config.ids.map(id => ({ id })) : [{ id: config.ids }] };\n`;
-  code += '          } else {\n';
-  code += '            // Single virtual relationship - use connect\n';
-  code += '            data[relationName] = { connect: { id: config.ids } };\n';
-  code += '          }\n';
+  if (operation === 'update') {
+    code += '          // List relationships: use set for updates on virtual relations, connect for foreign key relations\n';
+    code += '          const relationOperation = config.isVirtual ? \'set\' : \'connect\';\n';
+  } else {
+    code += '          // List relationships: always use connect for creates\n';
+    code += '          const relationOperation = \'connect\';\n';
+  }
+  
+  code += '          data[relationName] = { [relationOperation]: ids };\n';
   code += '        } else {\n';
-  code += '          // Foreign key relationships (belongs-to) - always use connect for single values\n';
-  code += '          if (config.isList) {\n';
-  code += '            // This case should be rare, but handle arrays if they exist\n';
-  code += '            data[relationName] = { connect: Array.isArray(config.ids) ? config.ids.map(id => ({ id })) : [{ id: config.ids }] };\n';
-  code += '          } else {\n';
-  code += '            // Single foreign key relationship - use connect\n';
-  code += '            data[relationName] = { connect: { id: config.ids } };\n';
-  code += '          }\n';
+  code += '          // Single relationship - always use connect\n';
+  code += '          data[relationName] = { connect: { id: config.ids } };\n';
   code += '        }\n';
   code += '      }\n';
   code += '    }\n';
@@ -179,8 +175,8 @@ export class ApiCrudDataAccessService {
     const count = await this.data['<%= model.modelPropertyName %>'].count({
       ...this.data.filter(input)
     });
-    const take = input?.take || 10
-    const skip = input?.skip || 0
+    const take = input?.take ?? 10
+    const skip = input?.skip ?? 0
     const page = Math.floor(skip / take)
      return {
       take,