@abaplint/core 2.88.2 → 2.88.3

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -924,6 +924,7 @@ ${indentation}    output = ${topTarget}.`;
             || forLoop.findDirectTokenByText("WHILE")) {
             const fieldDef = forLoop.findDirectExpression(Expressions.InlineFieldDefinition);
             const field = (_d = fieldDef === null || fieldDef === void 0 ? void 0 : fieldDef.findFirstExpression(Expressions.Field)) === null || _d === void 0 ? void 0 : _d.concatTokens();
+            const indexBackup = this.uniqueName(forLoop.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
             body += indentation + "DATA " + field + " TYPE i.\n";
             const second = fieldDef === null || fieldDef === void 0 ? void 0 : fieldDef.getChildren()[2];
             if ((second === null || second === void 0 ? void 0 : second.get()) instanceof Expressions.Source) {
@@ -931,15 +932,26 @@ ${indentation}    output = ${topTarget}.`;
             }
             const not = forLoop.findDirectTokenByText("UNTIL") ? " NOT" : "";
             const cond = forLoop.findFirstExpression(Expressions.Cond);
+            body += indentation + `DATA ${indexBackup} LIKE sy-index.\n`;
+            body += indentation + `${indexBackup} = sy-index.\n`;
             body += indentation + `WHILE${not} ${cond === null || cond === void 0 ? void 0 : cond.concatTokens()}.\n`;
-            end += `  ${field} = ${field} + 1.\n`;
+            body += indentation + `  sy-index = ${indexBackup}.\n`;
+            const then = forLoop.findExpressionAfterToken("THEN");
+            if (then) {
+                end += `  ${field} = ${then.concatTokens()}.\n`;
+            }
+            else {
+                end += `  ${field} = ${field} + 1.\n`;
+            }
             end += indentation + "ENDWHILE";
         }
         else if (loopTargetField) {
+            // todo, also backup sy-index / sy-tabix here?
             body += indentation + `LOOP AT ${loopSource} INTO DATA(${loopTargetField})${cond}.\n`;
             end = "ENDLOOP";
         }
         else if (loopTargetField === undefined) {
+            // todo, also backup sy-index / sy-tabix here?
             const loopTargetFieldSymbol = (_e = forLoop.findFirstExpression(Expressions.TargetFieldSymbol)) === null || _e === void 0 ? void 0 : _e.concatTokens();
             body += indentation + `LOOP AT ${loopSource} ASSIGNING FIELD-SYMBOL(${loopTargetFieldSymbol})${cond}.\n`;
             end = "ENDLOOP";

package/build/src/rules/message_exists.js

@@ -6,6 +6,7 @@ const issue_1 = require("../issue");
 const _abap_rule_1 = require("./_abap_rule");
 const _basic_rule_config_1 = require("./_basic_rule_config");
 const ddic_1 = require("../ddic");
+const _irule_1 = require("./_irule");
 class MessageExistsConf extends _basic_rule_config_1.BasicRuleConfig {
 }
 exports.MessageExistsConf = MessageExistsConf;
@@ -19,6 +20,7 @@ class MessageExistsRule extends _abap_rule_1.ABAPRule {
             key: "message_exists",
             title: "Check MESSAGE exists",
             shortDescription: `In message statements, check that the message class + id exist`,
+            tags: [_irule_1.RuleTag.Syntax],
         };
     }
     getConfig() {

package/build/src/rules/sicf_consistency.js

@@ -16,9 +16,8 @@ class SICFConsistency {
         return {
             key: "sicf_consistency",
             title: "SICF consistency",
-            shortDescription: `Checks the validity of ICF services:
-
-* Class defined in handler must exist
+            shortDescription: `Checks the validity of ICF services`,
+            extendedInformation: `* Class defined in handler must exist
 * Class must not have any syntax errors
 * Class must implement interface IF_HTTP_EXTENSION`,
         };

package/package.json

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