@abaplint/cli 2.115.26 → 2.115.28

package/build/cli.js

@@ -28359,7 +28359,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) {
@@ -28400,8 +28400,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) {
@@ -41957,10 +41985,13 @@ const cds_integer_1 = __webpack_require__(/*! ./cds_integer */ "./node_modules/@
 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));
@@ -42471,7 +42502,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;
@@ -42543,7 +42574,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;
@@ -55645,7 +55676,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.115.26";
+        return "2.115.28";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -69938,6 +69969,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],
         };
@@ -96148,12 +96181,6 @@ var lib = JSON5;
 /******/ 		if (cachedModule !== undefined) {
 /******/ 			return cachedModule.exports;
 /******/ 		}
-/******/ 		// Check if module exists (development only)
-/******/ 		if (__webpack_modules__[moduleId] === undefined) {
-/******/ 			var e = new Error("Cannot find module '" + moduleId + "'");
-/******/ 			e.code = 'MODULE_NOT_FOUND';
-/******/ 			throw e;
-/******/ 		}
 /******/ 		// Create a new module (and put it into the cache)
 /******/ 		var module = __webpack_module_cache__[moduleId] = {
 /******/ 			// no module.id needed
@@ -96162,6 +96189,12 @@ var lib = JSON5;
 /******/ 		};
 /******/ 	
 /******/ 		// Execute the module function
+/******/ 		if (!(moduleId in __webpack_modules__)) {
+/******/ 			delete __webpack_module_cache__[moduleId];
+/******/ 			var e = new Error("Cannot find module '" + moduleId + "'");
+/******/ 			e.code = 'MODULE_NOT_FOUND';
+/******/ 			throw e;
+/******/ 		}
 /******/ 		__webpack_modules__[moduleId].call(module.exports, module, module.exports, __webpack_require__);
 /******/ 	
 /******/ 		// Return the exports of the module

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.115.26",
+    "version": "2.115.28",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,11 +38,11 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.115.26",
+        "@abaplint/core": "^2.115.28",
         "@types/chai": "^4.3.20",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.10",
-        "@types/node": "^24.10.13",
+        "@types/node": "^24.11.0",
         "@types/progress": "^2.0.7",
         "chai": "^4.5.0",
         "p-limit": "^3.1.0",
@@ -55,7 +55,7 @@
         "mocha": "^11.7.5",
         "progress": "^2.0.3",
         "typescript": "^5.9.3",
-        "webpack": "^5.105.2",
+        "webpack": "^5.105.4",
         "webpack-cli": "^6.0.1",
         "xml-js": "^1.6.11"
     }