@abaplint/cli 2.119.12 → 2.119.14

package/build/cli.js

@@ -16601,7 +16601,7 @@ const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@a
 const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 class Return {
     getMatcher() {
-        return (0, combi_1.seq)((0, combi_1.str)("RETURN"), (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v758, expressions_1.Source)));
+        return (0, combi_1.seq)((0, combi_1.str)("RETURN"), (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v758, expressions_1.Source, version_1.Version.OpenABAP)));
     }
 }
 exports.Return = Return;
@@ -56244,7 +56244,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.119.12";
+        return "2.119.14";
     }
     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
@@ -78313,10 +78323,14 @@ const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "./
 const _irule_1 = __webpack_require__(/*! ./_irule */ "./node_modules/@abaplint/core/build/src/rules/_irule.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "./node_modules/@abaplint/core/build/src/edit_helper.js");
 const _statement_1 = __webpack_require__(/*! ../abap/2_statements/statements/_statement */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/_statement.js");
+const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplint/core/build/src/objects/index.js");
+const ddic_1 = __webpack_require__(/*! ../ddic */ "./node_modules/@abaplint/core/build/src/ddic.js");
 class UnnecessaryChainingConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
         this.maxIssuesPerFile = 10;
+        /** Ignore global exception classes */
+        this.ignoreExceptions = true;
     }
 }
 exports.UnnecessaryChainingConf = UnnecessaryChainingConf;
@@ -78342,8 +78356,15 @@ class UnnecessaryChaining extends _abap_rule_1.ABAPRule {
     setConfig(conf) {
         this.conf = conf;
     }
-    runParsed(file) {
+    runParsed(file, obj) {
         const issues = [];
+        if (obj instanceof objects_1.Class) {
+            const definition = obj.getClassDefinition();
+            const ddic = new ddic_1.DDIC(this.reg);
+            if (this.conf.ignoreExceptions === true && ddic.isException(definition, obj)) {
+                return issues;
+            }
+        }
         let max = this.getConfig().maxIssuesPerFile;
         if (max === undefined || max < 1) {
             max = 10;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.119.12",
+    "version": "2.119.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.119.12",
+        "@abaplint/core": "^2.119.14",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",