@abaplint/core 2.91.23 → 2.91.26

package/build/abaplint.d.ts

@@ -1866,6 +1866,8 @@ declare namespace Expressions {
         Language,
         Length,
         Let,
+        LoopGroupByComponent,
+        LoopGroupByTarget,
         LoopGroupBy,
         MacroName,
         MessageClass_2 as MessageClass,
@@ -1936,6 +1938,7 @@ declare namespace Expressions {
         SQLFieldListLoop,
         SQLFieldList,
         SQLFieldName,
+        SQLField,
         SQLForAllEntries,
         SQLFromSource,
         SQLFrom,
@@ -3548,6 +3551,14 @@ declare class LoopGroupBy extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class LoopGroupByComponent extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
+declare class LoopGroupByTarget extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class MacroName extends Expression {
     getRunnable(): IStatementRunnable;
 }
@@ -4938,6 +4949,10 @@ declare class SQLCond extends Expression {
     getRunnable(): IStatementRunnable;
 }
 
+declare class SQLField extends Expression {
+    getRunnable(): IStatementRunnable;
+}
+
 declare class SQLFieldList extends Expression {
     getRunnable(): IStatementRunnable;
 }

package/build/src/abap/2_statements/expressions/index.js

@@ -96,6 +96,8 @@ __exportStar(require("./interface_name"), exports);
 __exportStar(require("./language"), exports);
 __exportStar(require("./length"), exports);
 __exportStar(require("./let"), exports);
+__exportStar(require("./loop_group_by_component"), exports);
+__exportStar(require("./loop_group_by_target"), exports);
 __exportStar(require("./loop_group_by"), exports);
 __exportStar(require("./macro_name"), exports);
 __exportStar(require("./message_class"), exports);
@@ -166,6 +168,7 @@ __exportStar(require("./sql_cond"), exports);
 __exportStar(require("./sql_field_list_loop"), exports);
 __exportStar(require("./sql_field_list"), exports);
 __exportStar(require("./sql_field_name"), exports);
+__exportStar(require("./sql_field"), exports);
 __exportStar(require("./sql_for_all_entries"), exports);
 __exportStar(require("./sql_from_source"), exports);
 __exportStar(require("./sql_from"), exports);

package/build/src/abap/2_statements/expressions/loop_group_by.js

@@ -3,18 +3,13 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.LoopGroupBy = void 0;
 const combi_1 = require("../combi");
 const tokens_1 = require("../../1_lexer/tokens");
-const component_name_1 = require("./component_name");
-const component_compare_1 = require("./component_compare");
-const target_1 = require("./target");
-const fstarget_1 = require("./fstarget");
 const source_1 = require("./source");
+const loop_group_by_target_1 = require("./loop_group_by_target");
+const loop_group_by_component_1 = require("./loop_group_by_component");
 class LoopGroupBy extends combi_1.Expression {
     getRunnable() {
-        const into = (0, combi_1.seq)((0, combi_1.opt)("REFERENCE"), "INTO", target_1.Target);
-        const assigning = (0, combi_1.seq)("ASSIGNING", fstarget_1.FSTarget);
-        const groupSize = (0, combi_1.seq)(component_name_1.ComponentName, "=", "GROUP SIZE");
-        const components = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), (0, combi_1.plus)((0, combi_1.alt)(component_compare_1.ComponentCompare, groupSize)), (0, combi_1.tok)(tokens_1.WParenRightW));
-        const ret = (0, combi_1.seq)((0, combi_1.alt)(source_1.Source, components), (0, combi_1.optPrio)("ASCENDING"), (0, combi_1.optPrio)("WITHOUT MEMBERS"), (0, combi_1.optPrio)((0, combi_1.alt)(into, assigning)));
+        const components = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), (0, combi_1.plus)(loop_group_by_component_1.LoopGroupByComponent), (0, combi_1.tok)(tokens_1.WParenRightW));
+        const ret = (0, combi_1.seq)((0, combi_1.alt)(source_1.Source, components), (0, combi_1.optPrio)("ASCENDING"), (0, combi_1.optPrio)("WITHOUT MEMBERS"), loop_group_by_target_1.LoopGroupByTarget);
         return ret;
     }
 }

