@abaplint/transpiler-cli 2.7.125 → 2.7.127

package/build/bundle.js

@@ -572,6 +572,9 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Token = void 0;
 const position_1 = __webpack_require__(/*! ../../../position */ "./node_modules/@abaplint/core/build/src/position.js");
 class Token {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.str}`;
+    }
     constructor(start, str) {
         this.start = start;
         this.str = str;
@@ -4303,6 +4306,7 @@ __exportStar(__webpack_require__(/*! ./sql_order_by */ "./node_modules/@abaplint
 __exportStar(__webpack_require__(/*! ./sql_path */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_path.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_source_simple */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source_simple.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source.js"), exports);
+__exportStar(__webpack_require__(/*! ./sql_source_no_space */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source_no_space.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_target */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_target.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_up_to */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_up_to.js"), exports);
 __exportStar(__webpack_require__(/*! ./string_template_formatting */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/string_template_formatting.js"), exports);
@@ -6797,7 +6801,7 @@ const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@a
 class SQLIn extends combi_1.Expression {
     getRunnable() {
         const val = new _1.SQLSource();
-        const short = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.At), _1.SimpleSource3);
+        const short = new _1.SQLSourceNoSpace();
         const listOld = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeft), (0, combi_1.alt)((0, combi_1.ver)(version_1.Version.v740sp05, short), val), (0, combi_1.starPrio)((0, combi_1.seq)(",", val)), (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.ParenRight), (0, combi_1.tok)(tokens_1.ParenRightW), (0, combi_1.tok)(tokens_1.WParenRightW)));
         const listNew = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), val, (0, combi_1.starPrio)((0, combi_1.seq)(",", (0, combi_1.altPrio)(short, val))), (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenRight), (0, combi_1.tok)(tokens_1.WParenRightW)));
         const list = (0, combi_1.alt)(listOld, (0, combi_1.ver)(version_1.Version.v740sp02, listNew)); // version is a guess, https://github.com/abaplint/abaplint/issues/2530
@@ -6981,6 +6985,32 @@ exports.SQLSource = SQLSource;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source_no_space.js":
+/*!****************************************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source_no_space.js ***!
+  \****************************************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLSourceNoSpace = void 0;
+const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
+const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+class SQLSourceNoSpace extends combi_1.Expression {
+    getRunnable() {
+        const paren = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), _1.Source, (0, combi_1.tok)(tokens_1.WParenRightW));
+        const at = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.At), (0, combi_1.altPrio)(_1.SimpleSource3, paren)));
+        return (0, combi_1.alt)(_1.SQLAliasField, _1.SimpleSource3, at);
+    }
+}
+exports.SQLSourceNoSpace = SQLSourceNoSpace;
+//# sourceMappingURL=sql_source_no_space.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source_simple.js":
 /*!**************************************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/sql_source_simple.js ***!
@@ -19147,7 +19177,7 @@ BuiltIn.methods = [
     {
         name: "LINE_INDEX",
         mandatory: {
-            "val": basic_1.CLikeType.get(),
+            "val": new basic_1.AnyType(),
         },
         return: basic_1.IntegerType.get(),
         version: version_1.Version.v740sp02,
@@ -20888,9 +20918,10 @@ class TypeUtils {
                 }
                 return true;
             }
-            else if (target instanceof basic_1.CharacterType) {
-                return false;
+            else if (target instanceof basic_1.VoidType || target instanceof basic_1.AnyType) {
+                return true;
             }
+            return false;
         }
         else if (source instanceof basic_1.Integer8Type) {
             if (target instanceof basic_1.IntegerType || target instanceof basic_1.StringType) {
@@ -25265,12 +25296,16 @@ exports.SQLCompare = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const nodes_1 = __webpack_require__(/*! ../../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.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.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
         for (const s of node.findAllExpressions(Expressions.SQLSource)) {
             for (const child of s.getChildren()) {
                 if (child instanceof nodes_1.ExpressionNode) {
@@ -26075,6 +26110,9 @@ class ScopeData {
     }
 }
 class SpaghettiScopeNode extends ScopeData {
+    [Symbol.for("debug.description")]() {
+        return `SpaghettiSN ${this.identifier.sname} ${this.identifier.stype}`;
+    }
     constructor(identifier, parent) {
         super();
         this.identifier = identifier;
@@ -33659,6 +33697,9 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.TypedIdentifier = void 0;
 const _identifier_1 = __webpack_require__(/*! ../4_file_information/_identifier */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/_identifier.js");
 class TypedIdentifier extends _identifier_1.Identifier {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.getName()}:${this.getType().constructor.name}`;
