@abaplint/transpiler-cli 2.7.4 → 2.7.6

package/build/bundle.js

@@ -19455,6 +19455,9 @@ class CurrentScope {
     getVersion() {
         return this.reg.getConfig().getVersion();
     }
+    getRegistry() {
+        return this.reg;
+    }
     addType(type) {
         if (type === undefined) {
             return;
@@ -19554,6 +19557,10 @@ class CurrentScope {
         const position = new _identifier_1.Identifier(usage, filename);
         (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().references.push({ position, resolved: referencing, referenceType: type, extra });
     }
+    addSQLConversion(fieldName, message, token) {
+        var _a;
+        (_a = this.current) === null || _a === void 0 ? void 0 : _a.getData().sqlConversion.push({ fieldName, message, token });
+    }
     ///////////////////////////
     findFunctionModule(name) {
         if (name === undefined) {
@@ -22440,7 +22447,7 @@ class DatabaseTable {
         const name = token.getStr();
         if (name === "(") {
             // dynamic
-            return;
+            return undefined;
         }
         const found = scope.getDDIC().lookupTableOrView2(name);
         if (found === undefined && scope.getDDIC().inErrorNamespace(name) === true) {
@@ -22453,6 +22460,7 @@ class DatabaseTable {
             scope.addReference(token, found.getIdentifier(), _reference_1.ReferenceType.TableReference, filename);
             scope.getDDICReferences().addUsing(scope.getParentObj(), { object: found, token: token, filename: filename });
         }
+        return found;
     }
 }
 exports.DatabaseTable = DatabaseTable;
@@ -24337,17 +24345,16 @@ const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@ab
 const inline_data_1 = __webpack_require__(/*! ./inline_data */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/inline_data.js");
 const target_1 = __webpack_require__(/*! ./target */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/target.js");
 const sql_from_1 = __webpack_require__(/*! ./sql_from */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_from.js");
-const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 const sql_for_all_entries_1 = __webpack_require__(/*! ./sql_for_all_entries */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_for_all_entries.js");
 const _scope_type_1 = __webpack_require__(/*! ../_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
+const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
+const sql_compare_1 = __webpack_require__(/*! ./sql_compare */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_compare.js");
 class Select {
     runSyntax(node, scope, filename, skipImplicitInto = false) {
         var _a, _b;
         const token = node.getFirstToken();
         const from = node.findDirectExpression(Expressions.SQLFrom);
-        if (from) {
-            new sql_from_1.SQLFrom().runSyntax(from, scope, filename);
-        }
+        const dbSources = from ? new sql_from_1.SQLFrom().runSyntax(from, scope, filename) : [];
         for (const inline of node.findAllExpressions(Expressions.InlineData)) {
             // todo, for now these are voided
             new inline_data_1.InlineData().runSyntax(inline, scope, filename, new basic_1.VoidType("SELECT_todo"));
@@ -24373,11 +24380,22 @@ class Select {
                 }
             }
         }
-        for (const s of node.findAllExpressions(Expressions.Source)) {
-            new source_1.Source().runSyntax(s, scope, filename);
+        // OFFSET
+        for (const s of node.findDirectExpressions(Expressions.SQLSource)) {
+            new sql_source_1.SQLSource().runSyntax(s, scope, filename);
         }
-        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
-            new source_1.Source().runSyntax(s, scope, filename);
+        for (const up of node.findDirectExpressions(Expressions.SQLUpTo)) {
+            for (const s of up.findDirectExpressions(Expressions.SQLSource)) {
+                new sql_source_1.SQLSource().runSyntax(s, scope, filename);
+            }
+        }
+        for (const fae of node.findDirectExpressions(Expressions.SQLForAllEntries)) {
+            for (const s of fae.findDirectExpressions(Expressions.SQLSource)) {
+                new sql_source_1.SQLSource().runSyntax(s, scope, filename);
+            }
+        }
+        for (const s of node.findAllExpressions(Expressions.SQLCompare)) {
+            new sql_compare_1.SQLCompare().runSyntax(s, scope, filename, dbSources);
         }
         if (scope.getType() === _scope_type_1.ScopeType.OpenSQL) {
             scope.pop(node.getLastToken().getEnd());
@@ -24737,6 +24755,79 @@ exports.SourceFieldSymbol = SourceFieldSymbol;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_compare.js":
+/*!****************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_compare.js ***!
+  \****************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLCompare = void 0;
+const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
+class SQLCompare {
+    runSyntax(node, scope, filename, tables) {
+        var _a;
+        let sourceType;
+        let token;
+        for (const s of node.findAllExpressions(Expressions.SQLSource)) {
+            token = s.getFirstToken();
+            sourceType = new sql_source_1.SQLSource().runSyntax(s, scope, filename);
+        }
+        const fieldName = (_a = node.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
+        if (fieldName && sourceType && token) {
+            // check compatibility for rule sql_value_conversion
+            const targetType = this.findType(fieldName, tables, scope);
+            let message = "";
+            if (sourceType instanceof basic_1.IntegerType
+                && targetType instanceof basic_1.CharacterType) {
+                message = "Integer to CHAR conversion";
+            }
+            else if (sourceType instanceof basic_1.IntegerType
+                && targetType instanceof basic_1.NumericType) {
+                message = "Integer to NUMC conversion";
+            }
+            else if (sourceType instanceof basic_1.NumericType
+                && targetType instanceof basic_1.IntegerType) {
+                message = "NUMC to Integer conversion";
+            }
+            else if (sourceType instanceof basic_1.CharacterType
+                && targetType instanceof basic_1.IntegerType) {
+                message = "CHAR to Integer conversion";
+            }
+            else if (sourceType instanceof basic_1.CharacterType
+                && targetType instanceof basic_1.CharacterType
+                && sourceType.getLength() > targetType.getLength()) {
+                message = "Source field longer than database field, CHAR -> CHAR";
+            }
+            else if (sourceType instanceof basic_1.NumericType
+                && targetType instanceof basic_1.NumericType
+                && sourceType.getLength() > targetType.getLength()) {
+                message = "Source field longer than database field, NUMC -> NUMC";
+            }
+            if (message !== "") {
+                scope.addSQLConversion(fieldName, message, token);
+            }
+        }
+    }
+    findType(fieldName, tables, scope) {
+        for (const t of tables) {
+            const type = t === null || t === void 0 ? void 0 : t.parseType(scope.getRegistry());
+            if (type instanceof basic_1.StructureType) {
+                return type.getComponentByName(fieldName);
+            }
+        }
+        return undefined;
+    }
+}
+exports.SQLCompare = SQLCompare;
+//# sourceMappingURL=sql_compare.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_for_all_entries.js":
 /*!************************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_for_all_entries.js ***!
@@ -24789,6 +24880,7 @@ const dynamic_1 = __webpack_require__(/*! ./dynamic */ "./node_modules/@abaplint
 const database_table_1 = __webpack_require__(/*! ./database_table */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/database_table.js");
 class SQLFrom {
     runSyntax(node, scope, filename) {
+        const ret = [];
         const fromList = node.findAllExpressions(Expressions.SQLFromSource);
         for (const from of fromList) {
             for (const d of from.findAllExpressions(Expressions.Dynamic)) {
@@ -24796,9 +24888,10 @@ class SQLFrom {
             }
             const dbtab = from.findFirstExpression(Expressions.DatabaseTable);
             if (dbtab !== undefined) {
-                new database_table_1.DatabaseTable().runSyntax(dbtab, scope, filename);
+                ret.push(new database_table_1.DatabaseTable().runSyntax(dbtab, scope, filename));
             }
         }
+        return ret;
     }
 }
 exports.SQLFrom = SQLFrom;
@@ -24806,6 +24899,34 @@ exports.SQLFrom = SQLFrom;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js":
+/*!***************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js ***!
+  \***************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLSource = void 0;
+const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
+class SQLSource {
+    runSyntax(node, scope, filename) {
+        for (const s of node.findAllExpressions(Expressions.Source)) {
+            return new source_1.Source().runSyntax(s, scope, filename);
+        }
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            return new source_1.Source().runSyntax(s, scope, filename);
+        }
+        return undefined;
+    }
+}
+exports.SQLSource = SQLSource;
+//# sourceMappingURL=sql_source.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/string_template.js":
 /*!********************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/string_template.js ***!
@@ -25403,6 +25524,7 @@ class ScopeData {
             extraLikeTypes: {},
             deferred: [],
             references: [],
+            sqlConversion: [],
         };
     }
     getData() {
@@ -46811,7 +46933,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.100.3";
+        return "2.101.0";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -53453,11 +53575,12 @@ ${indentation}    output = ${topTarget}.`;
                             body += indentation + structureName + "-" + b.concatTokens() + ".\n";
                         }
                         else if (b.get() instanceof Expressions.Source) {
-                            body += indentation + "APPEND " + b.concatTokens() + ` TO ${uniqueName}.\n`;
+                            // note: it wont work with APPEND for Hashed/Sorted Tables, so use INSERT,
+                            body += indentation + "INSERT " + b.concatTokens() + ` INTO TABLE ${uniqueName}.\n`;
                             skip = true;
                         }
                         else if (b.get() instanceof Expressions.ValueBodyLines) {
-                            body += indentation + "APPEND " + b.concatTokens() + ` TO ${uniqueName}.\n`;
+                            body += indentation + "INSERT " + b.concatTokens() + ` INTO TABLE ${uniqueName}.\n`;
                             skip = true;
                         }
                         else if (b.concatTokens() === ")") {
@@ -53466,7 +53589,7 @@ ${indentation}    output = ${topTarget}.`;
                                 added = true;
                             }
                             if (skip === false) {
-                                body += indentation + `APPEND ${structureName} TO ${uniqueName}.\n`;
+                                body += indentation + `INSERT ${structureName} INTO TABLE ${uniqueName}.\n`;
                             }
                         }
                         previous = b;
@@ -56151,7 +56274,16 @@ IF condition1.
   ...
 ELSEIF condition2.
   ...
-ENDIF.`,
+ENDIF.
+
+CASE variable.
+  WHEN value1.
+  ...
+  WHEN value2.
+    IF condition2.
+      ...
+    ENDIF.
+ENDCASE.`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
         };
     }
@@ -56896,6 +57028,7 @@ __exportStar(__webpack_require__(/*! ./smim_consistency */ "./node_modules/@abap
 __exportStar(__webpack_require__(/*! ./space_before_colon */ "./node_modules/@abaplint/core/build/src/rules/space_before_colon.js"), exports);
 __exportStar(__webpack_require__(/*! ./space_before_dot */ "./node_modules/@abaplint/core/build/src/rules/space_before_dot.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_escape_host_variables */ "./node_modules/@abaplint/core/build/src/rules/sql_escape_host_variables.js"), exports);
+__exportStar(__webpack_require__(/*! ./sql_value_conversion */ "./node_modules/@abaplint/core/build/src/rules/sql_value_conversion.js"), exports);
 __exportStar(__webpack_require__(/*! ./start_at_tab */ "./node_modules/@abaplint/core/build/src/rules/start_at_tab.js"), exports);
 __exportStar(__webpack_require__(/*! ./static_call_via_instance */ "./node_modules/@abaplint/core/build/src/rules/static_call_via_instance.js"), exports);
 __exportStar(__webpack_require__(/*! ./strict_sql */ "./node_modules/@abaplint/core/build/src/rules/strict_sql.js"), exports);
@@ -62480,6 +62613,9 @@ class ReduceStringTemplates extends _abap_rule_1.ABAPRule {
                     for (const second of source.findDirectExpressions(Expressions.StringTemplate)) {
                         issues.push(issue_1.Issue.atToken(file, second.getFirstToken(), "Nested string templates, reduce", this.getMetadata().key, this.conf.severity));
                     }
+                    if (ts.findDirectExpression(Expressions.StringTemplateFormatting)) {
+                        continue;
+                    }
                     for (const constant of source.findDirectExpressions(Expressions.Constant)) {
                         for (const constantString of constant.findDirectExpressions(Expressions.ConstantString)) {
                             issues.push(issue_1.Issue.atToken(file, constantString.getFirstToken(), "Constant string in text template, reduce", this.getMetadata().key, this.conf.severity));
@@ -64021,6 +64157,81 @@ exports.SQLEscapeHostVariables = SQLEscapeHostVariables;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/rules/sql_value_conversion.js":
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/rules/sql_value_conversion.js ***!
+  \*****************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLValueConversion = exports.SQLValueConversionConf = void 0;
+const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
+const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const syntax_1 = __webpack_require__(/*! ../abap/5_syntax/syntax */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/syntax.js");
+const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
+class SQLValueConversionConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.SQLValueConversionConf = SQLValueConversionConf;
+class SQLValueConversion {
+    constructor() {
+        this.conf = new SQLValueConversionConf();
+    }
+    getMetadata() {
+        return {
+            key: "sql_value_conversion",
+            title: "Implicit SQL Value Conversion",
+            shortDescription: `Ensure types match when selecting from database`,
+            extendedInformation: `
+* Integer to CHAR conversion
+* Integer to NUMC conversion
+* NUMC to Integer conversion
+* CHAR to Integer conversion
+* Source field longer than database field, CHAR -> CHAR
+* Source field longer than database field, NUMC -> NUMC`,
+            tags: [],
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    initialize(reg) {
+        this.reg = reg;
+        return this;
+    }
+    run(obj) {
+        if (!(obj instanceof _abap_object_1.ABAPObject) || obj instanceof objects_1.Interface) {
+            return [];
+        }
+        // messages defined in sql_compare.ts
+        const issues = this.traverse(new syntax_1.SyntaxLogic(this.reg, obj).run().spaghetti.getTop());
+        return issues;
+    }
+    traverse(node) {
+        const ret = [];
+        for (const r of node.getData().sqlConversion) {
+            const file = this.reg.getFileByName(node.getIdentifier().filename);
+            if (file === undefined) {
+                continue;
+            }
+            ret.push(issue_1.Issue.atToken(file, r.token, r.message, this.getMetadata().key, this.getConfig().severity));
+        }
+        for (const c of node.getChildren()) {
+            ret.push(...this.traverse(c));
+        }
+        return ret;
+    }
+}
+exports.SQLValueConversion = SQLValueConversion;
+//# sourceMappingURL=sql_value_conversion.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/rules/start_at_tab.js":
 /*!*********************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/rules/start_at_tab.js ***!
@@ -64770,13 +64981,15 @@ class TypesNaming extends _abap_rule_1.ABAPRule {
             if (stat.get() instanceof Statements.Type && nesting === 0) {
                 expr = stat.findFirstExpression(Expressions.NamespaceSimpleName);
             }
-            else if (stat.get() instanceof Statements.TypeBegin) {
+            else if (stat.get() instanceof Statements.TypeBegin
+                || stat.get() instanceof Statements.TypeEnumBegin) {
                 if (nesting === 0) {
                     expr = stat.findFirstExpression(Expressions.NamespaceSimpleName);
                 }
                 nesting = nesting + 1;
             }
-            else if (stat.get() instanceof Statements.TypeEnd) {
+            else if (stat.get() instanceof Statements.TypeEnd
+                || stat.get() instanceof Statements.TypeEnumEnd) {
                 nesting = nesting - 1;
                 continue;
             }
@@ -70392,8 +70605,9 @@ const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/tr
 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");
+const transpile_types_1 = __webpack_require__(/*! ../transpile_types */ "./node_modules/@abaplint/transpiler/build/src/transpile_types.js");
 class SQLCondTranspiler {
-    transpile(node, traversal) {
+    transpile(node, traversal, table) {
         var _a;
         let ret = "";
         for (const c of node.getChildren()) {
@@ -70416,11 +70630,11 @@ class SQLCondTranspiler {
                     ret += this.sqlIn(c, traversal);
                 }
                 else {
-                    ret += this.basicCondition(c, traversal, traversal.getFilename());
+                    ret += this.basicCondition(c, traversal, traversal.getFilename(), table);
                 }
             }
             else if (c instanceof abaplint.Nodes.ExpressionNode) {
-                ret += " " + this.transpile(c, traversal).getCode();
+                ret += " " + this.transpile(c, traversal, table).getCode();
             }
             else {
                 ret += " " + c.concatTokens();
@@ -70439,10 +70653,10 @@ class SQLCondTranspiler {
         const ret = `" + abap.expandIN("${fieldName.concatTokens()}", ${source.concatTokens()}) + "`;
         return ret;
     }
-    basicCondition(c, traversal, filename) {
+    basicCondition(c, traversal, filename, table) {
         let ret = "";
         if (c.getChildren().length !== 3) {
-            return this.basicConditionNew(c, traversal, filename);
+            return this.basicConditionNew(c, traversal, filename, table);
         }
         let fieldName = undefined;
         const fieldNameExpression = c.findDirectExpression(abaplint.Expressions.SQLFieldName);
@@ -70453,7 +70667,7 @@ class SQLCondTranspiler {
         const source = c.findDirectExpression(abaplint.Expressions.SQLSource);
         if (fieldName && source && operator === undefined && c.findDirectTokenByText("LIKE")) {
             ret += fieldName + " LIKE ";
-            ret += this.sqlSource(source, traversal, filename);
+            ret += this.sqlSource(source, traversal, filename, table);
             return ret;
         }
         if (fieldName === undefined || operator === undefined || source === undefined) {
@@ -70467,10 +70681,10 @@ class SQLCondTranspiler {
             op = "<>";
         }
         ret += fieldName + " " + op + " ";
-        ret += this.sqlSource(source, traversal, filename);
+        ret += this.sqlSource(source, traversal, filename, table);
         return ret;
     }
-    sqlSource(source, traversal, filename) {
+    sqlSource(source, traversal, filename, table) {
         let ret = "";
         const simple = source.findDirectExpression(abaplint.Expressions.SimpleSource3);
         const alias = source.findDirectExpression(abaplint.Expressions.SQLAliasField);
@@ -70496,7 +70710,12 @@ class SQLCondTranspiler {
         }
         else {
             const concat = source.concatTokens();
-            if (concat.startsWith("`")) {
+            const conversionField = traversal.isSQLConversion(source.getFirstToken());
+            if (conversionField) {
+                const field = (table === null || table === void 0 ? void 0 : table.parseType(traversal.reg)).getComponentByName(conversionField);
+                ret += "'\" + " + new transpile_types_1.TranspileTypes().toType(field) + ".set(" + concat + ").get() + \"'";
+            }
+            else if (concat.startsWith("`")) {
                 ret += "'" + concat.substring(1, concat.length - 1) + "'";
             }
             else {
@@ -70505,7 +70724,7 @@ class SQLCondTranspiler {
         }
         return ret;
     }
-    basicConditionNew(node, traversal, filename) {
+    basicConditionNew(node, traversal, filename, table) {
         let ret = "";
         for (const child of node.getChildren()) {
             if (ret !== "") {
@@ -70517,7 +70736,7 @@ class SQLCondTranspiler {
             }
             else if (child.get() instanceof abaplint.Expressions.SQLSource
                 && child instanceof abaplint.Nodes.ExpressionNode) {
-                ret += this.sqlSource(child, traversal, filename);
+                ret += this.sqlSource(child, traversal, filename, table);
             }
             else {
                 ret += child.concatTokens();
@@ -70625,13 +70844,19 @@ class SQLFromTranspiler {
             }
             else {
                 if (c.findFirstExpression(abaplint.Expressions.Dynamic)) {
-                    chunk.appendString(c.concatTokens() + " ");
+                    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() + "' ");
+                    chunk.appendString("\\\"" + c.concatTokens() + "\\\" ");
                 }
                 else {
-                    chunk.appendString(c.concatTokens() + " ");
+                    const concat = c.concatTokens();
+                    chunk.appendString(concat + " ");
                 }
             }
         }
@@ -70664,7 +70889,11 @@ class SQLFromSourceTranspiler {
                 chunk.appendString(c.concatTokens() + " ");
             }
             else if (c.get() instanceof abaplint.Expressions.DatabaseTable && c.concatTokens().includes("/")) {
-                chunk.appendString("'" + c.concatTokens() + "' ");
+                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 {
                 chunk.appendString(c.concatTokens() + " ");
@@ -74752,7 +74981,7 @@ class InsertDatabaseTranspiler {
             const tvalues = traversal.traverse(fromTable);
             options.push(`"table": ` + tvalues.getCode());
         }
-        return new chunk_1.Chunk(`await abap.statements.insertDatabase(${table.getCode()}, {${options.join(", ")}});`);
+        return new chunk_1.Chunk(`await abap.statements.insertDatabase(${table.getCode().toLowerCase()}, {${options.join(", ")}});`);
     }
 }
 exports.InsertDatabaseTranspiler = InsertDatabaseTranspiler;
@@ -76228,6 +76457,7 @@ class SelectTranspiler {
         if (from) {
             select += new sql_from_1.SQLFromTranspiler().transpile(from, traversal).getCode();
         }
+        const { table, keys } = this.findTable(node, traversal);
         let where = undefined;
         for (const sqlCond of node.findAllExpressions(abaplint.Expressions.SQLCond)) {
             if (this.isWhereExpression(node, sqlCond)) {
@@ -76235,7 +76465,7 @@ class SelectTranspiler {
             }
         }
         if (where) {
-            select += "WHERE " + traversal.traverse(where).getCode() + " ";
+            select += "WHERE " + new expressions_1.SQLCondTranspiler().transpile(where, traversal, table).getCode() + " ";
         }
         const upTo = node.findFirstExpression(abaplint.Expressions.SQLUpTo);
         if (upTo) {
@@ -76271,7 +76501,6 @@ class SelectTranspiler {
             runtimeOptions = `, {` + runtimeOptionsList.join(", ") + `}`;
         }
         let extra = "";
-        const keys = this.findKeys(node, traversal);
         if (keys.length > 0) {
             extra = `, primaryKey: ${JSON.stringify(keys)}`;
         }
@@ -76304,16 +76533,17 @@ class SelectTranspiler {
             return new chunk_1.Chunk().append(`await abap.statements.select(${target}, {select: "${select.trim()}"${extra}}${runtimeOptions});`, node, traversal);
         }
     }
-    findKeys(node, traversal) {
+    findTable(node, traversal) {
         let keys = [];
+        let tabl = undefined;
         const from = node.findAllExpressions(abaplint.Expressions.SQLFromSource).map(e => e.concatTokens());
         if (from.length === 1) {
-            const tabl = traversal.findTable(from[0]);
+            tabl = traversal.findTable(from[0]);
             if (tabl) {
                 keys = tabl.listKeys(traversal.reg).map(k => k.toLowerCase());
             }
         }
-        return keys;
+        return { table: tabl, keys };
     }
     isWhereExpression(node, expression) {
         // check if previous token before sqlCond is "WHERE". It could also be "ON" in case of join condition
@@ -77097,7 +77327,7 @@ class UpdateDatabaseTranspiler {
             for (const set of sets) {
                 const name = (_a = set.findDirectExpression(abaplint.Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
                 const source = traversal.traverse(set.findDirectExpression(abaplint.Expressions.SQLSource));
-                s.push("\"'" + name + "' = '\" + " + source.getCode() + ".get() + \"'\"");
+                s.push("\"\\\"" + name + "\\\" = '\" + " + source.getCode() + ".get() + \"'\"");
             }
             options.push(`"set": [${s.join(",")}]`);
         }
@@ -78798,6 +79028,15 @@ class Traversal {
         }
         return false;
     }
+    isSQLConversion(token) {
+        const scope = this.findCurrentScopeByToken(token);
+        for (const s of (scope === null || scope === void 0 ? void 0 : scope.getData().sqlConversion) || []) {
+            if (s.token.getStart().equals(token.getStart())) {
+                return s.fieldName;
+            }
+        }
+        return undefined;
+    }
     findMethodReference(token, scope) {
         var _a, _b;
         let candidate = undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.4",
+  "version": "2.7.6",
   "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.4",
+    "@abaplint/transpiler": "^2.7.6",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
     "@types/node": "^20.2.1",
-    "@abaplint/core": "^2.100.3",
+    "@abaplint/core": "^2.101.0",
     "progress": "^2.0.3",
     "webpack": "^5.83.1",
     "webpack-cli": "^5.1.1",