@abaplint/cli 2.113.12 → 2.113.14

package/build/cli.js

@@ -20038,6 +20038,7 @@ class ABAPFileInformation {
                 identifier: new _identifier_1.Identifier(methodName, this.filename),
                 isRedefinition: def.findDirectExpression(Expressions.Redefinition) !== undefined,
                 isForTesting: def.concatTokens().toUpperCase().includes(" FOR TESTING"),
+                isFinal: def.concatTokens().toUpperCase().includes(" FINAL"),
                 isAbstract: def.findDirectExpression(Expressions.Abstract) !== undefined,
                 isEventHandler: def.findDirectExpression(Expressions.EventHandler) !== undefined,
                 visibility,
@@ -52746,7 +52747,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.113.12";
+        return "2.113.14";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -73986,7 +73987,7 @@ class UnusedMethods {
             extendedInformation: `Checks private and protected methods.
 
 Unused methods are not reported if the object contains parser or syntax errors.
-Quick fixes only appears for private methods.
+Quick fixes only appears for private methods or projected methods where the class doesnt have any subclasses.
 
 Skips:
 * methods FOR TESTING
@@ -74079,7 +74080,9 @@ Skips:
                 continue;
             }
             let fix = undefined;
-            if (i.visibility === visibility_1.Visibility.Private) {
+            if (i.visibility === visibility_1.Visibility.Private
+                || i.isFinal === true
+                || (i.visibility === visibility_1.Visibility.Protected && this.hasSubClass(obj) === false)) {
                 const implementation = this.findMethodImplementation(i, file);
                 if (implementation !== undefined) {
                     const fix1 = edit_helper_1.EditHelper.deleteStatement(file, statement);
@@ -74092,6 +74095,22 @@ Skips:
         }
         return issues;
     }
+    hasSubClass(obj) {
+        var _a, _b, _c;
+        if (!(obj instanceof objects_1.Class)) {
+            return false;
+        }
+        if (((_a = obj.getDefinition()) === null || _a === void 0 ? void 0 : _a.isFinal()) === true) {
+            return false;
+        }
+        for (const r of this.reg.getObjects()) {
+            if (r instanceof objects_1.Class
+                && ((_c = (_b = r.getDefinition()) === null || _b === void 0 ? void 0 : _b.getSuperClass()) === null || _c === void 0 ? void 0 : _c.toUpperCase()) === obj.getName().toUpperCase()) {
+                return true;
+            }
+        }
+        return false;
+    }
     findMethodImplementation(method, file) {
         var _a, _b;
         for (const classImplementation of ((_a = file.getStructure()) === null || _a === void 0 ? void 0 : _a.findAllStructures(Structures.ClassImplementation)) || []) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.113.12",
+    "version": "2.113.14",
     "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.12",
+        "@abaplint/core": "^2.113.14",
         "@types/chai": "^4.3.19",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",