@abaplint/transpiler-cli 2.5.47 → 2.5.49

package/build/bundle.js

@@ -68271,6 +68271,8 @@ __exportStar(__webpack_require__(/*! ./source_field_symbol */ "./node_modules/@a
 __exportStar(__webpack_require__(/*! ./source_field */ "./node_modules/@abaplint/transpiler/build/src/expressions/source_field.js"), exports);
 __exportStar(__webpack_require__(/*! ./source */ "./node_modules/@abaplint/transpiler/build/src/expressions/source.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_cond */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_cond.js"), exports);
+__exportStar(__webpack_require__(/*! ./sql_field */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_field.js"), exports);
+__exportStar(__webpack_require__(/*! ./sql_from_source */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_from_source.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_from */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_from.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_join */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_join.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_source_simple */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_source_simple.js"), exports);
@@ -69082,6 +69084,7 @@ const traversal_1 = __webpack_require__(/*! ../traversal */ "./node_modules/@aba
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const simple_source3_1 = __webpack_require__(/*! ./simple_source3 */ "./node_modules/@abaplint/transpiler/build/src/expressions/simple_source3.js");
 const field_chain_1 = __webpack_require__(/*! ./field_chain */ "./node_modules/@abaplint/transpiler/build/src/expressions/field_chain.js");
+const sql_field_name_1 = __webpack_require__(/*! ./sql_field_name */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_field_name.js");
 class SQLCondTranspiler {
     transpile(node, traversal) {
         var _a;
@@ -69134,11 +69137,15 @@ class SQLCondTranspiler {
         if (c.getChildren().length !== 3) {
             return this.basicConditionNew(c, traversal, filename);
         }
-        const fieldName = c.findDirectExpression(abaplint.Expressions.SQLFieldName);
+        let fieldName = undefined;
+        const fieldNameExpression = c.findDirectExpression(abaplint.Expressions.SQLFieldName);
+        if (fieldNameExpression) {
+            fieldName = new sql_field_name_1.SQLFieldNameTranspiler().transpile(fieldNameExpression, traversal).getCode();
+        }
         const operator = c.findDirectExpression(abaplint.Expressions.SQLCompareOperator);
         const source = c.findDirectExpression(abaplint.Expressions.SQLSource);
         if (fieldName && source && operator === undefined && c.findDirectTokenByText("LIKE")) {
-            ret += fieldName.concatTokens() + " LIKE ";
+            ret += fieldName + " LIKE ";
             ret += this.sqlSource(source, traversal, filename);
             return ret;
         }
@@ -69152,7 +69159,7 @@ class SQLCondTranspiler {
         else if (op.toUpperCase() === "NE") {
             op = "<>";
         }
-        ret += fieldName.concatTokens() + " " + op + " ";
+        ret += fieldName + " " + op + " ";
         ret += this.sqlSource(source, traversal, filename);
         return ret;
     }
@@ -69193,8 +69200,9 @@ class SQLCondTranspiler {
             if (ret !== "") {
                 ret += " ";
             }
-            if (child.get() instanceof abaplint.Expressions.SQLFieldName) {
-                ret += child.concatTokens();
+            if (child.get() instanceof abaplint.Expressions.SQLFieldName
+                && child instanceof abaplint.Nodes.ExpressionNode) {
+                ret += new sql_field_name_1.SQLFieldNameTranspiler().transpile(child, traversal).getCode();
             }
             else if (child.get() instanceof abaplint.Expressions.SQLSource
                 && child instanceof abaplint.Nodes.ExpressionNode) {
@@ -69212,6 +69220,69 @@ exports.SQLCondTranspiler = SQLCondTranspiler;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_field.js":
+/*!******************************************************************************!*\
+  !*** ./node_modules/@abaplint/transpiler/build/src/expressions/sql_field.js ***!
+  \******************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLFieldTranspiler = void 0;
+const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
+const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+const sql_field_name_1 = __webpack_require__(/*! ./sql_field_name */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_field_name.js");
+class SQLFieldTranspiler {
+    transpile(node, traversal) {
+        const chunk = new chunk_1.Chunk();
+        for (const c of node.getChildren()) {
+            if (c instanceof abaplint.Nodes.TokenNode) {
+                // keywords
+                chunk.appendString(c.concatTokens() + " ");
+            }
+            else if (c.get() instanceof abaplint.Expressions.SQLFieldName) {
+                chunk.appendChunk(new sql_field_name_1.SQLFieldNameTranspiler().transpile(c, traversal));
+            }
+            else {
+                chunk.appendString(c.concatTokens() + " ");
+            }
+        }
+        return chunk;
+    }
+}
+exports.SQLFieldTranspiler = SQLFieldTranspiler;
+//# sourceMappingURL=sql_field.js.map
+
+/***/ }),
+
+/***/ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_field_name.js":
+/*!***********************************************************************************!*\
+  !*** ./node_modules/@abaplint/transpiler/build/src/expressions/sql_field_name.js ***!
+  \***********************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLFieldNameTranspiler = void 0;
+const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+class SQLFieldNameTranspiler {
+    transpile(node, _traversal) {
+        const chunk = new chunk_1.Chunk();
+        let concat = node.concatTokens();
+        if (concat.includes("~") && concat.split("~")[0].includes("/")) {
+            concat = "'" + concat.replace("~", "'~");
+        }
+        chunk.appendString(concat);
+        return chunk;
+    }
+}
+exports.SQLFieldNameTranspiler = SQLFieldNameTranspiler;
+//# sourceMappingURL=sql_field_name.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_from.js":
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/transpiler/build/src/expressions/sql_from.js ***!
@@ -69225,19 +69296,32 @@ exports.SQLFromTranspiler = void 0;
 const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const sql_join_1 = __webpack_require__(/*! ./sql_join */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_join.js");
+const sql_from_source_1 = __webpack_require__(/*! ./sql_from_source */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_from_source.js");
 class SQLFromTranspiler {
     transpile(node, traversal) {
         const chunk = new chunk_1.Chunk();
         for (const c of node.getChildren()) {
             if (c instanceof abaplint.Nodes.TokenNode) {
+                // keywords
                 chunk.appendString(c.concatTokens() + " ");
             }
             else if (c.get() instanceof abaplint.Expressions.SQLJoin) {
                 chunk.appendChunk(new sql_join_1.SQLJoinTranspiler().transpile(c, traversal));
                 chunk.appendString(" ");
             }
+            else if (c.get() instanceof abaplint.Expressions.SQLFromSource) {
+                chunk.appendChunk(new sql_from_source_1.SQLFromSourceTranspiler().transpile(c, traversal));
+            }
             else {
-                chunk.appendString(c.concatTokens() + " ");
+                if (c.findFirstExpression(abaplint.Expressions.Dynamic)) {
+                    chunk.appendString(c.concatTokens() + " ");
+                }
+                else if (c.concatTokens().includes("/")) {
+                    chunk.appendString("'" + c.concatTokens() + "' ");
+                }
+                else {
+                    chunk.appendString(c.concatTokens() + " ");
+                }
             }
         }
         return chunk;
@@ -69248,6 +69332,41 @@ exports.SQLFromTranspiler = SQLFromTranspiler;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_from_source.js":
+/*!************************************************************************************!*\
+  !*** ./node_modules/@abaplint/transpiler/build/src/expressions/sql_from_source.js ***!
+  \************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLFromSourceTranspiler = void 0;
+const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
+const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+class SQLFromSourceTranspiler {
+    transpile(node, _traversal) {
+        const chunk = new chunk_1.Chunk();
+        for (const c of node.getChildren()) {
+            if (c instanceof abaplint.Nodes.TokenNode) {
+                // keywords
+                chunk.appendString(c.concatTokens() + " ");
+            }
+            else if (c.get() instanceof abaplint.Expressions.DatabaseTable && c.concatTokens().includes("/")) {
+                chunk.appendString("'" + c.concatTokens() + "' ");
+            }
+            else {
+                chunk.appendString(c.concatTokens() + " ");
+            }
+        }
+        return chunk;
+    }
+}
+exports.SQLFromSourceTranspiler = SQLFromSourceTranspiler;
+//# sourceMappingURL=sql_from_source.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_join.js":
 /*!*****************************************************************************!*\
   !*** ./node_modules/@abaplint/transpiler/build/src/expressions/sql_join.js ***!
@@ -69261,16 +69380,21 @@ exports.SQLJoinTranspiler = void 0;
 const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
 const sql_cond_1 = __webpack_require__(/*! ./sql_cond */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_cond.js");
+const sql_from_source_1 = __webpack_require__(/*! ./sql_from_source */ "./node_modules/@abaplint/transpiler/build/src/expressions/sql_from_source.js");
 class SQLJoinTranspiler {
     transpile(node, traversal) {
         const chunk = new chunk_1.Chunk();
         for (const c of node.getChildren()) {
             if (c instanceof abaplint.Nodes.TokenNode) {
+                // keywords
                 chunk.appendString(c.concatTokens() + " ");
             }
             else if (c.get() instanceof abaplint.Expressions.SQLCond) {
                 chunk.appendChunk(new sql_cond_1.SQLCondTranspiler().transpile(c, traversal));
             }
+            else if (c.get() instanceof abaplint.Expressions.SQLFromSource) {
+                chunk.appendChunk(new sql_from_source_1.SQLFromSourceTranspiler().transpile(c, traversal));
+            }
             else {
                 chunk.appendString(c.concatTokens() + " ");
             }
@@ -74637,14 +74761,17 @@ class SelectTranspiler {
         let select = "SELECT ";
         const fieldList = node.findFirstExpression(abaplint.Expressions.SQLFieldList)
             || node.findFirstExpression(abaplint.Expressions.SQLFieldListLoop);
-        if ((fieldList === null || fieldList === void 0 ? void 0 : fieldList.getChildren().length) === 1) {
-            select += fieldList.concatTokens();
-        }
-        else {
-            // add commas between field names, this is required for SQLite, and easy to remove in other clients?
-            select += fieldList === null || fieldList === void 0 ? void 0 : fieldList.findAllExpressions(abaplint.Expressions.SQLField).map(e => e.concatTokens()).join(", ");
+        const fields = [];
+        for (const f of (fieldList === null || fieldList === void 0 ? void 0 : fieldList.getChildren()) || []) {
+            if (f instanceof abaplint.Nodes.ExpressionNode && f.get() instanceof abaplint.Expressions.SQLField) {
+                const code = new expressions_1.SQLFieldTranspiler().transpile(f, traversal).getCode();
+                fields.push(code);
+            }
+            else {
+                fields.push(f.concatTokens());
+            }
         }
-        select += " ";
+        select += fields.join(", ") + " ";
         const from = node.findFirstExpression(abaplint.Expressions.SQLFrom);
         if (from) {
             select += new sql_from_1.SQLFromTranspiler().transpile(from, traversal).getCode();
@@ -74698,6 +74825,10 @@ class SelectTranspiler {
             const faeTranspiled = new expressions_1.SQLSourceTranspiler().transpile(fn, traversal).getCode();
             select = select.replace(new RegExp(" " + escapeRegExp(faeTranspiled), "g"), " " + unique);
             select = select.replace(unique + ".get().table_line.get()", unique + ".get()"); // there can be only one?
+            let by = `Object.keys(${target}.getRowType().get())`;
+            if (keys.length > 0) {
+                by = JSON.stringify(keys);
+            }
             const code = `if (${faeTranspiled}.array().length === 0) {
   throw "FAE, todo, empty table";
 } else {
@@ -74705,8 +74836,8 @@ class SelectTranspiler {
   for await (const ${unique} of abap.statements.loop(${faeTranspiled})) {
     await abap.statements.select(${target}, {select: "${select.trim()}"${extra}}, {appending: true});
   }
-  abap.statements.sort(${target}, {by: Object.keys(${target}.getRowType().get()).map(k => { return {component: k}; })});
-  await abap.statements.deleteInternal(${target}, {adjacent: true});
+  abap.statements.sort(${target}, {by: ${by}.map(k => { return {component: k}; })});
+  await abap.statements.deleteInternal(${target}, {adjacent: true, by: ${by}});
   abap.builtin.sy.get().dbcnt.set(${target}.array().length);
 }`;
             return new chunk_1.Chunk().append(code, node, traversal);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.5.47",
+  "version": "2.5.49",
   "description": "Transpiler - Command Line Interface",
   "bin": {
     "abap_transpile": "./abap_transpile"
@@ -25,7 +25,7 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.5.47",
+    "@abaplint/transpiler": "^2.5.49",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",