@abaplint/cli 2.119.29 → 2.119.30

package/build/cli.js

@@ -1541,6 +1541,179 @@ exports.WStaticArrowW = WStaticArrowW;
 
 /***/ },
 
+/***/ "../core/build/src/abap/2_statements/_select_reclassify.js"
+/*!*****************************************************************!*\
+  !*** ../core/build/src/abap/2_statements/_select_reclassify.js ***!
+  \*****************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.reclassifySelect = reclassifySelect;
+const Statements = __importStar(__webpack_require__(/*! ./statements */ "../core/build/src/abap/2_statements/statements/index.js"));
+const Expressions = __importStar(__webpack_require__(/*! ./expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const nodes_1 = __webpack_require__(/*! ../nodes */ "../core/build/src/abap/nodes/index.js");
+const nodes_2 = __webpack_require__(/*! ../nodes */ "../core/build/src/abap/nodes/index.js");
+const expression_node_1 = __webpack_require__(/*! ../nodes/expression_node */ "../core/build/src/abap/nodes/expression_node.js");
+function containsAggregation(children) {
+    for (const child of children) {
+        if (!(child instanceof expression_node_1.ExpressionNode)) {
+            continue;
+        }
+        if (child.get() instanceof Expressions.SQLAggregation) {
+            return true;
+        }
+        if (child.get() instanceof Expressions.SQLSetOpGroup) {
+            continue;
+        }
+        if (containsAggregation(child.getChildren())) {
+            return true;
+        }
+    }
+    return false;
+}
+function isCountAllPattern(children) {
+    for (const child of children) {
+        if (!(child instanceof expression_node_1.ExpressionNode)) {
+            continue;
+        }
+        const e = child.get();
+        if (e instanceof Expressions.SQLGroupBy ||
+            e instanceof Expressions.SQLHaving ||
+            e instanceof Expressions.SQLOrderBy) {
+            return false;
+        }
+    }
+    return containsAggregation(children);
+}
+function scanForExprTypes(children, found, targets) {
+    for (const child of children) {
+        if (found.size === targets.length) {
+            return;
+        }
+        if (!(child instanceof expression_node_1.ExpressionNode)) {
+            continue;
+        }
+        const e = child.get();
+        if (e instanceof Expressions.SQLSetOpGroup) {
+            continue;
+        }
+        for (const t of targets) {
+            if (e instanceof t) {
+                found.add(t);
+                break;
+            }
+        }
+        scanForExprTypes(child.getChildren(), found, targets);
+    }
+}
+function isSelectLoop(node) {
+    const selectExpr = node.findDirectExpression(Expressions.Select);
+    const top = selectExpr ? selectExpr.getChildren() : [];
+    if (top.some(c => c instanceof nodes_2.TokenNode && c.get().getStr().toUpperCase() === "SINGLE")) {
+        return false;
+    }
+    const targets = [
+        Expressions.SQLSetOp, Expressions.SQLPackageSize, Expressions.SQLIntoTable,
+        Expressions.SQLIntoStructure, Expressions.SQLIntoList, Expressions.SQLGroupBy,
+    ];
+    const found = new Set();
+    scanForExprTypes(top, found, targets);
+    const has = (t) => found.has(t);
+    if (has(Expressions.SQLPackageSize)) {
+        return true;
+    }
+    if (has(Expressions.SQLIntoTable)) {
+        return false;
+    }
+    const hasInto = has(Expressions.SQLIntoTable) || has(Expressions.SQLIntoStructure) || has(Expressions.SQLIntoList);
+    if (!hasInto && isCountAllPattern(top)) {
+        return false;
+    }
+    if (has(Expressions.SQLSetOp)) {
+        return true;
+    }
+    if (!has(Expressions.SQLGroupBy) && containsAggregation(top)) {
+        return false;
+    }
+    return true;
+}
+function isWithLoop(node) {
+    const selectExpr = node.findDirectExpression(Expressions.Select);
+    if (!selectExpr) {
+        return true;
+    }
+    const targets = [Expressions.SQLPackageSize, Expressions.SQLIntoTable];
+    const found = new Set();
+    scanForExprTypes(selectExpr.getChildren(), found, targets);
+    if (found.has(Expressions.SQLPackageSize)) {
+        return true;
+    }
+    if (found.has(Expressions.SQLIntoTable)) {
+        return false;
+    }
+    return true;
+}
+function reclassifySelect(node, stmt, pragmas) {
+    const children = [...node.getChildren()];
+    if (stmt instanceof Statements.Select || stmt instanceof Statements.SelectLoop) {
+        const loop = isSelectLoop(node);
+        if (loop && stmt instanceof Statements.Select) {
+            return new nodes_1.StatementNode(new Statements.SelectLoop(), node.getColon(), pragmas).setChildren(children);
+        }
+        else if (!loop && stmt instanceof Statements.SelectLoop) {
+            return new nodes_1.StatementNode(new Statements.Select(), node.getColon(), pragmas).setChildren(children);
+        }
+    }
+    else if (stmt instanceof Statements.With || stmt instanceof Statements.WithLoop) {
+        const loop = isWithLoop(node);
+        if (loop && stmt instanceof Statements.With) {
+            return new nodes_1.StatementNode(new Statements.WithLoop(), node.getColon(), pragmas).setChildren(children);
+        }
+        else if (!loop && stmt instanceof Statements.WithLoop) {
+            return new nodes_1.StatementNode(new Statements.With(), node.getColon(), pragmas).setChildren(children);
+        }
+    }
+    return node;
+}
+//# sourceMappingURL=_select_reclassify.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/2_statements/combi.js"
 /*!****************************************************!*\
   !*** ../core/build/src/abap/2_statements/combi.js ***!
@@ -1601,6 +1774,7 @@ exports.ver = ver;
 exports.verNot = verNot;
 exports.failCombinator = failCombinator;
 exports.failStar = failStar;
+exports.stopBefore = stopBefore;
 const Tokens = __importStar(__webpack_require__(/*! ../1_lexer/tokens */ "../core/build/src/abap/1_lexer/tokens/index.js"));
 const nodes_1 = __webpack_require__(/*! ../nodes */ "../core/build/src/abap/nodes/index.js");
 const version_1 = __webpack_require__(/*! ../../version */ "../core/build/src/version.js");
@@ -2493,6 +2667,36 @@ function failCombinator() {
 function failStar() {
     return new FailStar();
 }
+// Passes through results where the next two tokens do NOT match t1 followed by t2.
+class StopBefore2 {
+    constructor(t1, t2) {
+        this.t1 = t1.toUpperCase();
+        this.t2 = t2.toUpperCase();
+    }
+    listKeywords() { return []; }
+    getUsing() { return []; }
+    run(r) {
+        var _a;
+        const result = [];
+        for (const input of r) {
+            const next = input.peek();
+            if (next === undefined) {
+                continue;
+            }
+            if (next.getUpperStr() === this.t1 && ((_a = input.peekAt(1)) === null || _a === void 0 ? void 0 : _a.getUpperStr()) === this.t2) {
+                continue;
+            }
+            result.push(input);
+        }
+        return result;
+    }
+    railroad() { return "Railroad.Terminal('stopBefore(" + this.t1 + " " + this.t2 + ")')"; }
+    toStr() { return "stopBefore(" + this.t1 + "," + this.t2 + ")"; }
+    first() { return [""]; }
+}
+function stopBefore(t1, t2) {
+    return new StopBefore2(t1, t2);
+}
 //# sourceMappingURL=combi.js.map
 
 /***/ },
@@ -2769,6 +2973,59 @@ exports.ExpandMacros = ExpandMacros;
 
 /***/ },
 