package/build/src/abap/2_statements/expressions/loop_group_by_component.js

@@ -0,0 +1,15 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.LoopGroupByComponent = void 0;
+const combi_1 = require("../combi");
+const component_name_1 = require("./component_name");
+const component_compare_simple_1 = require("./component_compare_simple");
+class LoopGroupByComponent extends combi_1.Expression {
+    getRunnable() {
+        const groupSize = (0, combi_1.seq)(component_name_1.ComponentName, "=", "GROUP SIZE");
+        const components = (0, combi_1.alt)(component_compare_simple_1.ComponentCompareSimple, groupSize);
+        return components;
+    }
+}
+exports.LoopGroupByComponent = LoopGroupByComponent;
+//# sourceMappingURL=loop_group_by_component.js.map

package/build/src/abap/2_statements/expressions/loop_group_by_target.js

@@ -0,0 +1,15 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.LoopGroupByTarget = void 0;
+const combi_1 = require("../combi");
+const target_1 = require("./target");
+const fstarget_1 = require("./fstarget");
+class LoopGroupByTarget extends combi_1.Expression {
+    getRunnable() {
+        const into = (0, combi_1.seq)((0, combi_1.opt)("REFERENCE"), "INTO", target_1.Target);
+        const assigning = (0, combi_1.seq)("ASSIGNING", fstarget_1.FSTarget);
+        return (0, combi_1.optPrio)((0, combi_1.alt)(into, assigning));
+    }
+}
+exports.LoopGroupByTarget = LoopGroupByTarget;
+//# sourceMappingURL=loop_group_by_target.js.map

package/build/src/abap/2_statements/expressions/sql_field.js

@@ -0,0 +1,22 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.SQLField = void 0;
+const combi_1 = require("../combi");
+const _1 = require(".");
+const version_1 = require("../../../version");
+const tokens_1 = require("../../1_lexer/tokens");
+const sql_function_1 = require("./sql_function");
+const simple_field_chain_1 = require("./simple_field_chain");
+const sql_path_1 = require("./sql_path");
+class SQLField extends combi_1.Expression {
+    getRunnable() {
+        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), simple_field_chain_1.SimpleFieldChain));
+        const as = (0, combi_1.seq)("AS", _1.SQLAsName);
+        const field = (0, combi_1.altPrio)(_1.SQLAggregation, _1.SQLCase, sql_function_1.SQLFunction, sql_path_1.SQLPath, _1.SQLFieldName, abap, _1.Constant);
+        const sub = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenLeftW)), field, (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenRightW))));
+        const arith = (0, combi_1.ver)(version_1.Version.v740sp05, sub);
+        return (0, combi_1.seq)(field, (0, combi_1.optPrio)(arith), (0, combi_1.optPrio)(as));
+    }
+}
+exports.SQLField = SQLField;
+//# sourceMappingURL=sql_field.js.map

package/build/src/abap/2_statements/expressions/sql_field_list.js

@@ -4,19 +4,10 @@ exports.SQLFieldList = void 0;
 const combi_1 = require("../combi");
 const _1 = require(".");
 const version_1 = require("../../../version");
-const tokens_1 = require("../../1_lexer/tokens");
-const sql_function_1 = require("./sql_function");
-const simple_field_chain_1 = require("./simple_field_chain");
-const sql_path_1 = require("./sql_path");
 class SQLFieldList extends combi_1.Expression {
     getRunnable() {
         const comma = (0, combi_1.optPrio)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
-        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), simple_field_chain_1.SimpleFieldChain));
-        const as = (0, combi_1.seq)("AS", _1.SQLAsName);
-        const field = (0, combi_1.altPrio)(_1.SQLAggregation, _1.SQLCase, sql_function_1.SQLFunction, sql_path_1.SQLPath, _1.SQLFieldName, abap, _1.Constant);
-        const sub = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenLeftW)), field, (0, combi_1.optPrio)((0, combi_1.tok)(tokens_1.WParenRightW))));
-        const arith = (0, combi_1.ver)(version_1.Version.v740sp05, sub);
-        return (0, combi_1.altPrio)("*", _1.Dynamic, (0, combi_1.plusPrio)((0, combi_1.seq)(field, (0, combi_1.optPrio)(arith), (0, combi_1.optPrio)(as), comma)));
+        return (0, combi_1.altPrio)("*", _1.Dynamic, (0, combi_1.plusPrio)((0, combi_1.seq)(_1.SQLField, comma)));
     }
 }
 exports.SQLFieldList = SQLFieldList;

