@abaplint/core 2.86.5 → 2.86.6

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -267,6 +267,10 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         if (found) {
             return found;
         }
+        found = this.replaceMethodConditional(high, lowFile, highSyntax);
+        if (found) {
+            return found;
+        }
         found = this.replaceTableExpression(high, lowFile, highSyntax);
         if (found) {
             return found;
@@ -1428,6 +1432,18 @@ ${indentation}    output = ${topTarget}.`;
         }
         return undefined;
     }
+    replaceMethodConditional(node, lowFile, _highSyntax) {
+        for (const c of node.findAllExpressionsRecursive(Expressions.Compare)) {
+            const chain = c.findDirectExpression(Expressions.MethodCallChain);
+            if (chain === undefined) {
+                continue;
+            }
+            const end = chain.getLastToken().getEnd();
+            const fix = edit_helper_1.EditHelper.insertAt(lowFile, end, " IS NOT INITIAL");
+            return issue_1.Issue.atToken(lowFile, chain.getFirstToken(), "Downport method conditional", this.getMetadata().key, this.conf.severity, fix);
+        }
+        return undefined;
+    }
     replaceContains(node, lowFile, highSyntax) {
         const spag = highSyntax.spaghetti.lookupPosition(node.getFirstToken().getStart(), lowFile.getFilename());
         // only downport if its an single method call condition

package/build/src/rules/keyword_case.js

@@ -141,12 +141,16 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
             }
         }
         const skip = new Skip(this.getConfig());
+        let prev = undefined;
         for (const statement of file.getStatements()) {
             if (skip.skipStatement(statement) === true) {
                 continue;
             }
             let result = this.traverse(statement, statement.get());
             if (result.length > 0) {
+                if (prev && result[0].token.getStart().equals(prev.getStart())) {
+                    continue;
+                }
                 if (statement.getColon() !== undefined) {
                     // if its a chained statement, go token by token
                     result = [result[0]];
@@ -156,6 +160,7 @@ class KeywordCase extends _abap_rule_1.ABAPRule {
                 if (issues.length > MAX_ISSUES) {
                     break;
                 }
+                prev = result[0].token;
             }
         }
         return issues;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.86.5",
+    "version": "2.86.6",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",