+/***/ "../core/build/src/abap/2_statements/expressions/_select_core.js"
+/*!***********************************************************************!*\
+  !*** ../core/build/src/abap/2_statements/expressions/_select_core.js ***!
+  \***********************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.buildSelectCore = buildSelectCore;
+const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
+const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
+const sql_group_by_1 = __webpack_require__(/*! ./sql_group_by */ "../core/build/src/abap/2_statements/expressions/sql_group_by.js");
+const sql_into_structure_1 = __webpack_require__(/*! ./sql_into_structure */ "../core/build/src/abap/2_statements/expressions/sql_into_structure.js");
+const sql_up_to_1 = __webpack_require__(/*! ./sql_up_to */ "../core/build/src/abap/2_statements/expressions/sql_up_to.js");
+function buildSelectCore(into, allowOrderBy = true) {
+    const where = (0, combi_1.seq)("WHERE", _1.SQLCond);
+    const offset = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("OFFSET", _1.SQLSource));
+    const sqlFields = (0, combi_1.ver)(version_1.Version.v750, _1.SQLFields, version_1.Version.OpenABAP);
+    const privileged = (0, combi_1.ver)(version_1.Version.v758, _1.SQLPrivilegedAccess);
+    const fieldList = (0, combi_1.optPrio)(_1.SQLFieldList);
+    const client = (0, combi_1.optPrio)(_1.SQLClient);
+    const byp = (0, combi_1.optPrio)(_1.SQLBypassingBuffer);
+    const fae = (0, combi_1.optPrio)(_1.SQLForAllEntries);
+    const whereClause = (0, combi_1.optPrio)(where);
+    const groupHaving = (0, combi_1.seq)((0, combi_1.optPrio)(sql_group_by_1.SQLGroupBy), (0, combi_1.optPrio)(_1.SQLHaving));
+    const orderUpOff = allowOrderBy
+        ? [(0, combi_1.optPrio)((0, combi_1.seq)(_1.SQLOrderBy, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), (0, combi_1.optPrio)(offset)))]
+        : [];
+    const trailingOpts = (0, combi_1.seq)((0, combi_1.optPrio)(_1.DatabaseConnection), (0, combi_1.optPrio)(_1.SQLHints), (0, combi_1.optPrio)(privileged), (0, combi_1.optPrio)(_1.SQLOptions));
+    const intoSingle = (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList);
+    const intoAny = into ? (0, combi_1.altPrio)(into, intoSingle) : intoSingle;
+    const intoForPackSize = _1.SQLIntoTable;
+    if (!into) {
+        const afterFromNoInto = (0, combi_1.seq)(_1.SQLFrom, client, byp, (0, combi_1.altPrio)((0, combi_1.seq)(sqlFields, fae, whereClause, groupHaving, ...orderUpOff, trailingOpts), (0, combi_1.seq)(fae, whereClause, groupHaving, ...orderUpOff, trailingOpts)));
+        const singleBody = (0, combi_1.seq)(_1.SQLFrom, client, byp, whereClause, groupHaving, trailingOpts);
+        return (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)("SINGLE", (0, combi_1.optPrio)("FOR UPDATE"), fieldList, singleBody), (0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), fieldList, byp, afterFromNoInto)));
+    }
+    const fromPackSize = (0, combi_1.seq)((0, combi_1.optPrio)(_1.SQLPackageSize), (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), byp, (0, combi_1.optPrio)(_1.DatabaseConnection));
+    const trailingInto = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.seq)(intoForPackSize, (0, combi_1.optPrio)(_1.SQLPackageSize), (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), byp, (0, combi_1.optPrio)(_1.SQLOrderBy), (0, combi_1.optPrio)(_1.SQLOptions))), (0, combi_1.optPrio)((0, combi_1.seq)(intoSingle, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), byp, (0, combi_1.optPrio)(_1.SQLOptions))));
+    const afterFromWithInto = (0, combi_1.seq)((0, combi_1.optPrio)(_1.SQLFrom), client, byp, fromPackSize, (0, combi_1.altPrio)((0, combi_1.seq)(sqlFields, fae, whereClause, groupHaving, ...orderUpOff, trailingOpts, trailingInto), (0, combi_1.seq)(intoForPackSize, (0, combi_1.optPrio)(_1.SQLPackageSize), (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), byp, fae, whereClause, groupHaving, ...orderUpOff, trailingOpts, (0, combi_1.optPrio)(_1.SQLOptions)), (0, combi_1.seq)(intoSingle, byp, fae, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), byp, whereClause, groupHaving, ...orderUpOff, trailingOpts, (0, combi_1.optPrio)(_1.SQLOptions)), (0, combi_1.seq)(fae, whereClause, groupHaving, ...orderUpOff, trailingOpts, trailingInto)));
+    const selectTableIntoThenFrom = (0, combi_1.seq)(intoForPackSize, (0, combi_1.optPrio)(_1.SQLPackageSize), (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), afterFromWithInto);
+    const selectOtherIntoThenFrom = (0, combi_1.seq)(intoAny, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), byp, afterFromWithInto);
+    const nonSingleBody = (0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), fieldList, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), byp, (0, combi_1.altPrio)(selectTableIntoThenFrom, selectOtherIntoThenFrom, afterFromWithInto));
+    const singleAfterFrom = (0, combi_1.seq)(_1.SQLFrom, client, byp, (0, combi_1.altPrio)((0, combi_1.seq)(sqlFields, whereClause, groupHaving, trailingOpts, (0, combi_1.optPrio)(intoSingle)), (0, combi_1.seq)(intoSingle, byp, whereClause, groupHaving, trailingOpts), (0, combi_1.seq)(whereClause, groupHaving, trailingOpts, (0, combi_1.optPrio)(intoSingle))));
+    const singleIntoBeforeFrom = (0, combi_1.seq)(intoSingle, _1.SQLFrom, client, byp, whereClause, groupHaving, trailingOpts);
+    return (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)("SINGLE", (0, combi_1.optPrio)("FOR UPDATE"), fieldList, byp, (0, combi_1.altPrio)(singleIntoBeforeFrom, singleAfterFrom)), nonSingleBody));
+}
+//# sourceMappingURL=_select_core.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/2_statements/expressions/abstract.js"
 /*!*******************************************************************!*\
   !*** ../core/build/src/abap/2_statements/expressions/abstract.js ***!
@@ -4884,7 +5141,7 @@ __exportStar(__webpack_require__(/*! ./redefinition */ "../core/build/src/abap/2
 __exportStar(__webpack_require__(/*! ./reduce_body */ "../core/build/src/abap/2_statements/expressions/reduce_body.js"), exports);
 __exportStar(__webpack_require__(/*! ./reduce_next */ "../core/build/src/abap/2_statements/expressions/reduce_next.js"), exports);
 __exportStar(__webpack_require__(/*! ./report_name */ "../core/build/src/abap/2_statements/expressions/report_name.js"), exports);
-__exportStar(__webpack_require__(/*! ./select_loop */ "../core/build/src/abap/2_statements/expressions/select_loop.js"), exports);
+__exportStar(__webpack_require__(/*! ./select_cte */ "../core/build/src/abap/2_statements/expressions/select_cte.js"), exports);
 __exportStar(__webpack_require__(/*! ./select */ "../core/build/src/abap/2_statements/expressions/select.js"), exports);
 __exportStar(__webpack_require__(/*! ./simple_field_chain */ "../core/build/src/abap/2_statements/expressions/simple_field_chain.js"), exports);
 __exportStar(__webpack_require__(/*! ./simple_field_chain2 */ "../core/build/src/abap/2_statements/expressions/simple_field_chain2.js"), exports);