package/build/src/abap/2_statements/expressions/sql_field_list_loop.js

@@ -4,14 +4,13 @@ exports.SQLFieldListLoop = void 0;
 const combi_1 = require("../combi");
 const _1 = require(".");
 const version_1 = require("../../../version");
-const sql_function_1 = require("./sql_function");
 const sql_path_1 = require("./sql_path");
 // loop must include one field from the database table
 class SQLFieldListLoop extends combi_1.Expression {
     getRunnable() {
         const comma = (0, combi_1.opt)((0, combi_1.ver)(version_1.Version.v740sp05, ","));
         const as = (0, combi_1.seq)("AS", _1.SQLAsName);
-        const someField = (0, combi_1.seq)((0, combi_1.alt)(_1.SQLFieldName, sql_path_1.SQLPath, _1.SQLAggregation, sql_function_1.SQLFunction, _1.SQLCase), (0, combi_1.optPrio)(as), comma);
+        const someField = (0, combi_1.seq)(_1.SQLField, comma);
         const fieldList = (0, combi_1.seq)((0, combi_1.star)(someField), (0, combi_1.alt)(_1.SQLFieldName, sql_path_1.SQLPath), (0, combi_1.optPrio)(as), comma, (0, combi_1.star)(someField));
         const fields = (0, combi_1.alt)("*", _1.Dynamic, fieldList);
         return fields;

package/build/src/abap/5_syntax/basic_types.js

@@ -24,6 +24,7 @@ class BasicTypes {
     }
     lookupQualifiedName(name) {
         var _a;
+        // argh, todo, rewrite this entire method
         if (name === undefined) {
             return undefined;
         }
@@ -43,6 +44,21 @@ class BasicTypes {
                 }
             }
         }
+        else if (name.includes("-")) {
+            const split = name.split("-");
+            const typeName = split[0];
+            const fieldName = split[1];
+            const type = this.scope.findType(typeName);
+            if (type) {
+                const stru = type.getType();
+                if (stru instanceof basic_1.StructureType) {
+                    const f = stru.getComponentByName(fieldName);
+                    if (f) {
+                        return new _typed_identifier_1.TypedIdentifier(type.getToken(), type.getFilename(), f);
+                    }
+                }
+            }
+        }
         const lookup = this.scope.getDDIC().lookupNoVoid(name);
         const id = (_a = lookup === null || lookup === void 0 ? void 0 : lookup.object) === null || _a === void 0 ? void 0 : _a.getIdentifier();
         if (id && (lookup === null || lookup === void 0 ? void 0 : lookup.type)) {
@@ -193,7 +209,7 @@ class BasicTypes {
         }
         return undefined;
     }
-    simpleType(node, noQualifiedName) {
+    simpleType(node, qualifiedNamePrefix) {
         let nameExpr = node.findFirstExpression(Expressions.NamespaceSimpleName);
         if (nameExpr === undefined) {
             nameExpr = node.findFirstExpression(Expressions.DefinitionName);
@@ -211,15 +227,15 @@ class BasicTypes {
                 qualifiedName = name.getStr();
             }
             else {
-                qualifiedName = name.getStr();
+                qualifiedName = (qualifiedNamePrefix || "") + name.getStr();
                 if (this.scope.getType() === _scope_type_1.ScopeType.ClassDefinition
                     || this.scope.getType() === _scope_type_1.ScopeType.Interface) {
                     qualifiedName = this.scope.getName() + "=>" + qualifiedName;
                 }
             }
         }
-        if (noQualifiedName === true) {
-            qualifiedName = undefined;
+        else if (qualifiedNamePrefix) {
+            qualifiedName = qualifiedNamePrefix + qualifiedName;
         }
         const found = this.parseType(node, qualifiedName);
         if (found) {

package/build/src/abap/5_syntax/expressions/loop_group_by.js

@@ -10,10 +10,8 @@ const target_1 = require("./target");
 class LoopGroupBy {
     runSyntax(node, scope, filename) {
         const components = [];
-        for (const c of node.findDirectExpressions(Expressions.ComponentCompare)) {
-            for (const f of c.findDirectExpressions(Expressions.ComponentChainSimple)) {
-                components.push({ name: f.getFirstToken().getStr(), type: new basic_1.VoidType("todoGroupBy") });
-            }
+        for (const c of node.findDirectExpressions(Expressions.LoopGroupByComponent)) {
+            components.push({ name: c.getFirstToken().getStr(), type: new basic_1.VoidType("todoGroupBy") });
         }
         if (components.length === 0) {
             return;
@@ -32,8 +30,10 @@ class LoopGroupBy {
         if (inlinefs) {
             new inline_fs_1.InlineFS().runSyntax(inlinefs, scope, filename, sourceType);
         }
-        for (const t of node.findDirectExpressions(Expressions.ComponentCompare)) {
-            new component_compare_1.ComponentCompare().runSyntax(t, scope, filename);
+        for (const c of node.findDirectExpressions(Expressions.LoopGroupByComponent)) {
+            for (const t of c.findDirectExpressions(Expressions.ComponentCompareSimple)) {
+                new component_compare_1.ComponentCompare().runSyntax(t, scope, filename);
+            }
         }
     }
 }

package/build/src/abap/5_syntax/expressions/type_table.js

@@ -7,7 +7,7 @@ const Expressions = require("../../2_statements/expressions");
 const basic_1 = require("../../types/basic");
 const _scope_type_1 = require("../_scope_type");
 class TypeTable {
-    runSyntax(node, scope, filename) {
+    runSyntax(node, scope, filename, qualifiedNamePrefix) {
         // todo, input is currently the statement, but should be the expression?
         let nameExpr = node.findFirstExpression(Expressions.DefinitionName);
         if (nameExpr === undefined) {
@@ -17,9 +17,9 @@ class TypeTable {
             return undefined;
         }
         const name = nameExpr.getFirstToken();
-        let qualifiedName = "";
+        let qualifiedName = qualifiedNamePrefix || "";
         if (node.getFirstToken().getStr().toUpperCase() === "TYPES") {
-            qualifiedName = name.getStr();
+            qualifiedName = qualifiedName + name.getStr();
             if (scope.getType() === _scope_type_1.ScopeType.ClassDefinition
                 || scope.getType() === _scope_type_1.ScopeType.Interface) {
                 qualifiedName = scope.getName() + "=>" + qualifiedName;

package/build/src/abap/5_syntax/statements/append.js

@@ -6,6 +6,7 @@ const source_1 = require("../expressions/source");
 const target_1 = require("../expressions/target");
 const basic_1 = require("../../types/basic");
 const fstarget_1 = require("../expressions/fstarget");
+const inline_data_1 = require("../expressions/inline_data");
 class Append {
     runSyntax(node, scope, filename) {
         let targetType = undefined;
@@ -21,6 +22,14 @@ class Append {
             const rowType = targetType instanceof basic_1.TableType ? targetType.getRowType() : targetType;
             new fstarget_1.FSTarget().runSyntax(fsTarget, scope, filename, rowType);
         }
+        const dataTarget = node.findExpressionAfterToken("INTO");
+        if (dataTarget && node.concatTokens().toUpperCase().includes(" REFERENCE INTO DATA(")) {
+            if (!(targetType instanceof basic_1.TableType) && !(targetType instanceof basic_1.VoidType)) {
+                throw new Error("APPEND to non table type");
+            }
+            const rowType = targetType instanceof basic_1.TableType ? targetType.getRowType() : targetType;
+            new inline_data_1.InlineData().runSyntax(dataTarget, scope, filename, new basic_1.DataReference(rowType));
+        }
         let source = node.findDirectExpression(Expressions.Source);
         if (source === undefined) {
             source = node.findDirectExpression(Expressions.SimpleSource4);

package/build/src/abap/5_syntax/statements/type.js

@@ -7,12 +7,12 @@ const basic_1 = require("../../types/basic");
 const Expressions = require("../../2_statements/expressions");
 const type_table_1 = require("../expressions/type_table");
 class Type {
-    runSyntax(node, scope, filename, noQualifiedName) {
+    runSyntax(node, scope, filename, qualifiedNamePrefix) {
         const tt = node.findFirstExpression(Expressions.TypeTable);
         if (tt) {
-            return new type_table_1.TypeTable().runSyntax(node, scope, filename);
+            return new type_table_1.TypeTable().runSyntax(node, scope, filename, qualifiedNamePrefix);
         }
-        const found = new basic_types_1.BasicTypes(filename, scope).simpleType(node, noQualifiedName);
+        const found = new basic_types_1.BasicTypes(filename, scope).simpleType(node, qualifiedNamePrefix);
         if (found) {
             return found;
         }

package/build/src/abap/5_syntax/structures/types.js

@@ -20,7 +20,7 @@ class Types {
             const ctyp = c.get();
             if (c instanceof nodes_1.StatementNode) {
                 if (ctyp instanceof Statements.Type) {
-                    const found = new type_1.Type().runSyntax(c, scope, filename, true);
+                    const found = new type_1.Type().runSyntax(c, scope, filename, name.getStr() + "-");
                     if (found) {
                         components.push({ name: found.getName(), type: found.getType() });
                     }

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -208,6 +208,10 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         if (found) {
             return found;
         }
+        found = this.downportLoopGroup(high, lowFile, highSyntax, highFile);
+        if (found) {
+            return found;
+        }
         found = this.callFunctionParameterSimple(high, lowFile, highSyntax);
         if (found) {
             return found;
@@ -421,7 +425,7 @@ Only one transformation is applied to a statement at a time, so multiple steps m
             return undefined;
         }
         let fieldDefinition = "";
-        const fields = fieldList.findDirectExpressions(Expressions.SQLFieldName);
+        const fields = fieldList.findAllExpressions(Expressions.SQLFieldName);
         const name = ((_c = inlineData.findFirstExpression(Expressions.TargetField)) === null || _c === void 0 ? void 0 : _c.concatTokens()) || "error";
         if (fields.length === 1) {
             fieldDefinition = `DATA ${name} TYPE ${tableName}-${fields[0].concatTokens()}.`;
@@ -477,7 +481,7 @@ ${indentation}`);
             return undefined;
         }
         let fieldDefinitions = "";
-        for (const f of fieldList.findDirectExpressions(Expressions.SQLFieldName)) {
+        for (const f of fieldList.findAllExpressions(Expressions.SQLFieldName)) {
             let fieldName = f.concatTokens();
             if (fieldName.includes("~")) {
                 const split = fieldName.split("~");
@@ -860,6 +864,67 @@ ${indentation}RAISE EXCEPTION ${uniqueName2}.`;
         const fix = edit_helper_1.EditHelper.replaceRange(lowFile, start, end, code);
         return issue_1.Issue.atToken(lowFile, high.getFirstToken(), "Downport, simple REF move", this.getMetadata().key, this.conf.severity, fix);
     }
+    downportLoopGroup(high, lowFile, highSyntax, highFile) {
+        var _a, _b, _c, _d, _e, _f, _g, _h;
+        if (!(high.get() instanceof Statements.Loop)) {
+            return undefined;
+        }
+        const group = high.findDirectExpression(Expressions.LoopGroupBy);
+        if (group === undefined) {
+            return undefined;
+        }
+        const groupTargetName = ((_a = group.findFirstExpression(Expressions.TargetField)) === null || _a === void 0 ? void 0 : _a.concatTokens()) || "nameNotFound";
+        const loopSourceName = ((_b = high.findFirstExpression(Expressions.SimpleSource2)) === null || _b === void 0 ? void 0 : _b.concatTokens()) || "nameNotFound";
+        const loopTargetName = ((_c = high.findFirstExpression(Expressions.TargetField)) === null || _c === void 0 ? void 0 : _c.concatTokens()) || "nameNotFound";
+        const groupTarget = ((_d = group.findDirectExpression(Expressions.LoopGroupByTarget)) === null || _d === void 0 ? void 0 : _d.concatTokens()) || "";
+        let loopSourceRowType = "typeNotFound";
+        const spag = highSyntax.spaghetti.lookupPosition(high.getFirstToken().getStart(), lowFile.getFilename());
+        if (spag !== undefined) {
+            const found = spag.findVariable(loopSourceName);
+            const tt = found === null || found === void 0 ? void 0 : found.getType();
+            if (tt instanceof basic_1.TableType) {
+                loopSourceRowType = tt.getRowType().getQualifiedName() || "typeNotFound";
+            }
+        }
+        let code = `TYPES: BEGIN OF ${groupTargetName}#type,\n`;
+        for (const c of group.findAllExpressions(Expressions.LoopGroupByComponent)) {
+            const name = c.findFirstExpression(Expressions.ComponentName);
+            let type = ((_e = c.findFirstExpression(Expressions.Source)) === null || _e === void 0 ? void 0 : _e.concatTokens()) || "todo";
+            if ((_f = c.concatTokens()) === null || _f === void 0 ? void 0 : _f.toUpperCase().endsWith(" = GROUP SIZE")) {
+                type = "i";
+            }
+            else {
+                type = type.replace(loopTargetName, loopSourceRowType);
+                type = type.replace("->", "-");
+            }
+            code += `  ${name === null || name === void 0 ? void 0 : name.concatTokens()} TYPE ${type},\n`;
+        }
+        code += `  items LIKE ${loopSourceName},
+END OF ${groupTargetName}#type.
+DATA ${groupTargetName}#tab TYPE STANDARD TABLE OF ${groupTargetName}#type WITH DEFAULT KEY.
+* todo, aggregation code here
+LOOP AT ${groupTargetName}#tab ${groupTarget}.`;
+        let fix = edit_helper_1.EditHelper.replaceRange(lowFile, high.getFirstToken().getStart(), high.getLastToken().getEnd(), code);
+        for (const l of ((_g = highFile.getStructure()) === null || _g === void 0 ? void 0 : _g.findAllStructures(Structures.Loop)) || []) {
+            // make sure to find the correct/current loop statement
+            if (l.findDirectStatement(Statements.Loop) !== high) {
+                continue;
+            }
+            for (const loop of l.findAllStatements(Statements.Loop)) {
+                if ((_h = loop.concatTokens()) === null || _h === void 0 ? void 0 : _h.toUpperCase().startsWith("LOOP AT GROUP ")) {
+                    const subLoopSource = loop.findFirstExpression(Expressions.SimpleSource2);
+                    if (subLoopSource === undefined) {
+                        continue;
+                    }
+                    const subLoopSourceName = (subLoopSource === null || subLoopSource === void 0 ? void 0 : subLoopSource.concatTokens()) || "nameNotFound";
+                    const subCode = `LOOP AT ${subLoopSourceName}->items`;
+                    const subFix = edit_helper_1.EditHelper.replaceRange(lowFile, loop.getFirstToken().getStart(), subLoopSource.getLastToken().getEnd(), subCode);
+                    fix = edit_helper_1.EditHelper.merge(subFix, fix);
+                }
+            }
+        }
+        return issue_1.Issue.atToken(lowFile, high.getFirstToken(), "Downport, LOOP GROUP", this.getMetadata().key, this.conf.severity, fix);
+    }
     downportRef(high, lowFile, highSyntax) {
         let found = undefined;
         for (const s of high.findAllExpressionsRecursive(Expressions.Source)) {

package/build/src/rules/select_add_order_by.js

@@ -60,8 +60,8 @@ If the target is a sorted/hashed table, no issue is reported`,
                     continue;
                 }
                 // skip COUNT(*)
-                const list = s.findFirstExpression(Expressions.SQLFieldList);
-                if ((list === null || list === void 0 ? void 0 : list.getChildren().length) === 1 && ((_a = list.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.SQLAggregation) {
+                const list = s.findAllExpressions(Expressions.SQLField);
+                if (list.length === 1 && ((_a = list[0].getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.SQLAggregation) {
                     continue;
                 }
                 else if (s.findFirstExpression(Expressions.SQLOrderBy)) {

package/package.json

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