@abaplint/cli 2.105.10 → 2.105.12

package/build/cli.js

@@ -7374,13 +7374,14 @@ class Select extends combi_1.Expression {
         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);
-        const perm = (0, combi_1.per)(_1.SQLFrom, into, _1.SQLForAllEntries, where, _1.SQLOrderBy, sql_up_to_1.SQLUpTo, offset, _1.SQLClient, _1.SQLHaving, bypass, sql_group_by_1.SQLGroupBy, fields, _1.DatabaseConnection);
-        const permSingle = (0, combi_1.per)(_1.SQLFrom, (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList), where, _1.SQLClient, bypass, fields, _1.DatabaseConnection);
+        // todo, HINTS cannot be anywhere, need an expression dedicated for strict sql
+        const perm = (0, combi_1.per)(_1.SQLFrom, into, _1.SQLForAllEntries, where, _1.SQLOrderBy, sql_up_to_1.SQLUpTo, offset, _1.SQLClient, _1.SQLHaving, bypass, sql_group_by_1.SQLGroupBy, fields, _1.DatabaseConnection, _1.SQLHints);
+        const permSingle = (0, combi_1.per)(_1.SQLFrom, (0, combi_1.altPrio)(sql_into_structure_1.SQLIntoStructure, _1.SQLIntoList), where, _1.SQLClient, bypass, fields, _1.DatabaseConnection, _1.SQLHints);
         const paren = (0, combi_1.seq)((0, combi_1.tok)(tokens_1.WParenLeftW), sql_field_name_1.SQLFieldName, (0, combi_1.tok)(tokens_1.WParenRightW));
         const fieldList = (0, combi_1.optPrio)((0, combi_1.altPrio)(_1.SQLFieldList, paren));
         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);
-        const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)(single, other), (0, combi_1.optPrio)(_1.SQLHints));
+        const ret = (0, combi_1.seq)("SELECT", (0, combi_1.altPrio)(single, other));
         return ret;
     }
 }
@@ -26798,6 +26799,9 @@ class Source {
         while (children.length >= 0) {
             if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.MethodCallChain) {
                 context = new method_call_chain_1.MethodCallChain().runSyntax(first, scope, filename, targetType);
+                if (context === undefined) {
+                    throw new Error("Method has no RETURNING value");
+                }
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.FieldChain) {
                 context = new field_chain_1.FieldChain().runSyntax(first, scope, filename, type);
@@ -27002,6 +27006,19 @@ class SQLCompare {
             }
             sourceType = new sql_source_1.SQLSource().runSyntax(s, scope, filename);
         }
+        const sqlin = node.findDirectExpression(Expressions.SQLIn);
+        if (sqlin && sqlin.getChildren().length === 2) {
+            const insource = node.findFirstExpression(Expressions.SQLSource);
+            if (insource) {
+                const intype = new sql_source_1.SQLSource().runSyntax(insource, scope, filename);
+                if (intype &&
+                    !(intype instanceof basic_1.VoidType) &&
+                    !(intype instanceof basic_1.UnknownType) &&
+                    !(intype instanceof basic_1.TableType)) {
+                    throw new Error("IN, not a table");
+                }
+            }
+        }
         const fieldName = (_a = node.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
         if (fieldName && sourceType && token) {
             // check compatibility for rule sql_value_conversion
@@ -51125,7 +51142,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.105.10";
+        return "2.105.12";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.105.10",
+    "version": "2.105.12",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,14 +38,14 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.105.10",
+        "@abaplint/core": "^2.105.12",
         "@types/chai": "^4.3.11",
-        "@types/glob": "^7.2.0",
+        "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.6",
-        "@types/node": "^20.10.8",
+        "@types/node": "^20.11.5",
         "@types/progress": "^2.0.7",
-        "chai": "^4.4.0",
+        "chai": "^4.4.1",
         "chalk": "^5.3.0",
         "eslint": "^8.56.0",
         "glob": "^7.2.3",