@abaplint/cli 2.101.17 → 2.101.19

package/build/cli.js

@@ -21632,13 +21632,13 @@ class TypeUtils {
         }
         return false;
     }
-    isAssignableStrict(source, target, containsMethodCall = false) {
-        var _a, _b, _c, _d, _e, _f;
+    isAssignableStrict(source, target, calculated = false) {
+        var _a, _b, _c, _d, _e, _f, _g;
         /*
             console.dir(source);
             console.dir(target);
         */
-        if (containsMethodCall) {
+        if (calculated) {
             return this.isAssignable(source, target);
         }
         if (source instanceof basic_1.CharacterType) {
@@ -21679,12 +21679,18 @@ class TypeUtils {
             if (target instanceof basic_1.StructureType && this.structureContainsString(target)) {
                 return false;
             }
-            else if (target instanceof basic_1.XSequenceType || target instanceof basic_1.XStringType) {
+            else if (target instanceof basic_1.IntegerType) {
                 if (((_f = source.getAbstractTypeData()) === null || _f === void 0 ? void 0 : _f.derivedFromConstant) === true) {
                     return true;
                 }
                 return false;
             }
+            else if (target instanceof basic_1.XSequenceType || target instanceof basic_1.XStringType) {
+                if (((_g = source.getAbstractTypeData()) === null || _g === void 0 ? void 0 : _g.derivedFromConstant) === true) {
+                    return true;
+                }
+                return false;
+            }
             return true;
         }
         else if (source instanceof basic_1.StructureType && target instanceof basic_1.StructureType) {
@@ -21705,6 +21711,11 @@ class TypeUtils {
                 return false;
             }
         }
+        else if (source instanceof basic_1.FloatType) {
+            if (target instanceof basic_1.IntegerType) {
+                return false;
+            }
+        }
         else if (source instanceof basic_1.XStringType) {
             if (target instanceof basic_1.CLikeType) {
                 return false;
@@ -24614,10 +24625,12 @@ class MethodCallParam {
             if (child.get() instanceof Expressions.Source) {
                 sourceType = new source_1.Source().runSyntax(child, scope, filename, targetType);
             }
+            const calculated = child.findFirstExpression(Expressions.MethodCallChain) !== undefined
+                || child.findFirstExpression(Expressions.ArithOperator) !== undefined;
             if (sourceType === undefined) {
                 throw new Error("No source type determined, method source");
             }
-            else if (new _type_utils_1.TypeUtils(scope).isAssignableStrict(sourceType, targetType, child.findFirstExpression(Expressions.MethodCallChain) !== undefined) === false) {
+            else if (new _type_utils_1.TypeUtils(scope).isAssignableStrict(sourceType, targetType, calculated) === false) {
                 throw new Error("Method parameter type not compatible");
             }
         }
@@ -25664,19 +25677,24 @@ class Source {
                 context = new string_template_1.StringTemplate().runSyntax(first, scope, filename);
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.Source) {
-                context = new Source().runSyntax(first, scope, filename);
+                const found = new Source().runSyntax(first, scope, filename);
+                context = this.infer(context, found);
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.Constant) {
-                context = new constant_1.Constant().runSyntax(first);
+                const found = new constant_1.Constant().runSyntax(first);
+                context = this.infer(context, found);
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.Dereference) {
                 context = new dereference_1.Dereference().runSyntax(context);
-                //      } else if (first instanceof ExpressionNode && first.get() instanceof Expressions.ArrowOrDash) {
-                //        console.dir("dash");
             }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.ComponentChain) {
                 context = new component_chain_1.ComponentChain().runSyntax(context, first, scope, filename);
             }
+            else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.ArithOperator) {
+                if (first.concatTokens() === "**") {
+                    context = new basic_1.FloatType();
+                }
+            }
             else if (first instanceof nodes_1.ExpressionNode && first.get() instanceof Expressions.AttributeChain) {
                 context = new attribute_chain_1.AttributeChain().runSyntax(context, first, scope, filename, type);
             }
@@ -25691,6 +25709,14 @@ class Source {
         return context;
     }
     ////////////////////////////////
+    infer(context, found) {
+        if (context instanceof basic_1.FloatType && found instanceof basic_1.IntegerType) {
+            return context;
+        }
+        else {
+            return found;
+        }
+    }
     addIfInferred(node, scope, filename, inferredType) {
         const basic = new basic_types_1.BasicTypes(filename, scope);
         const typeExpression = node.findFirstExpression(Expressions.TypeNameOrInfer);
@@ -25832,6 +25858,7 @@ exports.SourceFieldSymbol = SourceFieldSymbol;
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.SQLCompare = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
+const nodes_1 = __webpack_require__(/*! ../../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
 class SQLCompare {
@@ -25840,7 +25867,12 @@ class SQLCompare {
         let sourceType;
         let token;
         for (const s of node.findAllExpressions(Expressions.SQLSource)) {
-            token = s.getFirstToken();
+            for (const child of s.getChildren()) {
+                if (child instanceof nodes_1.ExpressionNode) {
+                    token = child.getFirstToken();
+                    break;
+                }
+            }
             sourceType = new sql_source_1.SQLSource().runSyntax(s, scope, filename);
         }
         const fieldName = (_a = node.findDirectExpression(Expressions.SQLFieldName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
@@ -29303,7 +29335,6 @@ const dynamic_1 = __webpack_require__(/*! ../expressions/dynamic */ "./node_modu
 const loop_group_by_1 = __webpack_require__(/*! ../expressions/loop_group_by */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/loop_group_by.js");
 class Loop {
     runSyntax(node, scope, filename) {
-        var _a;
         const loopTarget = node.findDirectExpression(Expressions.LoopTarget);
         let target = loopTarget === null || loopTarget === void 0 ? void 0 : loopTarget.findDirectExpression(Expressions.Target);
         const targetType = target ? new target_1.Target().runSyntax(target, scope, filename) : undefined;
@@ -29337,14 +29368,24 @@ class Loop {
             && concat.startsWith("LOOP AT GROUP ") === false) {
             throw new Error("Loop, not a table type");
         }
+        else if (loopTarget === undefined
+            && sourceType instanceof basic_1.TableType
+            && sourceType.isWithHeader() === false) {
+            throw new Error("Loop, no header");
+        }
+        const targetConcat = loopTarget === null || loopTarget === void 0 ? void 0 : loopTarget.concatTokens().toUpperCase();
         if (sourceType instanceof basic_1.TableType) {
-            const targetConcat = (_a = node.findDirectExpression(Expressions.LoopTarget)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
             rowType = sourceType.getRowType();
             sourceType = rowType;
             if (targetConcat === null || targetConcat === void 0 ? void 0 : targetConcat.startsWith("REFERENCE INTO ")) {
                 sourceType = new basic_1.DataReference(sourceType);
             }
         }
+        if (targetConcat
+            && targetConcat.startsWith("TRANSPORTING ")
+            && node.findDirectTokenByText("WHERE") === undefined) {
+            throw new Error("Loop, TRANSPORTING NO FIELDS only with WHERE");
+        }
         const inline = target === null || target === void 0 ? void 0 : target.findDirectExpression(Expressions.InlineData);
         if (inline) {
             new inline_data_1.InlineData().runSyntax(inline, scope, filename, sourceType);
@@ -48412,7 +48453,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.101.17";
+        return "2.101.19";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.101.17",
+    "version": "2.101.19",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,12 +38,12 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.101.17",
+        "@abaplint/core": "^2.101.19",
         "@types/chai": "^4.3.5",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
         "@types/mocha": "^10.0.1",
-        "@types/node": "^20.2.6",
+        "@types/node": "^20.3.1",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.7",
         "chalk": "^5.2.0",