@abaplint/cli 2.108.2 → 2.108.3

package/build/cli.js

@@ -41422,8 +41422,8 @@ class Issue {
             severity,
         });
     }
-    static atStatement(file, statement, message, key, severity, fix) {
-        return this.atRange(file, statement.getStart(), statement.getEnd(), message, key, severity, fix);
+    static atStatement(file, statement, message, key, severity, fix, alternativeFixes) {
+        return this.atRange(file, statement.getStart(), statement.getEnd(), message, key, severity, fix, alternativeFixes);
     }
     static atPosition(file, start, message, key, severity, fix) {
         const row = start.getRow();
@@ -41453,7 +41453,7 @@ class Issue {
             severity,
         });
     }
-    static atRange(file, start, end, message, key, severity, fix) {
+    static atRange(file, start, end, message, key, severity, fix, alternativeFixes) {
         severity = severity !== null && severity !== void 0 ? severity : severity_1.Severity.Error;
         return new Issue({
             filename: file.getFilename(),
@@ -41463,6 +41463,7 @@ class Issue {
             end,
             defaultFix: fix,
             severity,
+            alternativeFixes,
         });
     }
     static atToken(file, token, message, key, severity, fix) {
@@ -42062,26 +42063,36 @@ class CodeActions {
         const ret = [];
         for (const i of issues) {
             const fix = i.getDefaultFix();
-            if (fix === undefined) {
-                continue;
-            }
-            if (totals[i.getKey()] === undefined) {
-                totals[i.getKey()] = 1;
-            }
-            else {
-                totals[i.getKey()]++;
+            if (fix !== undefined) {
+                if (totals[i.getKey()] === undefined) {
+                    totals[i.getKey()] = 1;
+                }
+                else {
+                    totals[i.getKey()]++;
+                }
+                if (this.inRange(i, params.range) === true) {
+                    ret.push({
+                        title: "Apply fix, " + i.getKey(),
+                        kind: LServer.CodeActionKind.QuickFix,
+                        diagnostics: [diagnostics_1.Diagnostics.mapDiagnostic(i)],
+                        isPreferred: true,
+                        edit: _edit_1.LSPEdit.mapEdit(fix),
+                    });
+                    shown.add(i.getKey());
+                }
             }
-            if (this.inRange(i, params.range) === false) {
-                continue;
+            for (const alternative of i.getAlternativeFixes() || []) {
+                if (this.inRange(i, params.range) === true) {
+                    ret.push({
+                        title: alternative.description,
+                        kind: LServer.CodeActionKind.QuickFix,
+                        diagnostics: [diagnostics_1.Diagnostics.mapDiagnostic(i)],
+                        isPreferred: true,
+                        edit: _edit_1.LSPEdit.mapEdit(alternative.edit),
+                    });
+                    shown.add(i.getKey());
+                }
             }
-            ret.push({
-                title: "Apply fix, " + i.getKey(),
-                kind: LServer.CodeActionKind.QuickFix,
-                diagnostics: [diagnostics_1.Diagnostics.mapDiagnostic(i)],
-                isPreferred: true,
-                edit: _edit_1.LSPEdit.mapEdit(fix),
-            });
-            shown.add(i.getKey());
         }
         for (const s of shown) {
             if (totals[s] > 1) {
@@ -51483,7 +51494,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.108.2";
+        return "2.108.3";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -54312,6 +54323,7 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 class CheckSubrcConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -54358,6 +54370,12 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
     setConfig(conf) {
         this.conf = conf;
     }
+    buildFix(file, statement) {
+        return {
+            description: "Add ##SUBRC_OK",
+            edit: edit_helper_1.EditHelper.insertAt(file, statement.getLastToken().getEnd(), " ##SUBRC_OK"),
+        };
+    }
     runParsed(file) {
         const issues = [];
         const statements = file.getStatements();
@@ -54372,11 +54390,13 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
             if (config.openDataset === true
                 && statement.get() instanceof Statements.OpenDataset
                 && this.isChecked(i, statements) === false) {
+                // it doesnt make sense to ignore the subrc for open dataset, so no quick fix
                 issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
             }
             else if (config.authorityCheck === true
                 && statement.get() instanceof Statements.AuthorityCheck
                 && this.isChecked(i, statements) === false) {
+                // it doesnt make sense to ignore the subrc for authority checks, so no quick fix
                 issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
             }
             else if (config.selectSingle === true
@@ -54388,7 +54408,8 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
                 if (concat.startsWith("SELECT SINGLE @ABAP_TRUE FROM ")) {
                     continue;
                 }
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.selectTable === true
                 && statement.get() instanceof Statements.Select
@@ -54397,42 +54418,49 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
                 && statement.concatTokens().toUpperCase().startsWith("SELECT COUNT(*) ") === false
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.updateDatabase === true
                 && statement.get() instanceof Statements.UpdateDatabase
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.insertDatabase === true
                 && statement.get() instanceof Statements.InsertDatabase
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.modifyDatabase === true
                 && statement.get() instanceof Statements.ModifyDatabase
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.readTable === true
                 && statement.get() instanceof Statements.ReadTable
                 && this.isChecked(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.assign === true
                 && statement.get() instanceof Statements.Assign
                 && this.isSimpleAssign(statement) === false
                 && this.isChecked(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
             else if (config.find === true
                 && statement.get() instanceof Statements.Find
                 && this.isExemptedFind(statement) === false
                 && this.isChecked(i, statements) === false) {
-                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));
+                const fix = this.buildFix(file, statement);
+                issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity, undefined, [fix]));
             }
         }
         return issues;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.108.2",
+    "version": "2.108.3",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.108.2",
+        "@abaplint/core": "^2.108.3",
         "@types/chai": "^4.3.16",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",