@@ -4928,9 +5185,11 @@ __exportStar(__webpack_require__(/*! ./sql_hints */ "../core/build/src/abap/2_st
 __exportStar(__webpack_require__(/*! ./sql_in */ "../core/build/src/abap/2_statements/expressions/sql_in.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_into_list */ "../core/build/src/abap/2_statements/expressions/sql_into_list.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_into_structure */ "../core/build/src/abap/2_statements/expressions/sql_into_structure.js"), exports);
+__exportStar(__webpack_require__(/*! ./sql_field_list_loop_greedy */ "../core/build/src/abap/2_statements/expressions/sql_field_list_loop_greedy.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_into_table */ "../core/build/src/abap/2_statements/expressions/sql_into_table.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_join */ "../core/build/src/abap/2_statements/expressions/sql_join.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_options */ "../core/build/src/abap/2_statements/expressions/sql_options.js"), exports);
+__exportStar(__webpack_require__(/*! ./sql_package_size */ "../core/build/src/abap/2_statements/expressions/sql_package_size.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_order_by */ "../core/build/src/abap/2_statements/expressions/sql_order_by.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_over */ "../core/build/src/abap/2_statements/expressions/sql_over.js"), exports);
 __exportStar(__webpack_require__(/*! ./sql_over */ "../core/build/src/abap/2_statements/expressions/sql_over.js"), exports);
@@ -6610,13 +6869,13 @@ const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_st
 const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
 const sql_into_structure_1 = __webpack_require__(/*! ./sql_into_structure */ "../core/build/src/abap/2_statements/expressions/sql_into_structure.js");
-const select_core_1 = __webpack_require__(/*! ./select_core */ "../core/build/src/abap/2_statements/expressions/select_core.js");
+const _select_core_1 = __webpack_require__(/*! ./_select_core */ "../core/build/src/abap/2_statements/expressions/_select_core.js");
 class Select extends combi_1.Expression {
     getRunnable() {
         const into = (0, combi_1.altPrio)(_1.SQLIntoTable, sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList);
-        const standalone = (0, select_core_1.buildSelectCore)(into);
+        const standalone = (0, _select_core_1.buildSelectCore)(into);
         const unionTail = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.plusPrio)(_1.SQLSetOp), version_1.Version.OpenABAP);
-        const chained = (0, combi_1.seq)((0, select_core_1.buildSelectCore)(undefined, false), unionTail, (0, combi_1.optPrio)(_1.SQLOrderBy), (0, combi_1.optPrio)(into));
+        const chained = (0, combi_1.seq)((0, _select_core_1.buildSelectCore)(undefined, false), unionTail, (0, combi_1.optPrio)(_1.SQLOrderBy), (0, combi_1.optPrio)(into));
         return (0, combi_1.altPrio)(chained, standalone);
     }
 }
@@ -6625,86 +6884,48 @@ exports.Select = Select;
 
 /***/ },
 
-/***/ "../core/build/src/abap/2_statements/expressions/select_core.js"
-/*!**********************************************************************!*\
-  !*** ../core/build/src/abap/2_statements/expressions/select_core.js ***!
-  \**********************************************************************/
-(__unused_webpack_module, exports, __webpack_require__) {
-
-"use strict";
-
-Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.buildSelectCore = buildSelectCore;
-const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
-const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
-const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
-const sql_group_by_1 = __webpack_require__(/*! ./sql_group_by */ "../core/build/src/abap/2_statements/expressions/sql_group_by.js");
-const sql_into_structure_1 = __webpack_require__(/*! ./sql_into_structure */ "../core/build/src/abap/2_statements/expressions/sql_into_structure.js");
-const sql_up_to_1 = __webpack_require__(/*! ./sql_up_to */ "../core/build/src/abap/2_statements/expressions/sql_up_to.js");
-function buildSelectCore(into, allowOrderBy = true) {
-    const where = (0, combi_1.seq)("WHERE", _1.SQLCond);
-    const offset = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("OFFSET", _1.SQLSource));
-    const bypass = (0, combi_1.str)("BYPASSING BUFFER");
-    const fields = (0, combi_1.ver)(version_1.Version.v750, _1.SQLFields, version_1.Version.OpenABAP);
-    const privileged = (0, combi_1.ver)(version_1.Version.v758, _1.SQLPrivilegedAccess);
-    const orderByItems = allowOrderBy ? [_1.SQLOrderBy, sql_up_to_1.SQLUpTo, offset] : [];
-    const permItems = [_1.SQLFrom, ...(into ? [into] : []), _1.SQLForAllEntries, where,
-        ...orderByItems, _1.SQLClient, _1.SQLHaving,
-        bypass, sql_group_by_1.SQLGroupBy, fields, _1.DatabaseConnection, _1.SQLHints, privileged, _1.SQLOptions];
-    const perm = (0, combi_1.per)(permItems[0], permItems[1], ...permItems.slice(2));
-    const intoSingle = (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList);
-    const permSingleItems = [_1.SQLFrom, ...(into ? [intoSingle] : []), where, _1.SQLClient,
-        bypass, sql_group_by_1.SQLGroupBy, _1.SQLHaving, fields, _1.DatabaseConnection, _1.SQLHints, privileged, _1.SQLOptions];
-    const permSingle = (0, combi_1.per)(permSingleItems[0], permSingleItems[1], ...permSingleItems.slice(2));
-    const fieldList = (0, combi_1.optPrio)(_1.SQLFieldList);
-    const single = (0, combi_1.seq)("SINGLE", (0, combi_1.optPrio)("FOR UPDATE"), fieldList, permSingle);
-    const other = (0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), fieldList, perm);
-    return (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)(single, other));
-}
-//# sourceMappingURL=select_core.js.map
-
-/***/ },
-
-/***/ "../core/build/src/abap/2_statements/expressions/select_loop.js"
-/*!**********************************************************************!*\
-  !*** ../core/build/src/abap/2_statements/expressions/select_loop.js ***!
-  \**********************************************************************/
+/***/ "../core/build/src/abap/2_statements/expressions/select_cte.js"
+/*!*********************************************************************!*\
+  !*** ../core/build/src/abap/2_statements/expressions/select_cte.js ***!
+  \*********************************************************************/
 (__unused_webpack_module, exports, __webpack_require__) {
 
 "use strict";
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.SelectLoop = void 0;
+exports.SelectCTE = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
 const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
-const sql_order_by_1 = __webpack_require__(/*! ./sql_order_by */ "../core/build/src/abap/2_statements/expressions/sql_order_by.js");
-const sql_having_1 = __webpack_require__(/*! ./sql_having */ "../core/build/src/abap/2_statements/expressions/sql_having.js");
-const sql_into_structure_1 = __webpack_require__(/*! ./sql_into_structure */ "../core/build/src/abap/2_statements/expressions/sql_into_structure.js");
-const sql_hints_1 = __webpack_require__(/*! ./sql_hints */ "../core/build/src/abap/2_statements/expressions/sql_hints.js");
-const sql_field_list_loop_1 = __webpack_require__(/*! ./sql_field_list_loop */ "../core/build/src/abap/2_statements/expressions/sql_field_list_loop.js");
-const sql_up_to_1 = __webpack_require__(/*! ./sql_up_to */ "../core/build/src/abap/2_statements/expressions/sql_up_to.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
+const sql_field_list_loop_greedy_1 = __webpack_require__(/*! ./sql_field_list_loop_greedy */ "../core/build/src/abap/2_statements/expressions/sql_field_list_loop_greedy.js");
 const sql_fields_loop_1 = __webpack_require__(/*! ./sql_fields_loop */ "../core/build/src/abap/2_statements/expressions/sql_fields_loop.js");
-// note: SELECT loops are matched before single statement SELECTs
-class SelectLoop extends combi_1.Expression {
+const sql_up_to_1 = __webpack_require__(/*! ./sql_up_to */ "../core/build/src/abap/2_statements/expressions/sql_up_to.js");
+const sql_source_1 = __webpack_require__(/*! ./sql_source */ "../core/build/src/abap/2_statements/expressions/sql_source.js");
+const sql_set_op_group_1 = __webpack_require__(/*! ./sql_set_op_group */ "../core/build/src/abap/2_statements/expressions/sql_set_op_group.js");
+class SelectCTE extends combi_1.Expression {
     getRunnable() {
         const where = (0, combi_1.seq)("WHERE", _1.SQLCond);
         const bypass = "BYPASSING BUFFER";
-        const pack = (0, combi_1.seq)("PACKAGE SIZE", _1.SQLSource);
         const privileged = (0, combi_1.ver)(version_1.Version.v758, _1.SQLPrivilegedAccess);
-        const tab = (0, combi_1.seq)(_1.SQLIntoTable, (0, combi_1.alt)(pack, (0, combi_1.seq)(_1.SQLFrom, pack), (0, combi_1.seq)(pack, _1.SQLFrom)));
-        const packTab = (0, combi_1.seq)(pack, _1.SQLIntoTable);
-        const into = (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList);
-        const perm = (0, combi_1.per)(_1.SQLFrom, where, sql_up_to_1.SQLUpTo, sql_order_by_1.SQLOrderBy, sql_having_1.SQLHaving, _1.SQLClient, bypass, _1.SQLGroupBy, _1.SQLForAllEntries, _1.DatabaseConnection, privileged, _1.SQLOptions, (0, combi_1.alt)(tab, sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList, packTab));
-        const strict = (0, combi_1.seq)(_1.SQLFrom, (0, combi_1.ver)(version_1.Version.v750, sql_fields_loop_1.SQLFieldsLoop), (0, combi_1.optPrio)(_1.SQLForAllEntries), (0, combi_1.optPrio)((0, combi_1.seq)(where, (0, combi_1.optPrio)(sql_order_by_1.SQLOrderBy), into, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo))));
-        const aggrIntoBeforeFrom = (0, combi_1.seq)((0, combi_1.plusPrio)(_1.SQLAggregation), into, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), _1.SQLFrom, (0, combi_1.optPrio)(_1.SQLClient), (0, combi_1.optPrio)(where), _1.SQLGroupBy);
-        const aggrIntoAfterFrom = (0, combi_1.seq)((0, combi_1.plusPrio)(_1.SQLAggregation), _1.SQLFrom, (0, combi_1.optPrio)(_1.SQLClient), into, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), (0, combi_1.optPrio)(where), _1.SQLGroupBy);
-        const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)((0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_1.SQLFieldListLoop, perm), strict, aggrIntoBeforeFrom, aggrIntoAfterFrom), (0, combi_1.optPrio)(sql_hints_1.SQLHints), (0, combi_1.optPrio)(privileged), (0, combi_1.optPrio)(_1.SQLOptions));
-        return ret;
+        const offset = (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("OFFSET", sql_source_1.SQLSource));
+        const groupHaving = (0, combi_1.seq)((0, combi_1.optPrio)(_1.SQLGroupBy), (0, combi_1.optPrio)(_1.SQLHaving));
+        const tail = (0, combi_1.seq)(groupHaving, (0, combi_1.optPrio)((0, combi_1.seq)(_1.SQLOrderBy, (0, combi_1.optPrio)(sql_up_to_1.SQLUpTo), (0, combi_1.optPrio)(offset))), (0, combi_1.optPrio)(_1.SQLHints), (0, combi_1.optPrio)(privileged), (0, combi_1.optPrio)(_1.SQLOptions), (0, combi_1.optPrio)(_1.DatabaseConnection));
+        const sqlStyle = (0, combi_1.seq)(_1.SQLFrom, (0, combi_1.optPrio)(_1.SQLClient), (0, combi_1.optPrio)(bypass), (0, combi_1.ver)(version_1.Version.v750, sql_fields_loop_1.SQLFieldsLoop, version_1.Version.OpenABAP), (0, combi_1.optPrio)(_1.SQLForAllEntries), (0, combi_1.optPrio)(where), tail);
+        const abapStyle = (0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_greedy_1.SQLFieldListLoopGreedy, _1.SQLFrom, (0, combi_1.optPrio)(_1.SQLClient), (0, combi_1.optPrio)(bypass), (0, combi_1.optPrio)(_1.SQLForAllEntries), (0, combi_1.optPrio)(where), tail);
+        const union = (0, combi_1.seq)("UNION", (0, combi_1.optPrio)((0, combi_1.altPrio)("DISTINCT", "ALL")));
+        const intersectExcept = (0, combi_1.altPrio)((0, combi_1.seq)("INTERSECT", (0, combi_1.optPrio)("DISTINCT")), (0, combi_1.seq)("EXCEPT", (0, combi_1.optPrio)("DISTINCT")));
+        const setOp = (0, combi_1.altPrio)((0, combi_1.ver)(version_1.Version.v750, union, version_1.Version.OpenABAP), (0, combi_1.ver)(version_1.Version.v756, intersectExcept));
+        const operandSql = (0, combi_1.seq)(_1.SQLFrom, (0, combi_1.optPrio)(_1.SQLClient), (0, combi_1.optPrio)(bypass), (0, combi_1.ver)(version_1.Version.v750, sql_fields_loop_1.SQLFieldsLoop, version_1.Version.OpenABAP), (0, combi_1.optPrio)(_1.SQLForAllEntries), (0, combi_1.optPrio)(where), groupHaving);
+        const operandAbap = (0, combi_1.seq)((0, combi_1.optPrio)("DISTINCT"), sql_field_list_loop_greedy_1.SQLFieldListLoopGreedy, _1.SQLFrom, (0, combi_1.optPrio)(_1.SQLClient), (0, combi_1.optPrio)(bypass), (0, combi_1.optPrio)(_1.SQLForAllEntries), (0, combi_1.optPrio)(where), groupHaving);
+        const operandCore = (0, combi_1.altPrio)(operandSql, operandAbap);
+        const unionOperand = (0, combi_1.altPrio)(sql_set_op_group_1.SQLSetOpGroup, (0, combi_1.seq)("SELECT", operandCore));
+        const unionTail = (0, combi_1.starPrio)((0, combi_1.seq)(setOp, unionOperand));
+        const selectBody = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)(sqlStyle, abapStyle));
+        return (0, combi_1.seq)(selectBody, unionTail);
     }
 }
