@nestledjs/api 0.1.6 → 0.1.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nestledjs/api",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "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,17 @@ 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';
-  
-  // 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';
+  code += '        // Virtual list relationships use set for updates, connect for creates\n';
+  code += '        // All other relationships always use connect\n';
+  code += '        const useSet = config.isVirtual && config.isList && operation === \'update\';\n';
+  code += '        const relationOperation = useSet ? \'set\' : \'connect\';\n';
+  code += '        const ids = Array.isArray(config.ids) ? config.ids.map(id => ({ id })) : [{ id: config.ids }];\n';
+  code += '        \n';
+  code += '        if (config.isList || useSet) {\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 - extract the single id\n';
+  code += '          data[relationName] = { connect: { id: config.ids } };\n';
   code += '        }\n';
   code += '      }\n';
   code += '    }\n';