@abaplint/core 2.86.4 → 2.86.5

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -894,15 +894,19 @@ ${indentation}    output = ${topTarget}.`;
         return undefined;
     }
     outlineFor(forLoop, indentation, lowFile, highSyntax) {
-        var _a, _b, _c, _d;
+        var _a, _b, _c, _d, _e;
         let body = "";
         let end = "";
         const loopSource = (_a = forLoop.findFirstExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens();
         const loopTargetField = (_b = forLoop.findFirstExpression(Expressions.TargetField)) === null || _b === void 0 ? void 0 : _b.concatTokens();
+        let cond = ((_c = forLoop.findDirectExpression(Expressions.ComponentCond)) === null || _c === void 0 ? void 0 : _c.concatTokens()) || "";
+        if (cond !== "") {
+            cond = " WHERE " + cond;
+        }
         if (forLoop.findDirectTokenByText("UNTIL")
             || forLoop.findDirectTokenByText("WHILE")) {
             const fieldDef = forLoop.findDirectExpression(Expressions.InlineFieldDefinition);
-            const field = (_c = fieldDef === null || fieldDef === void 0 ? void 0 : fieldDef.findFirstExpression(Expressions.Field)) === null || _c === void 0 ? void 0 : _c.concatTokens();
+            const field = (_d = fieldDef === null || fieldDef === void 0 ? void 0 : fieldDef.findFirstExpression(Expressions.Field)) === null || _d === void 0 ? void 0 : _d.concatTokens();
             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) {
@@ -915,12 +919,12 @@ ${indentation}    output = ${topTarget}.`;
             end += indentation + "ENDWHILE";
         }
         else if (loopTargetField) {
-            body += indentation + `LOOP AT ${loopSource} INTO DATA(${loopTargetField}).\n`;
+            body += indentation + `LOOP AT ${loopSource} INTO DATA(${loopTargetField})${cond}.\n`;
             end = "ENDLOOP";
         }
         else if (loopTargetField === undefined) {
-            const loopTargetFieldSymbol = (_d = forLoop.findFirstExpression(Expressions.TargetFieldSymbol)) === null || _d === void 0 ? void 0 : _d.concatTokens();
-            body += indentation + `LOOP AT ${loopSource} ASSIGNING FIELD-SYMBOL(${loopTargetFieldSymbol}).\n`;
+            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";
         }
         const l = forLoop.findDirectExpression(Expressions.Let);
@@ -1023,12 +1027,12 @@ ${indentation}    output = ${topTarget}.`;
                 name = init.getFirstToken().getStr();
                 body += indentation + `DATA(${name}) = ${(_a = reduceBody.findFirstExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens()}.\n`;
             }
-            const forLoop = reduceBody.findDirectExpression(Expressions.For);
-            if (forLoop === undefined) {
-                continue;
+            let end = "";
+            for (const forLoop of (reduceBody === null || reduceBody === void 0 ? void 0 : reduceBody.findDirectExpressions(Expressions.For)) || []) {
+                const outlineFor = this.outlineFor(forLoop, indentation, lowFile, highSyntax);
+                body += outlineFor.body;
+                end = outlineFor.end + `.\n` + end;
             }
-            const outlineFor = this.outlineFor(forLoop, indentation, lowFile, highSyntax);
-            body += outlineFor.body;
             const next = reduceBody.findDirectExpression(Expressions.ReduceNext);
             if (next === undefined) {
                 continue;
@@ -1048,7 +1052,7 @@ ${indentation}    output = ${topTarget}.`;
                     body += concat;
                 }
             }
-            body += indentation + outlineFor.end + `.\n`;
+            body += indentation + end;
             body += indentation + `${uniqueName} = ${name}.\n`;
             const abap = `DATA ${uniqueName} TYPE ${type}.\n` +
                 body +
@@ -1348,6 +1352,10 @@ ${indentation}    output = ${topTarget}.`;
             if (i.getFirstToken().getStr().toUpperCase() !== "CONV") {
                 continue;
             }
+            const end = i.findDirectTokenByText(")");
+            if (end === undefined) {
+                continue;
+            }
             const body = (_a = i.findDirectExpression(Expressions.ConvBody)) === null || _a === void 0 ? void 0 : _a.concatTokens();
             if (body === undefined) {
                 continue;
@@ -1359,7 +1367,7 @@ ${indentation}    output = ${topTarget}.`;
                 indent + `${uniqueName} = ${body}.\n` +
                 indent;
             const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, node.getFirstToken().getStart(), abap);
-            const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, i.getFirstToken().getStart(), i.getLastToken().getEnd(), uniqueName);
+            const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, i.getFirstToken().getStart(), end.getEnd(), uniqueName);
             const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
             return issue_1.Issue.atToken(lowFile, i.getFirstToken(), "Downport CONV", this.getMetadata().key, this.conf.severity, fix);
         }

package/package.json

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