@abaplint/cli 2.113.6 → 2.113.8

package/build/cli.js

@@ -21808,6 +21808,7 @@ exports.ObjectOriented = void 0;
 const Statements = __webpack_require__(/*! ../2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
 const Expressions = __webpack_require__(/*! ../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const visibility_1 = __webpack_require__(/*! ../4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
+const types_1 = __webpack_require__(/*! ../types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 // todo, think some of the public methods can be made private
 // todo: changet this class to static? for performance
 class ObjectOriented {
@@ -21881,6 +21882,18 @@ class ObjectOriented {
         }
         return undefined;
     }
+    methodReferenceExtras(foundDef, ooName) {
+        if (foundDef === undefined) {
+            return {
+                ooName: ooName,
+                ooType: undefined,
+            };
+        }
+        return {
+            ooName: foundDef === null || foundDef === void 0 ? void 0 : foundDef.getName(),
+            ooType: foundDef instanceof types_1.ClassDefinition ? "CLAS" : "INTF",
+        };
+    }
     findClassName(node) {
         if (!(node.get() instanceof Statements.ClassImplementation
             || node.get() instanceof Statements.ClassDefinition)) {
@@ -25718,7 +25731,6 @@ const method_call_param_1 = __webpack_require__(/*! ./method_call_param */ "./no
 const _reference_1 = __webpack_require__(/*! ../_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
 const component_name_1 = __webpack_require__(/*! ./component_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_name.js");
 const attribute_name_1 = __webpack_require__(/*! ./attribute_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/attribute_name.js");
-const class_definition_1 = __webpack_require__(/*! ../../types/class_definition */ "./node_modules/@abaplint/core/build/src/abap/types/class_definition.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 class MethodCallChain {
     runSyntax(node, input, targetType) {
@@ -25761,10 +25773,8 @@ class MethodCallChain {
                         input.issues.push((0, _syntax_input_1.syntaxIssue)(input, methodToken, message));
                         return new basic_1.VoidType(_syntax_input_1.CheckSyntaxKey);
                     }
-                    const extra = {
-                        ooName: foundDef === null || foundDef === void 0 ? void 0 : foundDef.getName(),
-                        ooType: foundDef instanceof class_definition_1.ClassDefinition ? "CLAS" : "INTF"
-                    };
+                    const voidedName = context instanceof basic_1.VoidType ? context.getVoided() : undefined;
+                    const extra = helper.methodReferenceExtras(foundDef, className || voidedName);
                     input.scope.addReference(methodToken, method, _reference_1.ReferenceType.MethodReference, input.filename, extra);
                 }
                 if (methodName === null || methodName === void 0 ? void 0 : methodName.includes("~")) {
@@ -26349,7 +26359,6 @@ const source_field_1 = __webpack_require__(/*! ./source_field */ "./node_modules
 const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "./node_modules/@abaplint/core/build/src/abap/1_lexer/tokens/index.js");
 const attribute_name_1 = __webpack_require__(/*! ./attribute_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/attribute_name.js");
 const component_name_1 = __webpack_require__(/*! ./component_name */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/component_name.js");
-const types_1 = __webpack_require__(/*! ../../types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@abaplint/core/build/src/version.js");
 const _syntax_input_1 = __webpack_require__(/*! ../_syntax_input */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_syntax_input.js");
 const assert_error_1 = __webpack_require__(/*! ../assert_error */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/assert_error.js");
@@ -26429,10 +26438,7 @@ class MethodSource {
                     return new basic_1.VoidType(_syntax_input_1.CheckSyntaxKey);
                 }
                 else if (method) {
-                    const extra = {
-                        ooName: foundDef === null || foundDef === void 0 ? void 0 : foundDef.getName(),
-                        ooType: foundDef instanceof types_1.ClassDefinition ? "CLAS" : "INTF"
-                    };
+                    const extra = helper.methodReferenceExtras(foundDef, className);
                     input.scope.addReference(methodToken, method, _reference_1.ReferenceType.MethodReference, input.filename, extra);
                     context = method;
                 }
@@ -52708,7 +52714,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.6";
+        return "2.113.8";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -64888,7 +64894,7 @@ https://docs.abapopenchecks.org/checks/16/`,
                 const position = new position_1.Position(i + 1, column);
                 // merge punc into previous row
                 let rowContent = rows[i].trim();
-                // if reported row contains a paranthesis, prefix with space if needed
+                // if reported row contains parentheses, prefix with space if needed
                 if (rowContent.length > 1 && !rows[i - 1].endsWith(" ") && !rows[i - 1].endsWith(" \r")) {
                     rowContent = " " + rowContent;
                 }
@@ -64898,7 +64904,10 @@ https://docs.abapopenchecks.org/checks/16/`,
                 }
                 const startPos = new position_1.Position(i, rows[i - 1].length + 1 + offset);
                 const endPos = new position_1.Position(i + 1, rows[i].length + 1);
-                const fix = edit_helper_1.EditHelper.replaceRange(file, startPos, endPos, rowContent);
+                let fix = edit_helper_1.EditHelper.replaceRange(file, startPos, endPos, rowContent);
+                if (rows[i - 1] === undefined || rows[i - 1].indexOf("*") === 0 || rows[i - 1].includes(`"`)) {
+                    fix = undefined;
+                }
                 const issue = issue_1.Issue.atPosition(file, position, this.getMessage(), this.getMetadata().key, this.conf.severity, fix);
                 issues.push(issue);
             }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.6",
+    "version": "2.113.8",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.113.6",
+        "@abaplint/core": "^2.113.8",
         "@types/chai": "^4.3.19",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",
@@ -47,14 +47,14 @@
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "chalk": "^5.3.0",
-        "eslint": "^9.9.1",
+        "eslint": "^9.10.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
         "memfs": "^4.11.1",
         "minimist": "^1.2.8",
         "mocha": "^10.7.3",
         "progress": "^2.0.3",
-        "typescript": "^5.5.4",
+        "typescript": "^5.6.2",
         "webpack": "^5.94.0",
         "webpack-cli": "^5.1.4",
         "xml-js": "^1.6.11"