-exports.SelectLoop = SelectLoop;
-//# sourceMappingURL=select_loop.js.map
+exports.SelectCTE = SelectCTE;
+//# sourceMappingURL=select_cte.js.map
 
 /***/ },
 
@@ -7297,12 +7518,12 @@ class SQLCompare extends combi_1.Expression {
         const between = (0, combi_1.seq)("BETWEEN", _1.SQLSource, "AND", _1.SQLSource);
         const like = (0, combi_1.seq)("LIKE", _1.SQLSource, (0, combi_1.optPrio)((0, combi_1.seq)("ESCAPE", _1.SQLSource)));
         const nul = (0, combi_1.seq)("IS", (0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)("NULL", (0, combi_1.ver)(version_1.Version.v753, "INITIAL")));
-        const source = new _1.SQLSource();
         const sub = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.altPrio)("ALL", "ANY", "SOME")), subSelect);
+        const source = new _1.SQLSource();
         const arith = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/"), _1.SQLFieldName)), version_1.Version.OpenABAP);
         const paren = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), _1.Source, (0, combi_1.tok)(tokens_1.WParenRightW));
         const at = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), (0, combi_1.altPrio)(_1.SimpleSource3, paren)), version_1.Version.OpenABAP);
-        const rett = (0, combi_1.seq)((0, combi_1.altPrio)(_1.SQLAggregation, _1.SQLFunction, _1.ConstantString, (0, combi_1.seq)((0, combi_1.altPrio)(_1.SQLPath, _1.SQLFieldName), (0, combi_1.optPrio)(arith)), at), (0, combi_1.altPrio)((0, combi_1.seq)(_1.SQLCompareOperator, (0, combi_1.altPrio)(sub, _1.SQLAggregation, _1.SQLFunction, source)), (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)(_1.SQLIn, like, between)), nul));
+        const rett = (0, combi_1.seq)((0, combi_1.altPrio)(_1.SQLCase, _1.SQLAggregation, _1.SQLFunction, _1.ConstantString, (0, combi_1.seq)((0, combi_1.altPrio)(_1.SQLPath, _1.SQLFieldName), (0, combi_1.optPrio)(arith)), at), (0, combi_1.altPrio)((0, combi_1.seq)(_1.SQLCompareOperator, (0, combi_1.altPrio)(sub, _1.SQLCase, _1.SQLAggregation, _1.SQLFunction, (0, combi_1.seq)(source, (0, combi_1.optPrio)(arith)))), (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)(_1.SQLIn, like, between)), nul));
         const exists = (0, combi_1.seq)("EXISTS", subSelect);
         return (0, combi_1.altPrio)(exists, _1.Dynamic, rett);
     }
@@ -7387,16 +7608,20 @@ class SQLField extends combi_1.Expression {
         const field = (0, combi_1.altPrio)(_1.SQLAggregation, fieldNoAgg);
         const parenField = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), field, (0, combi_1.tok)(tokens_1.WParenRightW));
         const parenFieldNoAgg = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), fieldNoAgg, (0, combi_1.tok)(tokens_1.WParenRightW));
-        // arithmetic without aggregates as operands: from v740sp05
+        // without aggregates: from v740sp05
         const subNoAgg = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.altPrio)(parenFieldNoAgg, fieldNoAgg)));
         const arithNoAgg = (0, combi_1.ver)(version_1.Version.v740sp05, subNoAgg);
