@abaplint/cli 2.119.13 → 2.119.15

package/build/cli.js

@@ -56244,7 +56244,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.119.13";
+        return "2.119.15";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -71601,9 +71601,14 @@ exports.NoAliases = exports.NoAliasesConf = void 0;
 const issue_1 = __webpack_require__(/*! ../issue */ "./node_modules/@abaplint/core/build/src/issue.js");
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./node_modules/@abaplint/core/build/src/rules/_basic_rule_config.js");
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "./node_modules/@abaplint/core/build/src/rules/_abap_rule.js");
-const Statements = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
+const visibility_1 = __webpack_require__(/*! ../abap/4_file_information/visibility */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/visibility.js");
 class NoAliasesConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** Skip reporting aliases in private sections. */
+        this.ignorePrivate = false;
+    }
 }
 exports.NoAliasesConf = NoAliasesConf;
 class NoAliases extends _abap_rule_1.ABAPRule {
@@ -71616,7 +71621,6 @@ class NoAliases extends _abap_rule_1.ABAPRule {
             key: "no_aliases",
             title: "No ALIASES",
             shortDescription: `Detects use of the ALIAS statement`,
-            extendedInformation: `Only one issue is reported for chained statements`,
             tags: [_irule_1.RuleTag.SingleFile],
         };
     }
@@ -71629,18 +71633,24 @@ class NoAliases extends _abap_rule_1.ABAPRule {
     runParsed(file) {
         const issues = [];
         const message = "Do not use ALIASES";
-        let prev = undefined;
-        for (const stat of file.getStatements()) {
-            if (stat.get() instanceof Statements.Aliases) {
-                if (prev && prev.getColon() === stat.getColon()) {
+        for (const classDef of file.getInfo().listClassDefinitions()) {
+            for (const alias of classDef.aliases) {
+                if (this.skipAlias(alias)) {
                     continue;
                 }
-                issues.push(issue_1.Issue.atStatement(file, stat, message, this.getMetadata().key, this.conf.severity));
-                prev = stat;
+                issues.push(issue_1.Issue.atIdentifier(alias.identifier, message, this.getMetadata().key, this.conf.severity));
+            }
+        }
+        for (const interfaceDef of file.getInfo().listInterfaceDefinitions()) {
+            for (const alias of interfaceDef.aliases) {
+                issues.push(issue_1.Issue.atIdentifier(alias.identifier, message, this.getMetadata().key, this.conf.severity));
             }
         }
         return issues;
     }
+    skipAlias(alias) {
+        return this.conf.ignorePrivate === true && alias.visibility === visibility_1.Visibility.Private;
+    }
 }
 exports.NoAliases = NoAliases;
 //# sourceMappingURL=no_aliases.js.map
@@ -78764,10 +78774,6 @@ class UnreachableCode extends _abap_rule_1.ABAPRule {
                 || node.get() instanceof _statement_1.Empty) {
                 continue;
             }
-            else if (this.isExit(node)) {
-                exit = true;
-                continue;
-            }
             else if (this.isStructure(node.get())) {
                 exit = false;
                 continue;
@@ -78776,6 +78782,11 @@ class UnreachableCode extends _abap_rule_1.ABAPRule {
                 const issue = issue_1.Issue.atStatement(file, node, this.getMessage(), this.getMetadata().key, this.conf.severity);
                 output.push(issue);
                 exit = false;
+                continue;
+            }
+            else if (this.isExit(node)) {
+                exit = true;
+                continue;
             }
         }
         return output;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.119.13",
+    "version": "2.119.15",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.119.13",
+        "@abaplint/core": "^2.119.15",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",