refactor(js,ts): move formatting logic into formatParamsAsJavaScriptObject()

Author: php-coder

src/cli.js

@@ -239,7 +239,9 @@ const createEndpoints = async (destDir, { lang }, config) => {
                 if (params.length === 0) {
                     return params
                 }
-                return Array.from(
+                const indentLevel = 12
+                const indent = ' '.repeat(indentLevel)
+                return `\n${indent}{ ` + Array.from(
                         new Set(params),
                         p => {
                             const bindTarget = p.substring(0, 1)
@@ -251,7 +253,7 @@ const createEndpoints = async (destDir, { lang }, config) => {
                             }
                             return `"${paramName}": ${prefix}.${paramName}`
                         }
-                    ).join(', ')
+                    ).join(', ') + ' },'
             },
 
             // "SELECT *\n   FROM foo WHERE id = :p.id" => "SELECT * FROM foo WHERE id = :id"

src/templates/routes.js.ejs

@@ -17,9 +17,7 @@ endpoints.forEach(function(endpoint) {
         const hasGetMany = method.name === 'get_list'
         const sql = formatQueryForJs(method.query, 12)
         const params = extractParamsFromQuery(method.query)
-        const formattedParams = params.length > 0
-            ? '\n            { ' + formatParamsAsJavaScriptObject(params, method) + ' },'
-            : ''
+        const formattedParams = formatParamsAsJavaScriptObject(params, method)
 
         if (hasGetOne || hasGetMany) {
 %>

src/templates/routes.ts.ejs

@@ -20,9 +20,7 @@ endpoints.forEach(function(endpoint) {
         const hasGetMany = method.name === 'get_list'
         const sql = formatQueryForJs(method.query, 12)
         const params = extractParamsFromQuery(method.query)
-        const formattedParams = params.length > 0
-            ? '\n            { ' + formatParamsAsJavaScriptObject(params, method) + ' },'
-            : ''
+        const formattedParams = formatParamsAsJavaScriptObject(params, method)
 
         if (hasGetOne || hasGetMany) {
 %>