-        // arithmetic with aggregates as operands: from v754
+        // with aggregates: from v754
         const subWithAgg = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.altPrio)(parenField, field)));
         const arithWithAgg = (0, combi_1.ver)(version_1.Version.v754, subWithAgg);
         const arith = (0, combi_1.altPrio)(arithWithAgg, arithNoAgg);
         const arithSequence = (0, combi_1.seq)(field, (0, combi_1.optPrio)(arith));
         const parenArithSequence = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), arithSequence, (0, combi_1.tok)(tokens_1.WParenRightW));
-        return (0, combi_1.seq)((0, combi_1.altPrio)(parenArithSequence, arithSequence), (0, combi_1.optPrio)(as));
+        // allows (a-b)*(c-d) — paren groups as operands, defined after parenArithSequence
+        const subExtWithAgg = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.altPrio)(parenArithSequence, parenField, field)));
+        const subExtNoAgg = (0, combi_1.plusPrio)((0, combi_1.seq)((0, combi_1.altPrio)("+", "-", "*", "/", "&&"), (0, combi_1.altPrio)(parenArithSequence, parenFieldNoAgg, fieldNoAgg)));
+        const arithExt = (0, combi_1.altPrio)((0, combi_1.ver)(version_1.Version.v754, subExtWithAgg), (0, combi_1.ver)(version_1.Version.v740sp05, subExtNoAgg));
+        return (0, combi_1.seq)((0, combi_1.altPrio)((0, combi_1.seq)(parenArithSequence, (0, combi_1.optPrio)(arithExt)), arithSequence), (0, combi_1.optPrio)(as));
     }
 }
 exports.SQLField = SQLField;
@@ -7450,7 +7675,7 @@ class SQLFieldList extends combi_1.Expression {
         const as = (0, combi_1.seq)("AS", _1.SQLAsName);
         const commaParenField = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), _1.SQLFieldName, (0, combi_1.altPrio)((0, combi_1.tok)(tokens_1.WParenRightW), (0, combi_1.tok)(tokens_1.WParenRight)), (0, combi_1.optPrio)(as));
         const nev = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.starPrio)((0, combi_1.seq)(",", (0, combi_1.altPrio)(_1.SQLField, commaParenField))), version_1.Version.OpenABAP);
-        const old = (0, combi_1.starPrio)(_1.SQLField);
+        const old = (0, combi_1.starPrio)((0, combi_1.seq)((0, combi_1.stopBefore)("UP", "TO"), _1.SQLField));
         return (0, combi_1.altPrio)("*", _1.Dynamic, (0, combi_1.seq)(_1.SQLField, (0, combi_1.alt)(nev, old)));
     }
 }
@@ -7491,6 +7716,36 @@ exports.SQLFieldListLoop = SQLFieldListLoop;
 
 /***/ },
 
+/***/ "../core/build/src/abap/2_statements/expressions/sql_field_list_loop_greedy.js"
+/*!*************************************************************************************!*\
+  !*** ../core/build/src/abap/2_statements/expressions/sql_field_list_loop_greedy.js ***!
+  \*************************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLFieldListLoopGreedy = void 0;
+const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
+const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
+const sql_path_1 = __webpack_require__(/*! ./sql_path */ "../core/build/src/abap/2_statements/expressions/sql_path.js");
+const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "../core/build/src/abap/1_lexer/tokens/index.js");
+class SQLFieldListLoopGreedy extends combi_1.Expression {
+    getRunnable() {
+        const comma = (0, combi_1.opt)((0, combi_1.ver)(version_1.Version.v740sp05, ",", version_1.Version.OpenABAP));
+        const as = (0, combi_1.seq)("AS", _1.SQLAsName);
+        const abap = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WAt), _1.SimpleFieldChain2), version_1.Version.OpenABAP);
+        const entry = (0, combi_1.seq)((0, combi_1.altPrio)(_1.SQLField, abap, sql_path_1.SQLPath, _1.SQLFieldName, _1.Constant), (0, combi_1.optPrio)(as), comma);
+        const fieldList = (0, combi_1.plusPrio)(entry);
+        return (0, combi_1.altPrio)("*", _1.Dynamic, fieldList);
+    }
+}
+exports.SQLFieldListLoopGreedy = SQLFieldListLoopGreedy;
+//# sourceMappingURL=sql_field_list_loop_greedy.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/2_statements/expressions/sql_field_name.js"
 /*!*************************************************************************!*\
   !*** ../core/build/src/abap/2_statements/expressions/sql_field_name.js ***!
@@ -7504,7 +7759,7 @@ exports.SQLFieldName = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
 class SQLFieldName extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.regex)(/^(?!(?:SINGLE|INTO|DISTINCT|AS|WHERE|FOR|HAVING|APPENDING|UP|FROM)$)(\/\w+\/)?(\*?\w+~(\/\w+\/)?(\w+|\*)|\w+)$/i);
+        return (0, combi_1.regex)(/^(?!(?:FROM|INTO|HAVING|FOR|DISTINCT|UNION|INTERSECT|EXCEPT|NOT|WHEN|CASE|AS|APPENDING|WHERE|CONNECTION|EXISTS|INSERT|MODIFY)$)(\/\w+\/)?(\*?\w+~(\/\w+\/)?(\w+|\*)|\w+)$/i);
     }
 }
 exports.SQLFieldName = SQLFieldName;
@@ -7548,7 +7803,7 @@ const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_st
 const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
 class SQLFieldsLoop extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.seq)("FIELDS", (0, combi_1.opt)("DISTINCT"), _1.SQLFieldListLoop);
+        return (0, combi_1.seq)("FIELDS", (0, combi_1.opt)("DISTINCT"), _1.SQLFieldListLoopGreedy);
     }
 }
 exports.SQLFieldsLoop = SQLFieldsLoop;
@@ -7662,7 +7917,7 @@ const sql_function_input_1 = __webpack_require__(/*! ./sql_function_input */ "..
 const sql_case_1 = __webpack_require__(/*! ./sql_case */ "../core/build/src/abap/2_statements/expressions/sql_case.js");
 class SQLFunction extends combi_1.Expression {
     getRunnable() {
-        const castTypes = (0, combi_1.altPrio)((0, combi_1.seq)("CHAR", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), (0, combi_1.seq)("DEC", (0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, ",", integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)), (0, combi_1.seq)("NUMC", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), "DATS", "FLTP", "INT2", "INT4", "INT8");
+        const castTypes = (0, combi_1.altPrio)((0, combi_1.seq)("CHAR", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), (0, combi_1.seq)("DEC", (0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, ",", integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)), (0, combi_1.seq)("NUMC", (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), integer_1.Integer, (0, combi_1.tok)(tokens_1.WParenRightW)))), "DATS", "FLTP", "INT1", "INT2", "INT4", "INT8");
         const commaParam = (0, combi_1.seq)(",", sql_function_input_1.SQLFunctionInput);
         // note: the function names are not keywords, they are usually in lower case
         const abs = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.regex)(/^abs$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, (0, combi_1.tok)(tokens_1.WParenRightW)));
@@ -7670,7 +7925,7 @@ class SQLFunction extends combi_1.Expression {
         const castInput = (0, combi_1.altPrio)(sql_case_1.SQLCase, sql_function_input_1.SQLFunctionInput);
         const cast = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)((0, combi_1.regex)(/^cast$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), castInput, "AS", castTypes, (0, combi_1.tok)(tokens_1.WParenRightW)));
         const ceil = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.regex)(/^ceil$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, (0, combi_1.tok)(tokens_1.WParenRightW)));
-        const coalesce = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.regex)(/^coalesce$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, (0, combi_1.optPrio)(commaParam), (0, combi_1.tok)(tokens_1.WParenRightW)));
+        const coalesce = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.regex)(/^coalesce$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, (0, combi_1.starPrio)(commaParam), (0, combi_1.tok)(tokens_1.WParenRightW)));
         const concat = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)((0, combi_1.regex)(/^concat$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
         const div = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.regex)(/^div$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, commaParam, (0, combi_1.tok)(tokens_1.WParenRightW)));
         const floor = (0, combi_1.ver)(version_1.Version.v740sp05, (0, combi_1.seq)((0, combi_1.regex)(/^floor$/i), (0, combi_1.tok)(tokens_1.ParenLeftW), sql_function_input_1.SQLFunctionInput, (0, combi_1.tok)(tokens_1.WParenRightW)));
