@abaplint/transpiler-cli 2.7.63 → 2.7.65

package/build/bundle.js

@@ -20477,6 +20477,7 @@ var ScopeType;
     ScopeType["FunctionModule"] = "function";
     ScopeType["Method"] = "method";
     ScopeType["MethodInstance"] = "method_instance";
+    ScopeType["MethodDefinition"] = "method_definition";
     ScopeType["For"] = "for";
     ScopeType["Let"] = "let";
     ScopeType["OpenSQL"] = "open_sql";
@@ -28161,7 +28162,7 @@ class IncludeType {
             const components = [];
             for (const c of ityp.getComponents()) {
                 if (c.name === as) {
-                    components.push(c);
+                    components.push(Object.assign(Object.assign({}, c), { renamingSuffix: suffix }));
                     continue;
                 }
                 components.push({
@@ -28636,11 +28637,12 @@ class MethodImplementation {
         if (methodDefinition === undefined) {
             throw new Error("Method definition \"" + methodName + "\" not found");
         }
+        const start = node.getFirstToken().getStart();
         if (methodDefinition.isStatic() === false) {
-            scope.push(_scope_type_1.ScopeType.MethodInstance, methodName, node.getFirstToken().getStart(), filename);
+            scope.push(_scope_type_1.ScopeType.MethodInstance, methodName, start, filename);
             scope.addList(classDefinition.getAttributes().getInstance());
         }
-        scope.push(_scope_type_1.ScopeType.Method, methodName, node.getFirstToken().getStart(), filename);
+        scope.push(_scope_type_1.ScopeType.Method, methodName, start, filename);
         scope.addReference(methodToken, methodDefinition, _reference_1.ReferenceType.MethodImplementationReference, filename);
         scope.addList(methodDefinition.getParameters().getAll());
         for (const i of helper.findInterfaces(classDefinition)) {
@@ -35840,6 +35842,7 @@ const method_param_1 = __webpack_require__(/*! ../5_syntax/expressions/method_pa
 const _object_oriented_1 = __webpack_require__(/*! ../5_syntax/_object_oriented */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_object_oriented.js");
 const _reference_1 = __webpack_require__(/*! ../5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const identifier_1 = __webpack_require__(/*! ../1_lexer/tokens/identifier */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/identifier.js");
+const _scope_type_1 = __webpack_require__(/*! ../5_syntax/_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
 // todo:
 // this.exceptions = [];
 // also consider RAISING vs EXCEPTIONS
@@ -35857,7 +35860,11 @@ class MethodParameters {
         this.preferred = undefined;
         this.exceptions = [];
         this.filename = filename;
-        this.parse(node, scope, filename);
+        // need the scope for LIKE typing inside method parameters
+        const parentName = scope.getName();
+        scope.push(_scope_type_1.ScopeType.MethodDefinition, "method definition", node.getStart(), filename);
+        this.parse(node, scope, filename, parentName);
+        scope.pop(node.getEnd());
     }
     getFilename() {
         return this.filename;
@@ -35932,7 +35939,7 @@ class MethodParameters {
         return this.defaults[parameter.toUpperCase()];
     }
     ///////////////////
-    parse(node, scope, filename) {
+    parse(node, scope, filename, parentName) {
         var _a, _b;
         const handler = node.findFirstExpression(Expressions.EventHandler);
         if (handler) {
@@ -35992,9 +35999,9 @@ class MethodParameters {
         if (returning) {
             this.returning = new method_def_returning_1.MethodDefReturning().runSyntax(returning, scope, this.filename, ["returning" /* IdentifierMeta.MethodReturning */]);
         }
-        this.workaroundRAP(node, scope, filename);
+        this.workaroundRAP(node, scope, filename, parentName);
     }
-    workaroundRAP(node, scope, filename) {
+    workaroundRAP(node, _scope, filename, parentName) {
         const resultName = node.findExpressionAfterToken("RESULT");
         const isRap = node.findExpressionAfterToken("IMPORTING");
         if (isRap) {
@@ -36019,7 +36026,7 @@ class MethodParameters {
             this.importing.push(new _typed_identifier_1.TypedIdentifier(token, filename, new basic_1.VoidType("RapMethodParameter"), ["exporting" /* IdentifierMeta.MethodExporting */]));
         }
         // its some kind of magic
-        if (scope.getName().toUpperCase() === "CL_ABAP_BEHAVIOR_SAVER") {
+        if (parentName.toUpperCase() === "CL_ABAP_BEHAVIOR_SAVER") {
             const tempChanging = this.changing.map(c => new _typed_identifier_1.TypedIdentifier(c.getToken(), filename, new basic_1.VoidType("RapMethodParameter"), c.getMeta()));
             while (this.changing.length > 0) {
                 this.changing.shift();
@@ -36046,7 +36053,9 @@ class MethodParameters {
             else if (meta.includes("importing" /* IdentifierMeta.MethodImporting */)) {
                 extraMeta.push("read_only" /* IdentifierMeta.ReadOnly */);
             }
-            target.push(new method_param_1.MethodParam().runSyntax(p, scope, this.filename, [...meta, ...extraMeta]));
+            const id = new method_param_1.MethodParam().runSyntax(p, scope, this.filename, [...meta, ...extraMeta]);
+            scope.addIdentifier(id);
+            target.push(id);
             if (opt.getLastToken().getStr().toUpperCase() === "OPTIONAL") {
                 const name = target[target.length - 1].getName().toUpperCase();
                 this.optional.push(name);
@@ -38991,7 +39000,7 @@ class LSPLookup {
             const hover = "Type Definition, " + cursor.token.getStr() + "\n\n" + this.dumpType(type);
             return { hover, definition: found, definitionId: type, scope: bottomScope };
         }
-        const method = this.findMethodDefinition(cursor, bottomScope);
+        const method = this.findMethodDefinition(cursor, bottomScope.getParent());
         if (method !== undefined && method.getStart().equals(cursor.token.getStart())) {
             const found = _lsp_utils_1.LSPUtils.identiferToLocation(method);
             const hover = "Method Definition \"" + method.getName() + "\"";
@@ -39208,6 +39217,9 @@ class LSPLookup {
     }
     static findMethodDefinition(found, scope) {
         var _a, _b, _c, _d;
+        if (scope === undefined) {
+            return undefined;
+        }
         if (scope.getIdentifier().stype !== _scope_type_1.ScopeType.ClassDefinition
             || !(found.snode.get() instanceof Statements.MethodDef)) {
             return undefined;
@@ -47610,7 +47622,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.14";
+        return "2.102.15";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -56338,6 +56350,7 @@ class FunctionModuleRecommendationsConf extends _basic_rule_config_1.BasicRuleCo
             { name: "POPUP_TO_CONFIRM_STEP", replace: "use POPUP_TO_CONFIRM" },
             { name: "POPUP_TO_DECIDE", replace: "use POPUP_TO_CONFIRM" },
             { name: "POPUP_TO_GET_VALUE", replace: "use POPUP_GET_VALUES" },
+            { name: "QF05_RANDOM_INTEGER", replace: "use CL_ABAP_RANDOM_INT" },
             { name: "REUSE_ALV_GRID_DISPLAY", replace: "use CL_SALV_TABLE=>FACTORY or CL_GUI_ALV_GRID" },
             { name: "ROUND", replace: "use built in function: round()" },
             { name: "SAPGUI_PROGRESS_INDICATOR", replace: "use CL_PROGRESS_INDICATOR" },
@@ -79937,8 +79950,12 @@ class TranspileTypes {
         else if (type instanceof abaplint.BasicTypes.StructureType) {
             resolved = "Structure";
             const list = [];
+            const renamingSuffix = {};
             for (const c of type.getComponents()) {
                 list.push(`"` + c.name.toLowerCase() + `": ` + this.toType(c.type));
+                if (c.renamingSuffix) {
+                    renamingSuffix[c.name.toLowerCase()] = c.renamingSuffix;
+                }
             }
             extra = "{" + list.join(", ") + "}";
             if (type.getQualifiedName() !== undefined) {
@@ -79950,6 +79967,10 @@ class TranspileTypes {
             if (type.getDDICName() !== undefined) {
                 extra += ", \"" + type.getQualifiedName() + "\"";
             }
+            else {
+                extra += ", undefined";
+            }
+            extra += ", " + JSON.stringify(renamingSuffix);
         }
         else if (type instanceof abaplint.BasicTypes.CLikeType
             || type instanceof abaplint.BasicTypes.CGenericType

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.63",
+  "version": "2.7.65",
   "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.63",
+    "@abaplint/transpiler": "^2.7.65",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
-    "@types/node": "^20.4.7",
-    "@abaplint/core": "^2.102.14",
+    "@types/node": "^20.4.8",
+    "@abaplint/core": "^2.102.15",
     "progress": "^2.0.3",
     "webpack": "^5.88.2",
     "webpack-cli": "^5.1.4",