@abaplint/core 2.105.10 → 2.105.12

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

@@ -16,13 +16,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;
     }
 }

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

@@ -170,6 +170,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);

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

@@ -23,6 +23,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

package/build/src/registry.js

@@ -65,7 +65,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/core",
-    "version": "2.105.10",
+    "version": "2.105.12",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",
@@ -53,11 +53,11 @@
         "@microsoft/api-extractor": "^7.39.1",
         "@types/chai": "^4.3.11",
         "@types/mocha": "^10.0.6",
-        "@types/node": "^20.10.8",
-        "chai": "^4.4.0",
+        "@types/node": "^20.11.5",
+        "chai": "^4.4.1",
         "eslint": "^8.56.0",
         "mocha": "^10.2.0",
-        "c8": "^9.0.0",
+        "c8": "^9.1.0",
         "source-map-support": "^0.5.21",
         "ts-json-schema-generator": "^1.5.0",
         "typescript": "^5.3.3"