@abaplint/transpiler-cli 2.5.33 → 2.5.35

package/build/bundle.js

@@ -76000,7 +76000,20 @@ class ClassImplementationTranspiler {
         }
         // this is not correct, ABAP does not invocate the class constructor at require time,
         // but this will probably work
-        if (cdef.getMethodDefinitions().getByName("class_constructor")) {
+        if (traversal.getCurrentObject().getType() === "CLAS") {
+            // gather call of all class constructors together for a global class, also local classes
+            // as they might use the global class
+            if (cdef.isGlobal() === true) {
+                for (const f of traversal.getCurrentObject().getABAPFiles()) {
+                    for (const def of f.getInfo().listClassDefinitions()) {
+                        if (def.methods.some(m => m.name.toLowerCase() === "class_constructor")) {
+                            ret += "await " + traversal.lookupClassOrInterface(def.name, node.getFirstToken()) + ".class_constructor();\n";
+                        }
+                    }
+                }
+            }
+        }
+        else if (cdef.getMethodDefinitions().getByName("class_constructor")) {
             ret += "await " + traversal_1.Traversal.escapeNamespace(node.getFirstToken().getStr().toLowerCase()) + ".class_constructor();\n";
         }
         return ret;
@@ -76223,7 +76236,7 @@ class FunctionModuleTranspiler {
         const scope = traversal.findCurrentScopeByToken(node.getLastToken());
         let ret = "";
         for (const p of module.getParameters()) {
-            ret += `// ${p.direction} ${p.name} ${p.type}\n`;
+            ret += `// ${p.direction} ${p.name} ${p.type} ${p.optional}\n`;
             let direction = p.direction;
             if (direction === "importing") {
                 direction = "exporting";
@@ -76237,15 +76250,12 @@ class FunctionModuleTranspiler {
                 name += "_";
             }
             ret += `let ${name} = INPUT.${direction}?.${name};\n`;
-            if (direction === "exporting" || direction === "importing" || direction === "changing") {
-                const type = (_c = scope === null || scope === void 0 ? void 0 : scope.findVariable(name)) === null || _c === void 0 ? void 0 : _c.getType();
-                if (type !== undefined) {
-                    // todo, set DEFAULT value
-                    // todo, check for OPTIONALness and raise exceptions and stuff
-                    ret += `if (${name} === undefined) {
-            ${name} = ${new transpile_types_1.TranspileTypes().toType(type)};
-          }\n`;
-                }
+            const type = (_c = scope === null || scope === void 0 ? void 0 : scope.findVariable(name)) === null || _c === void 0 ? void 0 : _c.getType();
+            if (type !== undefined && p.optional === true) {
+                // todo, set DEFAULT value
+                ret += `if (${name} === undefined) {
+  ${name} = ${new transpile_types_1.TranspileTypes().toType(type)};
+}\n`;
             }
         }
         return ret;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.5.33",
+  "version": "2.5.35",
   "description": "Transpiler - Command Line Interface",
   "bin": {
     "abap_transpile": "./abap_transpile"
@@ -25,7 +25,7 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.5.33",
+    "@abaplint/transpiler": "^2.5.35",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",