@abaplint/core 2.115.26 → 2.115.28

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

@@ -245,7 +245,7 @@ class Select {
         }
     }
     static buildStructureType(fields, dbSources, scope) {
-        var _a, _b, _c;
+        var _a, _b, _c, _d, _e, _f;
         if (fields.length === 1 && dbSources.length === 1) {
             const dbType = (_a = dbSources[0]) === null || _a === void 0 ? void 0 : _a.parseType(scope.getRegistry());
             if (dbType === undefined) {
@@ -286,8 +286,36 @@ class Select {
                 }
             }
         }
+        else if (dbSources.length === 1) {
+            const dbType = (_d = dbSources[0]) === null || _d === void 0 ? void 0 : _d.parseType(scope.getRegistry());
+            if (dbType === undefined) {
+                const name = ((_e = dbSources[0]) === null || _e === void 0 ? void 0 : _e.getName()) || "buildStructureTypeError";
+                if (scope.getRegistry().inErrorNamespace(name) === true) {
+                    return new basic_1.UnknownType("Select, " + name + " not found");
+                }
+                else {
+                    return basic_1.VoidType.get((_f = dbSources[0]) === null || _f === void 0 ? void 0 : _f.getName());
+                }
+            }
+            if (!(dbType instanceof basic_1.StructureType)) {
+                return basic_1.VoidType.get("SELECT_todo14");
+            }
+            const allFieldsSimple = fields.every(f => isSimple.test(f.code));
+            if (allFieldsSimple === true) {
+                const components = [];
+                for (const field of fields) {
+                    const type = dbType.getComponentByName(field.code);
+                    if (type === undefined) {
+                        return basic_1.VoidType.get("SELECT_todo9");
+                    }
+                    components.push({ name: field.as || field.code, type });
+                }
+                return new basic_1.StructureType(components);
+            }
+            return basic_1.VoidType.get("SELECT_todo12");
+        }
         else {
-            return basic_1.VoidType.get("SELECT_todo9");
+            return basic_1.VoidType.get("SELECT_todo13");
         }
     }
     static buildTableType(fields, dbSources, scope) {

package/build/src/cds/expressions/cds_condition.js

@@ -7,10 +7,13 @@ const cds_integer_1 = require("./cds_integer");
 class CDSCondition extends combi_1.Expression {
     getRunnable() {
         const left = (0, combi_1.altPrio)(_1.CDSString, _1.CDSFunction, _1.CDSAggregate, _1.CDSPrefixedName);
-        const operators = (0, combi_1.altPrio)("=", (0, combi_1.seq)("!", "="), (0, combi_1.seq)("<", ">"), (0, combi_1.seq)(">", "="), (0, combi_1.seq)("<", "="), "<", ">", "LIKE", "NOT LIKE");
-        // Right side of comparison: arithmetic expressions, parenthesized sub-expressions, or simple values
-        const right = (0, combi_1.altPrio)(_1.CDSArithmetics, _1.CDSArithParen, left, cds_integer_1.CDSInteger);
-        const compare = (0, combi_1.seq)(operators, right);
+        const nonLikeOperators = (0, combi_1.altPrio)("=", (0, combi_1.seq)("!", "="), (0, combi_1.seq)("<", ">"), (0, combi_1.seq)(">", "="), (0, combi_1.seq)("<", "="), "<", ">");
+        const likeOperators = (0, combi_1.altPrio)("LIKE", "NOT LIKE");
+        // Right side of comparison: simple values first, then parenthesized, then full arithmetic last.
+        // CDSArithmetics is last to avoid triggering CDSPrefixedName→CDSCondition→CDSArithmetics cycle.
+        const right = (0, combi_1.altPrio)(_1.CDSArithParen, left, cds_integer_1.CDSInteger, _1.CDSArithmetics);
+        // ESCAPE is only valid with LIKE/NOT LIKE, not with other comparison operators.
+        const compare = (0, combi_1.altPrio)((0, combi_1.seq)(likeOperators, right, (0, combi_1.opt)((0, combi_1.seq)("ESCAPE", _1.CDSString))), (0, combi_1.seq)(nonLikeOperators, right));
         const is = (0, combi_1.seq)("IS", (0, combi_1.optPrio)("NOT"), (0, combi_1.altPrio)("INITIAL", "NULL"));
         const between = (0, combi_1.seq)("BETWEEN", left, "AND", left);
         const condition = (0, combi_1.seq)((0, combi_1.optPrio)("NOT"), left, (0, combi_1.altPrio)(compare, is, between));

package/build/src/cds/expressions/cds_prefixed_name.js

@@ -25,7 +25,7 @@ class CDSPrefixedName extends combi_1.Expression {
         // The final segment may also be a string literal: #enum.'value'
         // A segment may have a parameterized call: _Assoc( P_Key : value ) or _Assoc[filter]
         const segment = (0, combi_1.seq)(".", (0, combi_1.altPrio)(cds_string_1.CDSString, cds_name_1.CDSName), (0, combi_1.opt)((0, combi_1.altPrio)(cds_parameters_select_1.CDSParametersSelect, cds_parameters_1.CDSParameters)), (0, combi_1.opt)(pathFilter));
-        return (0, combi_1.seq)(cds_name_1.CDSName, (0, combi_1.opt)((0, combi_1.altPrio)(cds_parameters_1.CDSParameters, cds_parameters_select_1.CDSParametersSelect)), (0, combi_1.opt)(pathFilter), (0, combi_1.star)(segment));
+        return (0, combi_1.seq)(cds_name_1.CDSName, (0, combi_1.opt)((0, combi_1.altPrio)(cds_parameters_1.CDSParameters, cds_parameters_select_1.CDSParametersSelect)), (0, combi_1.opt)(pathFilter), (0, combi_1.starPrio)(segment));
     }
 }
 exports.CDSPrefixedName = CDSPrefixedName;

package/build/src/cds/expressions/cds_select.js

@@ -11,7 +11,7 @@ class CDSSelect extends combi_1.Expression {
         const distinct = (0, combi_1.str)("DISTINCT");
         const elementList = (0, combi_1.seq)(_1.CDSElement, (0, combi_1.starPrio)((0, combi_1.seq)(",", _1.CDSElement)));
         const elements = (0, combi_1.seq)((0, combi_1.str)("{"), (0, combi_1.altPrio)("*", elementList), (0, combi_1.str)("}"));
-        return (0, combi_1.seq)("SELECT", (0, combi_1.optPrio)(distinct), (0, combi_1.opt)((0, combi_1.alt)("*", fields)), "FROM", _1.CDSSource, (0, combi_1.star)(cds_join_1.CDSJoin), (0, combi_1.star)((0, combi_1.alt)(_1.CDSComposition, cds_association_1.CDSAssociation)), (0, combi_1.opt)(elements), (0, combi_1.optPrio)(_1.CDSWhere), (0, combi_1.optPrio)(_1.CDSGroupBy), (0, combi_1.optPrio)(_1.CDSHaving), (0, combi_1.optPrio)((0, combi_1.seq)("UNION", (0, combi_1.optPrio)("ALL"), CDSSelect)));
+        return (0, combi_1.seq)("SELECT", (0, combi_1.optPrio)(distinct), (0, combi_1.opt)((0, combi_1.altPrio)("*", fields)), "FROM", _1.CDSSource, (0, combi_1.star)(cds_join_1.CDSJoin), (0, combi_1.star)((0, combi_1.altPrio)(_1.CDSComposition, cds_association_1.CDSAssociation)), (0, combi_1.opt)(elements), (0, combi_1.optPrio)(_1.CDSWhere), (0, combi_1.optPrio)(_1.CDSGroupBy), (0, combi_1.optPrio)(_1.CDSHaving), (0, combi_1.optPrio)((0, combi_1.seq)("UNION", (0, combi_1.optPrio)("ALL"), CDSSelect)));
     }
 }
 exports.CDSSelect = CDSSelect;

package/build/src/registry.js

@@ -74,7 +74,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.115.26";
+        return "2.115.28";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/modify_only_own_db_tables.js

@@ -26,6 +26,8 @@ class ModifyOnlyOwnDBTables {
             key: "modify_only_own_db_tables",
             title: "Modify only own DB tables",
             shortDescription: `Modify only own DB tables`,
+            badExample: `DELETE FROM mara WHERE matnr = '123456'.`,
+            goodExample: `DELETE FROM yflight WHERE carrid = 'LH'.`,
             extendedInformation: `https://docs.abapopenchecks.org/checks/26/`,
             tags: [_irule_1.RuleTag.Security],
         };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.115.26",
+    "version": "2.115.28",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -50,14 +50,14 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@microsoft/api-extractor": "^7.57.5",
+        "@microsoft/api-extractor": "^7.57.6",
         "@types/chai": "^4.3.20",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.10.13",
+        "@types/node": "^24.11.0",
         "chai": "^4.5.0",
         "eslint": "^9.39.2",
         "mocha": "^11.7.5",
-        "c8": "^10.1.3",
+        "c8": "^11.0.0",
         "source-map-support": "^0.5.21",
         "ts-json-schema-generator": "=2.4.0",
         "typescript": "^5.9.3"