@abaplint/cli 2.102.41 → 2.102.43

package/build/cli.js

@@ -36801,7 +36801,7 @@ class MethodDefinition extends _identifier_1.Identifier {
             this.exceptions.push(name);
         }
         this.visibility = visibility;
-        this.parameters = new method_parameters_1.MethodParameters(node, this.filename, scope);
+        this.parameters = new method_parameters_1.MethodParameters(node, this.filename, scope, this.abstract);
     }
     getVisibility() {
         return this.visibility;
@@ -36959,7 +36959,7 @@ const _scope_type_1 = __webpack_require__(/*! ../5_syntax/_scope_type */ "./node
 // this.exceptions = [];
 // also consider RAISING vs EXCEPTIONS
 class MethodParameters {
-    constructor(node, filename, scope) {
+    constructor(node, filename, scope, abstractMethod) {
         if (!(node.get() instanceof method_def_1.MethodDef)) {
             throw new Error("MethodDefinition, expected MethodDef as part of input node");
         }
@@ -36975,7 +36975,7 @@ class MethodParameters {
         // 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);
+        this.parse(node, scope, filename, parentName, abstractMethod);
         scope.pop(node.getEnd());
     }
     getFilename() {
@@ -37051,7 +37051,7 @@ class MethodParameters {
         return this.defaults[parameter.toUpperCase()];
     }
     ///////////////////
-    parse(node, scope, filename, parentName) {
+    parse(node, scope, filename, parentName, abstractMethod) {
         var _a, _b;
         const handler = node.findFirstExpression(Expressions.EventHandler);
         if (handler) {
@@ -37091,7 +37091,7 @@ class MethodParameters {
         }
         const importing = node.findFirstExpression(Expressions.MethodDefImporting);
         if (importing) {
-            this.add(this.importing, importing, scope, ["importing" /* IdentifierMeta.MethodImporting */]);
+            this.add(this.importing, importing, scope, ["importing" /* IdentifierMeta.MethodImporting */], abstractMethod);
             if (importing.concatTokens().toUpperCase().includes(" PREFERRED PARAMETER")) {
                 this.preferred = importing.getLastToken().getStr().toUpperCase();
                 if (this.preferred.startsWith("!")) {
@@ -37101,11 +37101,11 @@ class MethodParameters {
         }
         const exporting = node.findFirstExpression(Expressions.MethodDefExporting);
         if (exporting) {
-            this.add(this.exporting, exporting, scope, ["exporting" /* IdentifierMeta.MethodExporting */]);
+            this.add(this.exporting, exporting, scope, ["exporting" /* IdentifierMeta.MethodExporting */], abstractMethod);
         }
         const changing = node.findFirstExpression(Expressions.MethodDefChanging);
         if (changing) {
-            this.add(this.changing, changing, scope, ["changing" /* IdentifierMeta.MethodChanging */]);
+            this.add(this.changing, changing, scope, ["changing" /* IdentifierMeta.MethodChanging */], abstractMethod);
         }
         const returning = node.findFirstExpression(Expressions.MethodDefReturning);
         if (returning) {
@@ -37151,7 +37151,7 @@ class MethodParameters {
             this.importing.push(...tempImporting);
         }
     }
-    add(target, source, scope, meta) {
+    add(target, source, scope, meta, abstractMethod) {
         var _a;
         for (const opt of source.findAllExpressions(Expressions.MethodParamOptional)) {
             const p = opt.findDirectExpression(Expressions.MethodParam);
@@ -37165,6 +37165,9 @@ class MethodParameters {
             else if (meta.includes("importing" /* IdentifierMeta.MethodImporting */)) {
                 extraMeta.push("read_only" /* IdentifierMeta.ReadOnly */);
             }
+            if (abstractMethod === true) {
+                extraMeta.push("abstract" /* IdentifierMeta.Abstract */);
+            }
             const id = new method_param_1.MethodParam().runSyntax(p, scope, this.filename, [...meta, ...extraMeta]);
             scope.addIdentifier(id);
             target.push(id);
@@ -48893,7 +48896,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.41";
+        return "2.102.43";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -51561,7 +51564,8 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
             if (statement.get() instanceof _statement_1.Comment) {
                 continue;
             }
-            else if (statement.get() instanceof Statements.EndIf) {
+            else if (statement.get() instanceof Statements.EndIf
+                || statement.get() instanceof Statements.EndTestSeam) {
                 continue;
             }
             else {
@@ -51588,7 +51592,8 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
                     return true;
                 }
             }
-            else if (statement.get() instanceof Statements.EndIf) {
+            else if (statement.get() instanceof Statements.EndIf
+                || statement.get() instanceof Statements.EndTestSeam) {
                 continue;
             }
             else {
@@ -61182,7 +61187,9 @@ class MethodLength {
             key: "method_length",
             title: "Method/Form Length",
             shortDescription: `Checks relating to method/form length.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-methods-small`,
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#keep-methods-small
+
+Abstract methods without statements are considered okay.`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
         };
     }
@@ -61227,6 +61234,9 @@ class MethodLength {
                 continue;
             }
             if (s.count === 0 && this.conf.errorWhenEmpty === true) {
+                if (this.isAbstract(s)) {
+                    continue;
+                }
                 const issue = issue_1.Issue.atPosition(s.file, s.pos, this.getDescription(IssueType.EmptyMethod, "0", type), this.getMetadata().key, this.conf.severity);
                 issues.push(issue);
                 continue;
@@ -61239,6 +61249,10 @@ class MethodLength {
         }
         return issues;
     }
+    isAbstract(result) {
+        const cdef = result.file.getInfo().getClassDefinitionByName(result.className);
+        return (cdef === null || cdef === void 0 ? void 0 : cdef.isAbstract) === true;
+    }
 }
 exports.MethodLength = MethodLength;
 //# sourceMappingURL=method_length.js.map
@@ -68809,6 +68823,8 @@ class UnusedVariablesConf extends _basic_rule_config_1.BasicRuleConfig {
          * @uniqueItems true
         */
         this.skipNames = [];
+        /** skip parameters from abstract methods */
+        this.skipAbstract = false;
     }
 }
 exports.UnusedVariablesConf = UnusedVariablesConf;
@@ -68951,6 +68967,9 @@ Errors found in INCLUDES are reported for the main program.`,
                     && this.conf.skipNames.some((a) => a.toUpperCase() === name)) {
                     continue;
                 }
+                else if (this.conf.skipAbstract === true && meta.includes("abstract" /* IdentifierMeta.Abstract */)) {
+                    continue;
+                }
                 else if (name === "ME"
                     || name === "SUPER"
                     || meta.includes("selection_screen_tab" /* IdentifierMeta.SelectionScreenTab */)
@@ -70096,7 +70115,7 @@ class FormLengthStats {
                 }
                 else if (type instanceof Statements.EndForm) {
                     if (pos) {
-                        res.push({ name, count, file, pos });
+                        res.push({ name: name, className: "", count, file, pos });
                     }
                     else {
                         continue;
@@ -70349,31 +70368,37 @@ exports.IncludeGraph = IncludeGraph;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.MethodLengthStats = void 0;
 const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const Expressions = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const expressions_1 = __webpack_require__(/*! ../abap/2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const _abap_object_1 = __webpack_require__(/*! ../objects/_abap_object */ "./node_modules/@abaplint/core/build/src/objects/_abap_object.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
 class MethodLengthStats {
     static run(obj) {
+        var _a;
         const res = [];
         let pos = undefined;
-        let name = "";
+        let methodName = "";
         let count = 0;
         let method = false;
         if (!(obj instanceof _abap_object_1.ABAPObject)) {
             return [];
         }
         for (const file of obj.getABAPFiles()) {
+            let className = "";
             for (const stat of file.getStatements()) {
                 const type = stat.get();
                 if (type instanceof Statements.MethodImplementation) {
                     pos = stat.getFirstToken().getStart();
-                    name = this.findName(stat);
+                    methodName = this.findName(stat);
                     method = true;
                     count = 0;
                 }
+                else if (type instanceof Statements.ClassImplementation) {
+                    className = ((_a = stat.findFirstExpression(Expressions.ClassName)) === null || _a === void 0 ? void 0 : _a.concatTokens()) || "INTERNAL_ERROR";
+                }
                 else if (type instanceof Statements.EndMethod) {
                     if (pos) {
-                        res.push({ name, count, file, pos });
+                        res.push({ name: methodName, className, count, file, pos });
                     }
                     else {
                         continue;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.41",
+    "version": "2.102.43",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,19 +38,19 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.41",
+        "@abaplint/core": "^2.102.43",
         "@types/chai": "^4.3.6",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
         "@types/mocha": "^10.0.1",
-        "@types/node": "^20.6.0",
+        "@types/node": "^20.6.2",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.8",
         "chalk": "^5.3.0",
         "eslint": "^8.49.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
-        "memfs": "^4.2.1",
+        "memfs": "^4.3.0",
         "minimist": "^1.2.8",
         "mocha": "^10.2.0",
         "progress": "^2.0.3",