@@ -7927,11 +8182,13 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SQLJoin = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
 const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
+const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
 class SQLJoin extends combi_1.Expression {
     getRunnable() {
         const joinType = (0, combi_1.seq)((0, combi_1.optPrio)((0, combi_1.altPrio)("INNER", "LEFT OUTER", "LEFT", "RIGHT OUTER", "RIGHT")), "JOIN");
         const join = (0, combi_1.seq)(joinType, _1.SQLFromSource, "ON", _1.SQLCond);
-        return join;
+        const crossJoin = (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("CROSS JOIN", _1.SQLFromSource), version_1.Version.OpenABAP);
+        return (0, combi_1.altPrio)(crossJoin, join);
     }
 }
 exports.SQLJoin = SQLJoin;
@@ -8028,6 +8285,28 @@ exports.SQLOver = SQLOver;
 
 /***/ },
 
+/***/ "../core/build/src/abap/2_statements/expressions/sql_package_size.js"
+/*!***************************************************************************!*\
+  !*** ../core/build/src/abap/2_statements/expressions/sql_package_size.js ***!
+  \***************************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.SQLPackageSize = void 0;
+const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
+const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
+class SQLPackageSize extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)("PACKAGE SIZE", _1.SQLSource);
+    }
+}
+exports.SQLPackageSize = SQLPackageSize;
+//# sourceMappingURL=sql_package_size.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/2_statements/expressions/sql_path.js"
 /*!*******************************************************************!*\
   !*** ../core/build/src/abap/2_statements/expressions/sql_path.js ***!
@@ -8092,10 +8371,10 @@ exports.SQLSetOp = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
 const sql_set_op_group_1 = __webpack_require__(/*! ./sql_set_op_group */ "../core/build/src/abap/2_statements/expressions/sql_set_op_group.js");
-const select_core_1 = __webpack_require__(/*! ./select_core */ "../core/build/src/abap/2_statements/expressions/select_core.js");
+const _select_core_1 = __webpack_require__(/*! ./_select_core */ "../core/build/src/abap/2_statements/expressions/_select_core.js");
 class SQLSetOp extends combi_1.Expression {
     getRunnable() {
-        const operand = (0, combi_1.altPrio)(sql_set_op_group_1.SQLSetOpGroup, (0, select_core_1.buildSelectCore)(undefined, false));
+        const operand = (0, combi_1.altPrio)(sql_set_op_group_1.SQLSetOpGroup, (0, _select_core_1.buildSelectCore)(undefined, false));
         const union = (0, combi_1.seq)("UNION", (0, combi_1.optPrio)((0, combi_1.altPrio)("DISTINCT", "ALL")));
         const intersectExcept = (0, combi_1.altPrio)((0, combi_1.seq)("INTERSECT", (0, combi_1.optPrio)("DISTINCT")), (0, combi_1.seq)("EXCEPT", (0, combi_1.optPrio)("DISTINCT")));
         const op = (0, combi_1.altPrio)((0, combi_1.ver)(version_1.Version.v750, union, version_1.Version.OpenABAP), (0, combi_1.ver)(version_1.Version.v756, intersectExcept));
@@ -8120,10 +8399,10 @@ exports.SQLSetOpGroup = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
 const _1 = __webpack_require__(/*! . */ "../core/build/src/abap/2_statements/expressions/index.js");
 const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
-const select_core_1 = __webpack_require__(/*! ./select_core */ "../core/build/src/abap/2_statements/expressions/select_core.js");
+const _select_core_1 = __webpack_require__(/*! ./_select_core */ "../core/build/src/abap/2_statements/expressions/_select_core.js");
 class SQLSetOpGroup extends combi_1.Expression {
     getRunnable() {
-        const operand = (0, combi_1.altPrio)(SQLSetOpGroup, (0, select_core_1.buildSelectCore)(undefined, false));
+        const operand = (0, combi_1.altPrio)(SQLSetOpGroup, (0, _select_core_1.buildSelectCore)(undefined, false));
         const chain = (0, combi_1.seq)(operand, (0, combi_1.optPrio)((0, combi_1.seq)((0, combi_1.ver)(version_1.Version.v750, (0, combi_1.plusPrio)(_1.SQLSetOp), version_1.Version.OpenABAP), (0, combi_1.optPrio)(_1.SQLOrderBy))));
         return (0, combi_1.ver)(version_1.Version.v750, (0, combi_1.seq)("(", chain, ")"), version_1.Version.OpenABAP);
     }
@@ -9044,9 +9323,12 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.WithName = void 0;
 const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "../core/build/src/abap/1_lexer/tokens/index.js");
 const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
+const sql_field_name_1 = __webpack_require__(/*! ./sql_field_name */ "../core/build/src/abap/2_statements/expressions/sql_field_name.js");
 class WithName extends combi_1.Expression {
     getRunnable() {
-        return (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WPlus), (0, combi_1.regex)(/^\w+$/));
+        // ( has no leading space, ) has leading space
+        const colList = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.ParenLeftW), sql_field_name_1.SQLFieldName, (0, combi_1.star)((0, combi_1.seq)(",", sql_field_name_1.SQLFieldName)), (0, combi_1.tok)(tokens_1.WParenRightW));
+        return (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WPlus), (0, combi_1.regex)(/^\w+$/), (0, combi_1.optPrio)(colList));
     }
 }
 exports.WithName = WithName;
