@abaplint/transpiler-cli 2.7.83 → 2.7.84

package/build/bundle.js

@@ -8667,7 +8667,7 @@ class CallFunction {
         const update = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.str)("IN UPDATE TASK"));
         const unit = (0, combi_1.seq)("UNIT", expressions_1.Source);
         const background = (0, combi_1.verNot)(version_1.Version.Cloud, (0, combi_1.seq)("IN BACKGROUND", (0, combi_1.altPrio)("TASK", unit)));
-        const calling = (0, combi_1.seq)("CALLING", expressions_1.MethodName, "ON END OF TASK");
+        const calling = (0, combi_1.seq)("CALLING", expressions_1.MethodSource, "ON END OF TASK");
         const performing = (0, combi_1.seq)("PERFORMING", expressions_1.FormName, "ON END OF TASK");
         const separate = (0, combi_1.str)("AS SEPARATE UNIT");
         const keeping = (0, combi_1.str)("KEEPING LOGICAL UNIT OF WORK");
@@ -19570,6 +19570,9 @@ class CurrentScope {
         if (this.current.getData().types[upper] !== undefined) {
             throw new Error(`Type name "${name}" already defined`);
         }
+        else if (this.isOO() && this.current.getData().vars[upper] !== undefined) {
+            throw new Error(`"${name}" already defined`);
+        }
         this.current.getData().types[upper] = type;
     }
     addExtraLikeType(type) {
@@ -19619,6 +19622,9 @@ class CurrentScope {
         if (this.current.getData().vars[upper] !== undefined) {
             throw new Error(`Variable name "${name}" already defined`);
         }
+        else if (this.isOO() && this.current.getData().types[upper] !== undefined) {
+            throw new Error(`"${name}" already defined`);
+        }
         this.current.getData().vars[upper] = identifier;
     }
     addIdentifier(identifier) {
@@ -19885,7 +19891,7 @@ class CurrentScope {
         while (curr !== undefined) {
             const stype = curr.getIdentifier().stype;
             if (stype === _scope_type_1.ScopeType.ClassDefinition
-                || stype === _scope_type_1.ScopeType.ClassImplementation
+                //          || stype === ScopeType.ClassImplementation
                 || stype === _scope_type_1.ScopeType.Interface) {
                 return true;
             }
@@ -20529,6 +20535,7 @@ exports.TypeUtils = void 0;
 const types_1 = __webpack_require__(/*! ../types */ "./node_modules/@abaplint/core/build/src/abap/types/index.js");
 const basic_1 = __webpack_require__(/*! ../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 const cgeneric_type_1 = __webpack_require__(/*! ../types/basic/cgeneric_type */ "./node_modules/@abaplint/core/build/src/abap/types/basic/cgeneric_type.js");
+// todo: refactor to static? for performance
 class TypeUtils {
     constructor(scope) {
         this.scope = scope;
@@ -20735,7 +20742,7 @@ class TypeUtils {
         return false;
     }
     isAssignableStrict(source, target, calculated = false) {
-        var _a, _b, _c, _d, _e, _f, _g;
+        var _a, _b, _c, _d, _e, _f, _g, _h;
         /*
             console.dir(source);
             console.dir(target);
@@ -20823,6 +20830,17 @@ class TypeUtils {
                 return false;
             }
         }
+        else if (source instanceof basic_1.IntegerType) {
+            if (target instanceof basic_1.StringType) {
+                return false;
+            }
+            else if (target instanceof basic_1.Integer8Type) {
+                if (((_h = source.getAbstractTypeData()) === null || _h === void 0 ? void 0 : _h.derivedFromConstant) === true) {
+                    return true;
+                }
+                return false;
+            }
+        }
         else if (source instanceof basic_1.FloatType) {
             if (target instanceof basic_1.IntegerType) {
                 return false;
@@ -22478,7 +22496,7 @@ class Constant {
     runSyntax(node) {
         // todo: ConcatenatedConstant is not used?
         if (node.findDirectExpression(expressions_1.Integer)) {
-            return basic_1.IntegerType.get();
+            return basic_1.IntegerType.get({ derivedFromConstant: true });
         }
         else if (node.getFirstToken().getStr().startsWith("'")) {
             let len = node.getFirstToken().getStr().length - 2;
@@ -28369,6 +28387,7 @@ const target_1 = __webpack_require__(/*! ../expressions/target */ "./node_module
 const fstarget_1 = __webpack_require__(/*! ../expressions/fstarget */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/fstarget.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
 const inline_data_1 = __webpack_require__(/*! ../expressions/inline_data */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/inline_data.js");
+const _type_utils_1 = __webpack_require__(/*! ../_type_utils */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_type_utils.js");
 class InsertInternal {
     runSyntax(node, scope, filename) {
         let targetType;
@@ -28376,7 +28395,7 @@ class InsertInternal {
         if (t) {
             targetType = new target_1.Target().runSyntax(t, scope, filename);
         }
-        if (targetType instanceof basic_1.TableType) {
+        if (targetType instanceof basic_1.TableType && node.findDirectTokenByText("LINES") === undefined) {
             targetType = targetType.getRowType();
         }
         let source = node.findDirectExpression(Expressions.SimpleSource4);
@@ -28394,6 +28413,9 @@ class InsertInternal {
                 new fstarget_1.FSTarget().runSyntax(afterAssigning, scope, filename, sourceType);
             }
         }
+        if (new _type_utils_1.TypeUtils(scope).isAssignableStrict(sourceType, targetType) === false) {
+            throw new Error("Types not compatible");
+        }
         const afterInto = node.findExpressionAfterToken("INTO");
         if ((afterInto === null || afterInto === void 0 ? void 0 : afterInto.get()) instanceof Expressions.Target && sourceType) {
             const inline = afterInto.findDirectExpression(Expressions.InlineData);
@@ -47797,7 +47819,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.29";
+        return "2.102.32";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -70644,7 +70666,14 @@ class ConstantTranspiler {
         const int = node.findFirstExpression(core_1.Expressions.Integer);
         if (int) {
             const concat = int.concatTokens().trim();
-            let code = `new abap.types.Integer().set(${concat})`;
+            const parsed = Number.parseInt(concat, 10);
+            let code = "";
+            if (parsed > 2147483647 || parsed < -2147483648) {
+                code = `new abap.types.Integer8().set(${concat})`;
+            }
+            else {
+                code = `new abap.types.Integer().set(${concat})`;
+            }
             if (this.addGet === true) {
                 code += ".get()";
             }
@@ -77483,7 +77512,6 @@ class PerformTranspiler {
         const params = [];
         let index = 0;
         for (const t of ((_a = node.findDirectExpression(abaplint.Expressions.PerformTables)) === null || _a === void 0 ? void 0 : _a.findDirectExpressions(abaplint.Expressions.Source)) || []) {
-            console.dir(t);
             const name = def === null || def === void 0 ? void 0 : def.getTablesParameters()[index].getName().toLowerCase();
             if (name === undefined) {
                 continue;
@@ -77493,7 +77521,6 @@ class PerformTranspiler {
         }
         index = 0;
         for (const u of ((_b = node.findDirectExpression(abaplint.Expressions.PerformUsing)) === null || _b === void 0 ? void 0 : _b.findDirectExpressions(abaplint.Expressions.Source)) || []) {
-            console.dir(u);
             const name = def === null || def === void 0 ? void 0 : def.getUsingParameters()[index].getName().toLowerCase();
             if (name === undefined) {
                 continue;
@@ -77503,7 +77530,6 @@ class PerformTranspiler {
         }
         index = 0;
         for (const c of ((_c = node.findDirectExpression(abaplint.Expressions.PerformChanging)) === null || _c === void 0 ? void 0 : _c.findDirectExpressions(abaplint.Expressions.Source)) || []) {
-            console.dir(c);
             const name = def === null || def === void 0 ? void 0 : def.getChangingParameters()[index].getName().toLowerCase();
             if (name === undefined) {
                 continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.83",
+  "version": "2.7.84",
   "description": "Transpiler - Command Line Interface",
   "funding": "https://github.com/sponsors/larshp",
   "bin": {
@@ -26,12 +26,12 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.7.83",
+    "@abaplint/transpiler": "^2.7.84",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
     "@types/node": "^20.5.7",
-    "@abaplint/core": "^2.102.29",
+    "@abaplint/core": "^2.102.32",
     "progress": "^2.0.3",
     "webpack": "^5.88.2",
     "webpack-cli": "^5.1.4",