@abaplint/transpiler-cli 2.7.13 → 2.7.15

package/build/bundle.js

@@ -42089,7 +42089,14 @@ class DataDefinition extends _abstract_object_1.AbstractObject {
     }
     findFieldNames(tree) {
         var _a, _b;
-        for (const e of tree.findAllExpressions(expressions_1.CDSElement)) {
+        let expr = tree.findFirstExpression(expressions_1.CDSSelect);
+        if (expr === undefined) {
+            expr = tree.findFirstExpression(expressions_1.CDSAnnotate);
+        }
+        if (expr === undefined) {
+            expr = tree.findFirstExpression(expressions_1.CDSDefineProjection);
+        }
+        for (const e of (expr === null || expr === void 0 ? void 0 : expr.findDirectExpressions(expressions_1.CDSElement)) || []) {
             let found = (_a = e.findDirectExpression(expressions_1.CDSAs)) === null || _a === void 0 ? void 0 : _a.findDirectExpression(expressions_1.CDSName);
             if (found === undefined) {
                 const list = e.findDirectExpressions(expressions_1.CDSName);
@@ -47086,7 +47093,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.101.6";
+        return "2.101.7";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -51819,7 +51826,9 @@ Current rules:
 * ENUMs, but does not nessesarily give the correct type and value
 * MESSAGE with non simple source
 
-Only one transformation is applied to a statement at a time, so multiple steps might be required to do the full downport.`,
+Only one transformation is applied to a statement at a time, so multiple steps might be required to do the full downport.
+
+Make sure to test the downported code, it might not always be completely correct.`,
             tags: [_irule_1.RuleTag.Downport, _irule_1.RuleTag.Quickfix],
         };
     }
@@ -69569,20 +69578,45 @@ exports.ConstantTranspiler = ConstantTranspiler;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DatabaseTableTranspiler = void 0;
-const core_1 = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
+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 DatabaseTableTranspiler {
+    constructor(prefix = true) {
+        this.prefix = prefix;
+    }
     transpile(node, traversal) {
-        const ret = new chunk_1.Chunk();
-        const dyn = node.findDirectExpression(core_1.Expressions.Dynamic);
+        const chunk = new chunk_1.Chunk();
+        const concat = node.concatTokens();
+        let val = "";
+        if (this.prefix === true) {
+            val = `" + abap.dbo.schemaPrefix + "\\"" + abap.dbo.tablePrefix + "`;
+        }
+        else {
+            val = "\"";
+        }
+        const dyn = node.findDirectExpression(abaplint.Expressions.Dynamic);
         if (dyn) {
-            const sub = dyn.getChildren()[1];
-            return traversal.traverse(sub);
+            if (concat.startsWith("('")) {
+                val += concat.substring(2, concat.length - 2).toLowerCase();
+            }
+            else {
+                const foo = traversal.traverse(dyn.findDirectExpression(abaplint.Expressions.FieldChain));
+                if (this.prefix === false) {
+                    return foo;
+                }
+                val += "\" + " + foo.getCode() + ".get() + \"";
+            }
         }
         else {
-            ret.appendString('"' + node.concatTokens() + '"');
+            val += concat.toLowerCase();
         }
-        return ret;
+        if (this.prefix === true) {
+            val += "\\\"";
+        }
+        else {
+            val += "\"";
+        }
+        return chunk.appendString(val);
     }
 }
 exports.DatabaseTableTranspiler = DatabaseTableTranspiler;
@@ -70047,9 +70081,14 @@ class MethodCallBodyTranspiler {
         let pt = false;
         for (const c of node.getChildren()) {
             if (c instanceof core_1.Nodes.TokenNode) {
-                // PARAMETER-TABLE
-                pt = true;
-                continue;
+                if (c.concatTokens().toUpperCase() === "PARAMETER") {
+                    // PARAMETER-TABLE
+                    pt = true;
+                }
+                else if (c.concatTokens().toUpperCase() === "EXCEPTION") {
+                    // todo: handle EXCEPTION-TABLE, for now just produce valid javascript
+                    return ret;
+                }
             }
             else if (c.get() instanceof abaplint.Expressions.MethodCallParam) {
                 ret.appendChunk(new method_call_param_1.MethodCallParamTranspiler(this.m).transpile(c, traversal));
@@ -71008,21 +71047,8 @@ class SQLFromTranspiler {
                 chunk.appendChunk(new sql_from_source_1.SQLFromSourceTranspiler().transpile(c, traversal));
             }
             else {
-                if (c.findFirstExpression(abaplint.Expressions.Dynamic)) {
-                    let concat = c.concatTokens();
-                    if (concat.startsWith("'")) {
-                        // no escaping needed, guess its not possible to have table names with single or double quote part of name
-                        concat = "\\\"" + concat.substring(1, concat.length - 1) + "\\\"";
-                    }
-                    chunk.appendString(concat + " ");
-                }
-                else if (c.concatTokens().includes("/")) {
-                    chunk.appendString("\\\"" + c.concatTokens() + "\\\" ");
-                }
-                else {
-                    const concat = c.concatTokens();
-                    chunk.appendString(concat + " ");
-                }
+                const concat = c.concatTokens();
+                chunk.appendString(concat + " ");
             }
         }
         return chunk;
@@ -71045,23 +71071,22 @@ 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");
+const database_table_1 = __webpack_require__(/*! ./database_table */ "./node_modules/@abaplint/transpiler/build/src/expressions/database_table.js");
 class SQLFromSourceTranspiler {
-    transpile(node, _traversal) {
+    transpile(node, traversal) {
         const chunk = new chunk_1.Chunk();
         for (const c of node.getChildren()) {
+            const concat = c.concatTokens();
             if (c instanceof abaplint.Nodes.TokenNode) {
                 // keywords
-                chunk.appendString(c.concatTokens() + " ");
+                chunk.appendString(concat + " ");
             }
-            else if (c.get() instanceof abaplint.Expressions.DatabaseTable && c.concatTokens().includes("/")) {
-                chunk.appendString("\\\"" + c.concatTokens() + "\\\" ");
-            }
-            else if (c.get() instanceof abaplint.Expressions.DatabaseTable && c.concatTokens().startsWith("('")) {
-                const concat = c.concatTokens();
-                chunk.appendString("\\\"" + concat.substring(2, concat.length - 2).toLowerCase() + "\\\" ");
+            else if (c.get() instanceof abaplint.Expressions.DatabaseTable) {
+                chunk.appendChunk(new database_table_1.DatabaseTableTranspiler().transpile(c, traversal));
+                chunk.appendString(" ");
             }
             else {
-                chunk.appendString(c.concatTokens() + " ");
+                chunk.appendString(concat + " ");
             }
         }
         return chunk;
@@ -73784,9 +73809,14 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.DeleteDatabaseTranspiler = 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 expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 class DeleteDatabaseTranspiler {
     transpile(node, traversal) {
-        const table = traversal.traverse(node.findFirstExpression(abaplint.Expressions.DatabaseTable));
+        const dbtab = node.findFirstExpression(abaplint.Expressions.DatabaseTable);
+        if (dbtab === undefined) {
+            throw new Error("internal error, InsertDatabaseTranspiler");
+        }
+        const table = new expressions_1.DatabaseTableTranspiler(false).transpile(dbtab, traversal);
         const options = [];
         const tab = node.findExpressionAfterToken("TABLE");
         if (tab) {
@@ -75127,9 +75157,14 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.InsertDatabaseTranspiler = 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 expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 class InsertDatabaseTranspiler {
     transpile(node, traversal) {
-        const table = traversal.traverse(node.findFirstExpression(abaplint.Expressions.DatabaseTable));
+        const dbtab = node.findFirstExpression(abaplint.Expressions.DatabaseTable);
+        if (dbtab === undefined) {
+            throw new Error("internal error, InsertDatabaseTranspiler");
+        }
+        const table = new expressions_1.DatabaseTableTranspiler(false).transpile(dbtab, traversal);
         const options = [];
         const values = node.findExpressionAfterToken("VALUES");
         if (values) {
@@ -75146,7 +75181,7 @@ class InsertDatabaseTranspiler {
             const tvalues = traversal.traverse(fromTable);
             options.push(`"table": ` + tvalues.getCode());
         }
-        return new chunk_1.Chunk(`await abap.statements.insertDatabase(${table.getCode().toLowerCase()}, {${options.join(", ")}});`);
+        return new chunk_1.Chunk(`await abap.statements.insertDatabase(${table.getCode()}, {${options.join(", ")}});`);
     }
 }
 exports.InsertDatabaseTranspiler = InsertDatabaseTranspiler;
@@ -75760,9 +75795,14 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.ModifyDatabaseTranspiler = 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 expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 class ModifyDatabaseTranspiler {
     transpile(node, traversal) {
-        const table = traversal.traverse(node.findFirstExpression(abaplint.Expressions.DatabaseTable));
+        const dbtab = node.findFirstExpression(abaplint.Expressions.DatabaseTable);
+        if (dbtab === undefined) {
+            throw new Error("internal error, ModifyDatabaseTranspiler");
+        }
+        const table = new expressions_1.DatabaseTableTranspiler(false).transpile(dbtab, traversal);
         const options = [];
         const tab = node.findExpressionAfterToken("TABLE");
         if (tab) {
@@ -77476,10 +77516,15 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.UpdateDatabaseTranspiler = 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 expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 class UpdateDatabaseTranspiler {
     transpile(node, traversal) {
         var _a;
-        const table = traversal.traverse(node.findFirstExpression(abaplint.Expressions.DatabaseTable));
+        const dbtab = node.findFirstExpression(abaplint.Expressions.DatabaseTable);
+        if (dbtab === undefined) {
+            throw new Error("internal error, UpdateDatabaseTranspiler");
+        }
+        const table = new expressions_1.DatabaseTableTranspiler(false).transpile(dbtab, traversal);
         const options = [];
         const cond = node.findFirstExpression(abaplint.Expressions.SQLCond);
         if (cond) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.13",
+  "version": "2.7.15",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -26,12 +26,12 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.7.13",
+    "@abaplint/transpiler": "^2.7.15",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
-    "@types/node": "^20.2.4",
-    "@abaplint/core": "^2.101.6",
+    "@types/node": "^20.2.5",
+    "@abaplint/core": "^2.101.7",
     "progress": "^2.0.3",
     "webpack": "^5.84.1",
     "webpack-cli": "^5.1.1",