@@ -9106,6 +9388,9 @@ class Result {
     peek() {
         return this.tokens[this.tokenIndex];
     }
+    peekAt(offset) {
+        return this.tokens[this.tokenIndex + offset];
+    }
     shift(node) {
         const cp = this.nodes.slice();
         cp.push(node);
@@ -9181,7 +9466,8 @@ const combi_1 = __webpack_require__(/*! ./combi */ "../core/build/src/abap/2_sta
 const _statement_1 = __webpack_require__(/*! ./statements/_statement */ "../core/build/src/abap/2_statements/statements/_statement.js");
 const expand_macros_1 = __webpack_require__(/*! ./expand_macros */ "../core/build/src/abap/2_statements/expand_macros.js");
 const tokens_1 = __webpack_require__(/*! ../1_lexer/tokens */ "../core/build/src/abap/1_lexer/tokens/index.js");
-exports.STATEMENT_MAX_TOKENS = 1000;
+const _select_reclassify_1 = __webpack_require__(/*! ./_select_reclassify */ "../core/build/src/abap/2_statements/_select_reclassify.js");
+exports.STATEMENT_MAX_TOKENS = 20000;
 class StatementMap {
     constructor() {
         this.map = {};
@@ -9404,7 +9690,8 @@ class StatementParser {
             if (match) {
                 const last = tokens[tokens.length - 1];
                 match.push(new nodes_1.TokenNode(last));
-                return new nodes_1.StatementNode(st.statement, statement.getColon(), pragmas).setChildren(match);
+                const matched = new nodes_1.StatementNode(st.statement, statement.getColon(), pragmas).setChildren(match);
+                return (0, _select_reclassify_1.reclassifySelect)(matched, st.statement, pragmas);
             }
         }
         // next try the statements without specific keywords
@@ -9413,7 +9700,8 @@ class StatementParser {
             if (match) {
                 const last = tokens[tokens.length - 1];
                 match.push(new nodes_1.TokenNode(last));
-                return new nodes_1.StatementNode(st.statement, statement.getColon(), pragmas).setChildren(match);
+                const matched = new nodes_1.StatementNode(st.statement, statement.getColon(), pragmas).setChildren(match);
+                return (0, _select_reclassify_1.reclassifySelect)(matched, st.statement, pragmas);
             }
         }
         return statement;
@@ -16111,7 +16399,7 @@ exports.SelectLoop = void 0;
 const expressions_1 = __webpack_require__(/*! ../expressions */ "../core/build/src/abap/2_statements/expressions/index.js");
 class SelectLoop {
     getMatcher() {
-        return new expressions_1.SelectLoop();
+        return new expressions_1.Select();
     }
 }
 exports.SelectLoop = SelectLoop;
@@ -18027,8 +18315,8 @@ const expressions_1 = __webpack_require__(/*! ../expressions */ "../core/build/s
 const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "../core/build/src/abap/1_lexer/tokens/index.js");
 class With {
     getMatcher() {
-        const as = (0, combi_1.seq)(expressions_1.WithName, "AS", (0, combi_1.tok)(tokens_1.WParenLeftW), expressions_1.SelectLoop, (0, combi_1.tok)(tokens_1.WParenRightW));
-        return (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("WITH", (0, combi_1.plus)(as), expressions_1.Select));
+        const cte = (0, combi_1.seq)(expressions_1.WithName, "AS", (0, combi_1.tok)(tokens_1.WParenLeftW), expressions_1.SelectCTE, (0, combi_1.tok)(tokens_1.WParenRightW));
+        return (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("WITH", cte, (0, combi_1.star)((0, combi_1.seq)(",", cte)), expressions_1.Select));
     }
 }
 exports.With = With;
@@ -18046,14 +18334,10 @@ exports.With = With;
 
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.WithLoop = void 0;
-const combi_1 = __webpack_require__(/*! ../combi */ "../core/build/src/abap/2_statements/combi.js");
-const version_1 = __webpack_require__(/*! ../../../version */ "../core/build/src/version.js");
-const tokens_1 = __webpack_require__(/*! ../../1_lexer/tokens */ "../core/build/src/abap/1_lexer/tokens/index.js");
-const expressions_1 = __webpack_require__(/*! ../expressions */ "../core/build/src/abap/2_statements/expressions/index.js");
+const with_1 = __webpack_require__(/*! ./with */ "../core/build/src/abap/2_statements/statements/with.js");
 class WithLoop {
     getMatcher() {
-        const as = (0, combi_1.seq)(expressions_1.WithName, "AS", (0, combi_1.tok)(tokens_1.WParenLeftW), expressions_1.SelectLoop, (0, combi_1.tok)(tokens_1.WParenRightW));
-        return (0, combi_1.ver)(version_1.Version.v751, (0, combi_1.seq)("WITH", (0, combi_1.plus)(as), expressions_1.SelectLoop));
+        return new with_1.With().getMatcher();
     }
 }
 exports.WithLoop = WithLoop;
@@ -31482,15 +31766,7 @@ class Select {
         var _a, _b;
         let expr = undefined;
         const ret = [];
-        if (node.get() instanceof Expressions.SelectLoop) {
-            expr = node.findFirstExpression(Expressions.SQLFieldListLoop);
-            if (expr === undefined) {
-                expr = node;
-            }
-        }
-        else {
-            expr = node.findFirstExpression(Expressions.SQLFieldList);
-        }
+        expr = node.findFirstExpression(Expressions.SQLFieldList);
         if (((_a = expr === null || expr === void 0 ? void 0 : expr.getFirstChild()) === null || _a === void 0 ? void 0 : _a.get()) instanceof Expressions.Dynamic) {
             dynamic_1.Dynamic.runSyntax(expr.getFirstChild(), input);
         }
@@ -31514,28 +31790,6 @@ exports.Select = Select;
 
 /***/ },
 
-/***/ "../core/build/src/abap/5_syntax/expressions/select_loop.js"
-/*!******************************************************************!*\
-  !*** ../core/build/src/abap/5_syntax/expressions/select_loop.js ***!
-  \******************************************************************/
-(__unused_webpack_module, exports, __webpack_require__) {
-
-"use strict";
-
-Object.defineProperty(exports, "__esModule", ({ value: true }));
-exports.SelectLoop = void 0;
-const select_1 = __webpack_require__(/*! ./select */ "../core/build/src/abap/5_syntax/expressions/select.js");
-class SelectLoop {
-    static runSyntax(node, input) {
-        // try using the other Select, they should look very much the same
-        select_1.Select.runSyntax(node, input);
-    }
-}
-exports.SelectLoop = SelectLoop;
-//# sourceMappingURL=select_loop.js.map
-
-/***/ },
-
 /***/ "../core/build/src/abap/5_syntax/expressions/source.js"
 /*!*************************************************************!*\
   !*** ../core/build/src/abap/5_syntax/expressions/source.js ***!
@@ -40705,6 +40959,67 @@ exports.Ranges = Ranges;
 
 /***/ },
 
+/***/ "../core/build/src/abap/5_syntax/statements/read_dataset.js"
+/*!******************************************************************!*\
+  !*** ../core/build/src/abap/5_syntax/statements/read_dataset.js ***!
+  \******************************************************************/
+(__unused_webpack_module, exports, __webpack_require__) {
+
+"use strict";
+
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.ReadDataset = void 0;
+const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
+const source_1 = __webpack_require__(/*! ../expressions/source */ "../core/build/src/abap/5_syntax/expressions/source.js");
+const target_1 = __webpack_require__(/*! ../expressions/target */ "../core/build/src/abap/5_syntax/expressions/target.js");
+class ReadDataset {
+    runSyntax(node, input) {
+        for (const s of node.findDirectExpressions(Expressions.Source)) {
+            source_1.Source.runSyntax(s, input);
+        }
+        for (const t of node.findDirectExpressions(Expressions.Target)) {
+            target_1.Target.runSyntax(t, input);
+        }
+    }
+}
+exports.ReadDataset = ReadDataset;
+//# sourceMappingURL=read_dataset.js.map
+
+/***/ },
+
 /***/ "../core/build/src/abap/5_syntax/statements/read_entities.js"
 /*!*******************************************************************!*\
   !*** ../core/build/src/abap/5_syntax/statements/read_entities.js ***!
@@ -41546,12 +41861,12 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SelectLoop = void 0;
 const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
-const select_loop_1 = __webpack_require__(/*! ../expressions/select_loop */ "../core/build/src/abap/5_syntax/expressions/select_loop.js");
+const select_1 = __webpack_require__(/*! ../expressions/select */ "../core/build/src/abap/5_syntax/expressions/select.js");
 class SelectLoop {
     runSyntax(node, input) {
-        const s = node.findDirectExpression(Expressions.SelectLoop);
+        const s = node.findDirectExpression(Expressions.Select);
         if (s) {
-            select_loop_1.SelectLoop.runSyntax(s, input);
+            select_1.Select.runSyntax(s, input);
         }
     }
 }
@@ -43653,15 +43968,11 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.With = void 0;
 const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
 const select_1 = __webpack_require__(/*! ../expressions/select */ "../core/build/src/abap/5_syntax/expressions/select.js");
-const select_loop_1 = __webpack_require__(/*! ../expressions/select_loop */ "../core/build/src/abap/5_syntax/expressions/select_loop.js");
 class With {
     runSyntax(node, input) {
         for (const s of node.findAllExpressions(Expressions.Select)) {
             select_1.Select.runSyntax(s, input);
         }
-        for (const s of node.findAllExpressions(Expressions.SelectLoop)) {
-            select_loop_1.SelectLoop.runSyntax(s, input);
-        }
     }
 }
 exports.With = With;
@@ -43714,15 +44025,11 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.WithLoop = void 0;
 const Expressions = __importStar(__webpack_require__(/*! ../../2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js"));
 const select_1 = __webpack_require__(/*! ../expressions/select */ "../core/build/src/abap/5_syntax/expressions/select.js");
-const select_loop_1 = __webpack_require__(/*! ../expressions/select_loop */ "../core/build/src/abap/5_syntax/expressions/select_loop.js");
 class WithLoop {
     runSyntax(node, input) {
         for (const s of node.findAllExpressions(Expressions.Select)) {
             select_1.Select.runSyntax(s, input);
         }
-        for (const s of node.findAllExpressions(Expressions.SelectLoop)) {
-            select_loop_1.SelectLoop.runSyntax(s, input);
-        }
     }
 }
 exports.WithLoop = WithLoop;
@@ -44707,6 +45014,7 @@ const insert_field_group_1 = __webpack_require__(/*! ./statements/insert_field_g
 const read_entities_1 = __webpack_require__(/*! ./statements/read_entities */ "../core/build/src/abap/5_syntax/statements/read_entities.js");
 const modify_entities_1 = __webpack_require__(/*! ./statements/modify_entities */ "../core/build/src/abap/5_syntax/statements/modify_entities.js");
 const commit_entities_1 = __webpack_require__(/*! ./statements/commit_entities */ "../core/build/src/abap/5_syntax/statements/commit_entities.js");
+const read_dataset_1 = __webpack_require__(/*! ./statements/read_dataset */ "../core/build/src/abap/5_syntax/statements/read_dataset.js");
 const _syntax_input_1 = __webpack_require__(/*! ./_syntax_input */ "../core/build/src/abap/5_syntax/_syntax_input.js");
 const assert_error_1 = __webpack_require__(/*! ./assert_error */ "../core/build/src/abap/5_syntax/assert_error.js");
 const field_group_1 = __webpack_require__(/*! ./statements/field_group */ "../core/build/src/abap/5_syntax/statements/field_group.js");
@@ -44835,6 +45143,7 @@ if (Object.keys(map).length === 0) {
     addToMap(new parameter_1.Parameter());
     addToMap(new fieldsymbol_1.FieldSymbol());
     addToMap(new read_entities_1.ReadEntities());
+    addToMap(new read_dataset_1.ReadDataset());
     addToMap(new modify_entities_1.ModifyEntities());
     addToMap(new commit_entities_1.CommitEntities());
     addToMap(new call_kernel_1.CallKernel());
@@ -65683,7 +65992,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.119.29";
+        return "2.119.30";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -72625,6 +72934,9 @@ Make sure to test the downported code, it might not always be completely correct
         if (where === undefined) {
             return undefined;
         }
+        if (high.findFirstExpression(Expressions.SQLFields) !== undefined) {
+            return undefined;
+        }
         let into = high.findFirstExpression(Expressions.SQLIntoList);
         if (into === undefined) {
             into = high.findFirstExpression(Expressions.SQLIntoStructure);
@@ -87047,11 +87359,13 @@ exports.PrefixIsCurrentClass = exports.PrefixIsCurrentClassConf = void 0;
 const issue_1 = __webpack_require__(/*! ../issue */ "../core/build/src/issue.js");
 const _abap_rule_1 = __webpack_require__(/*! ./_abap_rule */ "../core/build/src/rules/_abap_rule.js");
 const Structures = __importStar(__webpack_require__(/*! ../abap/3_structures/structures */ "../core/build/src/abap/3_structures/structures/index.js"));
+const Statements = __importStar(__webpack_require__(/*! ../abap/2_statements/statements */ "../core/build/src/abap/2_statements/statements/index.js"));
 const _basic_rule_config_1 = __webpack_require__(/*! ./_basic_rule_config */ "../core/build/src/rules/_basic_rule_config.js");
 const expressions_1 = __webpack_require__(/*! ../abap/2_statements/expressions */ "../core/build/src/abap/2_statements/expressions/index.js");
 const position_1 = __webpack_require__(/*! ../position */ "../core/build/src/position.js");
 const edit_helper_1 = __webpack_require__(/*! ../edit_helper */ "../core/build/src/edit_helper.js");
 const _irule_1 = __webpack_require__(/*! ./_irule */ "../core/build/src/rules/_irule.js");
+const comment_1 = __webpack_require__(/*! ../abap/1_lexer/tokens/comment */ "../core/build/src/abap/1_lexer/tokens/comment.js");
 class PrefixIsCurrentClassConf extends _basic_rule_config_1.BasicRuleConfig {
     constructor() {
         super(...arguments);
@@ -87127,11 +87441,16 @@ class PrefixIsCurrentClass extends _abap_rule_1.ABAPRule {
         for (const c of classStructures) {
             const className = c.findFirstExpression(expressions_1.ClassName).getFirstToken().getStr().toUpperCase();
             const staticAccess = className + "=>";
+            const shadowed = this.buildShadowedNames(struc, c, className);
             for (const s of c.findAllStatementNodes()) {
                 const concat = s.concatTokensWithoutStringsAndComments().toUpperCase();
                 if (concat.includes(staticAccess)) {
-                    const tokenPos = s.findTokenSequencePosition(className, "=>");
-                    if (tokenPos) {
+                    // when the referenced member is shadowed by a method parameter or local
+                    // declaration, the class prefix is required and cannot be omitted, see issues #3755 and #3707
+                    const names = shadowed.get(s);
+                    const ref = this.findStaticReferences(s, className).find(r => r.member === undefined || (names === null || names === void 0 ? void 0 : names.has(r.member)) !== true);
+                    if (ref) {
+                        const tokenPos = ref.pos;
                         const end = new position_1.Position(tokenPos.getRow(), tokenPos.getCol() + className.length + 2);
                         const fix = edit_helper_1.EditHelper.deleteRange(file, tokenPos, end);
                         issues.push(issue_1.Issue.atRange(file, tokenPos, end, "Reference to current class can be omitted: \"" + staticAccess + "\"", this.getMetadata().key, this.conf.severity, fix));
@@ -87151,6 +87470,62 @@ class PrefixIsCurrentClass extends _abap_rule_1.ABAPRule {
         }
         return issues;
     }
+    /** finds "className=>member" references in the statement, position is the start of the class name */
+    findStaticReferences(s, className) {
+        var _a;
+        const refs = [];
+        const tokens = s.getTokens().filter(t => !(t instanceof comment_1.Comment));
+        for (let i = 0; i < tokens.length - 1; i++) {
+            if (tokens[i].getStr().toUpperCase() === className
+                && tokens[i + 1].getStr() === "=>") {
+                refs.push({
+                    pos: tokens[i].getStart(),
+                    member: (_a = tokens[i + 2]) === null || _a === void 0 ? void 0 : _a.getStr().toUpperCase(),
+                });
+            }
+        }
+        return refs;
+    }
+    /** for each statement in a method implementation: the method parameter and local
+     *  declaration names that shadow class members of the same name */
+    buildShadowedNames(struc, impl, className) {
+        var _a, _b, _c;
+        const map = new Map();
+        if (!(impl.get() instanceof Structures.ClassImplementation)) {
+            return map;
+        }
+        const definition = struc.findDirectStructures(Structures.ClassDefinition).find(d => { var _a; return ((_a = d.findFirstExpression(expressions_1.ClassName)) === null || _a === void 0 ? void 0 : _a.getFirstToken().getStr().toUpperCase()) === className; });
+        for (const method of impl.findAllStructuresRecursive(Structures.Method)) {
+            const names = new Set();
+            const methodName = (_b = (_a = method.findFirstStatement(Statements.MethodImplementation)) === null || _a === void 0 ? void 0 : _a.findFirstExpression(expressions_1.MethodName)) === null || _b === void 0 ? void 0 : _b.concatTokens().toUpperCase();
+            if (definition !== undefined && methodName !== undefined) {
+                for (const def of definition.findAllStatements(Statements.MethodDef)) {
+                    if (((_c = def.findFirstExpression(expressions_1.MethodName)) === null || _c === void 0 ? void 0 : _c.concatTokens().toUpperCase()) === methodName) {
+                        for (const param of def.findAllExpressions(expressions_1.MethodParamName)) {
+                            names.add(param.concatTokens().toUpperCase());
+                        }
+                        break;
+                    }
+                }
+            }
+            for (const d of method.findAllExpressions(expressions_1.DefinitionName)) {
+                names.add(d.concatTokens().toUpperCase());
+            }
+            for (const inline of method.findAllExpressions(expressions_1.InlineData)) {
+                const field = inline.findFirstExpression(expressions_1.TargetField);
+                if (field) {
+                    names.add(field.concatTokens().toUpperCase());
+                }
+            }
+            if (names.size === 0) {
+                continue;
+            }
+            for (const statement of method.findAllStatementNodes()) {
+                map.set(statement, names);
+            }
+        }
+        return map;
+    }
 }
 exports.PrefixIsCurrentClass = PrefixIsCurrentClass;
 //# sourceMappingURL=prefix_is_current_class.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.119.29",
+    "version": "2.119.30",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.119.29",
+        "@abaplint/core": "^2.119.30",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",