@abaplint/core 2.93.15 → 2.93.16

package/build/src/abap/2_statements/statement_parser.js

@@ -104,18 +104,21 @@ class StatementParser {
         const result = [];
         for (let statement of wa.statements) {
             // dont use CALL METHOD, when executing lazy, it easily gives a Move for the last statment if lazy logic is evaluated
-            if (statement.get() instanceof _statement_1.Unknown
-                && statement.concatTokens().toUpperCase().startsWith("CALL METHOD ") === false
-                && statement.concatTokens().toUpperCase().startsWith("CALL FUNCTION ") === false) {
-                for (const { first, second } of this.buildSplits(statement.getTokens())) {
-                    if (second.length === 1) {
-                        continue; // probably punctuation
-                    }
-                    const s = this.categorizeStatement(new nodes_1.StatementNode(new _statement_1.Unknown()).setChildren(this.tokensToNodes(second)));
-                    if (!(s.get() instanceof _statement_1.Unknown)) {
-                        result.push(new nodes_1.StatementNode(new _statement_1.Unknown()).setChildren(this.tokensToNodes(first)));
-                        statement = s;
-                        break;
+            if (statement.get() instanceof _statement_1.Unknown) {
+                const concat = statement.concatTokens().toUpperCase();
+                if (concat.startsWith("CALL METHOD ") === false
+                    && concat.startsWith("RAISE EXCEPTION TYPE ") === false
+                    && concat.startsWith("CALL FUNCTION ") === false) {
+                    for (const { first, second } of this.buildSplits(statement.getTokens())) {
+                        if (second.length === 1) {
+                            continue; // probably punctuation
+                        }
+                        const s = this.categorizeStatement(new nodes_1.StatementNode(new _statement_1.Unknown()).setChildren(this.tokensToNodes(second)));
+                        if (!(s.get() instanceof _statement_1.Unknown)) {
+                            result.push(new nodes_1.StatementNode(new _statement_1.Unknown()).setChildren(this.tokensToNodes(first)));
+                            statement = s;
+                            break;
+                        }
                     }
                 }
             }

package/build/src/registry.js

@@ -63,7 +63,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.93.15";
+        return "2.93.16";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/change_if_to_case.js

@@ -9,6 +9,13 @@ const _abap_rule_1 = require("./_abap_rule");
 const _basic_rule_config_1 = require("./_basic_rule_config");
 const _irule_1 = require("./_irule");
 class ChangeIfToCaseConf extends _basic_rule_config_1.BasicRuleConfig {
+    constructor() {
+        super(...arguments);
+        /** skip specific names, case insensitive regular expression
+         * @uniqueItems true
+         */
+        this.skipNames = [];
+    }
 }
 exports.ChangeIfToCaseConf = ChangeIfToCaseConf;
 class ChangeIfToCase extends _abap_rule_1.ABAPRule {
@@ -113,6 +120,12 @@ ENDCASE.`,
         else {
             return false;
         }
+        for (const skip of this.getConfig().skipNames || []) {
+            const reg = new RegExp(skip, "i");
+            if (chain.match(reg)) {
+                return false;
+            }
+        }
         for (const t of tuples) {
             if (t.left !== chain && t.right !== chain) {
                 return false;

package/build/src/rules/downport.js

@@ -139,6 +139,7 @@ Only one transformation is applied to a statement at a time, so multiple steps m
                 const message = "Internal Error: Statement lengths does not match";
                 ret.push(Issue.atStatement(lowFile, lowStatements[0], message, this.getMetadata().key));
                 */
+                // hmm, add some way to disable lazyUnknown() in statement_parser.ts
                 continue;
             }
             for (let i = 0; i < lowStatements.length; i++) {

package/build/src/rules/sicf_consistency.js

@@ -8,7 +8,9 @@ const position_1 = require("../position");
 class SICFConsistencyConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
-        /** skip specific names, case insensitive */
+        /** skip specific names, case insensitive
+         * @uniqueItems true
+         */
         this.skipNames = [];
     }
 }

package/build/src/rules/unused_types.js

@@ -39,7 +39,9 @@ class WorkArea {
 class UnusedTypesConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
-        /** skip specific names, case insensitive */
+        /** skip specific names, case insensitive
+         * @uniqueItems true
+         */
         this.skipNames = [];
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.93.15",
+    "version": "2.93.16",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -46,7 +46,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.31.0",
+        "@microsoft/api-extractor": "^7.31.1",
         "@types/chai": "^4.3.3",
         "@types/mocha": "^9.1.1",
         "@types/node": "^18.7.18",