@abaplint/core 2.93.27 → 2.93.28

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -937,6 +937,9 @@ ${indentation}RAISE EXCEPTION ${uniqueName2}.`;
         if (sourceRef.findFirstExpression(Expressions.TableExpression)) {
             const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
             code = `ASSIGN ${sourceRef.concatTokens()} TO FIELD-SYMBOL(<${uniqueName}>).
+IF sy-subrc <> 0.
+  RAISE EXCEPTION TYPE cx_sy_itab_line_not_found.
+ENDIF.
 GET REFERENCE OF <${uniqueName}> INTO ${target.concatTokens()}`;
         }
         else {
@@ -1473,16 +1476,36 @@ ${indentation}    output = ${topTarget}.`;
             if (gby !== "") {
                 gby = " GROUP BY ( " + gby + " )";
             }
+            const fc = forLoop.findDirectExpression(Expressions.FieldChain);
+            if (fc) {
+                gby = " GROUP BY " + fc.concatTokens();
+            }
+            if (forLoop.findDirectTokenByText("ASCENDING")) {
+                gby += " ASCENDING";
+            }
+            if (forLoop.findDirectTokenByText("DESCENDING")) {
+                gby += " DESCENDING";
+            }
             const groups = forLoop.findExpressionAfterToken("GROUPS");
             if (groups) {
-                gby += " INTO DATA(" + groups.concatTokens() + ")";
+                const concat = groups.concatTokens();
+                if (concat.startsWith("<")) {
+                    gby += " ASSIGNING FIELD-SYMBOL(" + concat + ")";
+                }
+                else {
+                    gby += " INTO DATA(" + concat + ")";
+                }
             }
             let inGroup = "";
             if (forLoop.concatTokens().toUpperCase().includes(" IN GROUP ")) {
                 inGroup = "-items";
             }
+            let into = "INTO DATA";
+            if (loopTargetField.startsWith("<")) {
+                into = "ASSIGNING FIELD-SYMBOL";
+            }
             // todo, also backup sy-index / sy-tabix here?
-            body += indentation + `LOOP AT ${loopSource}${inGroup} INTO DATA(${loopTargetField})${from}${to}${cond}${gby}.\n`;
+            body += indentation + `LOOP AT ${loopSource}${inGroup} ${into}(${loopTargetField})${from}${to}${cond}${gby}.\n`;
             if (indexInto) {
                 body += indentation + "  DATA(" + indexInto + ") = sy-tabix.\n";
             }

package/package.json

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