+    }
     static from(id, type, meta) {
         return new TypedIdentifier(id.getToken(), id.getFilename(), type, meta);
     }
@@ -41535,6 +41576,9 @@ const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node
 const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
 const abap_parser_1 = __webpack_require__(/*! ../abap/abap_parser */ "./node_modules/@abaplint/core/build/src/abap/abap_parser.js");
 class ABAPObject extends _abstract_object_1.AbstractObject {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.getName()}`;
+    }
     constructor(name) {
         super(name);
         this.parsed = [];
@@ -49047,7 +49091,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.61";
+        return "2.102.64";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -54232,6 +54276,7 @@ Make sure to test the downported code, it might not always be completely correct
         };
         const candidates = [high.findAllExpressionsRecursive(Expressions.SQLTarget),
             high.findAllExpressionsRecursive(Expressions.SQLSource),
+            high.findAllExpressionsRecursive(Expressions.SQLSourceNoSpace),
             high.findAllExpressionsRecursive(Expressions.SQLSourceSimple)].flat();
         for (const c of candidates.reverse()) {
             if (c.getFirstToken() instanceof tokens_1.WAt
@@ -78664,9 +78709,18 @@ class MethodImplementationTranspiler {
                     && type.isGeneric() === false) {
                     after += `let ${varName} = ${unique}?.${varName};\n`;
                     if (identifier.getType().getQualifiedName() !== undefined && identifier.getType().getQualifiedName() !== "") {
-                        after += `if (${varName}.getQualifiedName === undefined || ${varName}.getQualifiedName() !== "${(_a = identifier.getType().getQualifiedName()) === null || _a === void 0 ? void 0 : _a.toUpperCase()}") { ${varName} = undefined; }\n`;
+                        after += `if (${varName}?.getQualifiedName === undefined || ${varName}.getQualifiedName() !== "${(_a = identifier.getType().getQualifiedName()) === null || _a === void 0 ? void 0 : _a.toUpperCase()}") { ${varName} = undefined; }\n`;
                     }
                     after += `if (${varName} === undefined) { ${varName} = ${new transpile_types_1.TranspileTypes().toType(identifier.getType())}.set(${unique}.${varName}); }\n`;
+                }
+                else if (identifier.getMeta().includes("importing" /* abaplint.IdentifierMeta.MethodImporting */)
+                    && type.isGeneric() === true) {
+                    if (isOptional === true) {
+                        after += `let ${varName} = ${unique}?.${varName} || ${new transpile_types_1.TranspileTypes().toType(identifier.getType())};\n`;
+                    }
+                    else {
+                        after += `let ${varName} = ${unique}?.${varName};\n`;
+                    }
                     // end new
                 }
                 else if (identifier.getMeta().includes("importing" /* abaplint.IdentifierMeta.MethodImporting */)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.125",
+  "version": "2.7.127",
   "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.125",
+    "@abaplint/transpiler": "^2.7.127",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.6",
-    "@types/node": "^20.8.8",
-    "@abaplint/core": "^2.102.61",
+    "@types/node": "^20.8.9",
+    "@abaplint/core": "^2.102.64",
     "progress": "^2.0.3",
     "webpack": "^5.89.0",
     "webpack-cli": "^5.1.4",