@abaplint/transpiler 1.7.0 → 1.7.4

package/build/src/statements/do.js

@@ -12,14 +12,14 @@ class DoTranspiler {
             const source = new expressions_1.SourceTranspiler(true).transpile(found, traversal).getCode();
             const idSource = unique_identifier_1.UniqueIdentifier.get();
             const id = unique_identifier_1.UniqueIdentifier.get();
-            return new chunk_1.Chunk(`const ${idSource} = ${source};
-for (let ${id} = 0; ${id} < ${idSource}; ${id}++) {
+            return new chunk_1.Chunk(`const ${idSource} = ${source};
+for (let ${id} = 0; ${id} < ${idSource}; ${id}++) {
 abap.builtin.sy.get().index.set(${id} + 1);`);
         }
         else {
             const unique = unique_identifier_1.UniqueIdentifier.get();
-            return new chunk_1.Chunk(`let ${unique} = 1;
-while (true) {
+            return new chunk_1.Chunk(`let ${unique} = 1;
+while (true) {
 abap.builtin.sy.get().index.set(${unique}++);`);
         }
     }

package/build/src/statements/end_method.d.ts

@@ -4,4 +4,5 @@ import { Traversal } from "../traversal";
 import { Chunk } from "../chunk";
 export declare class EndMethodTranspiler implements IStatementTranspiler {
     transpile(node: abaplint.Nodes.StatementNode, traversal: Traversal): Chunk;
+    private setSubrc;
 }

package/build/src/statements/end_method.js

@@ -5,28 +5,13 @@ const abaplint = require("@abaplint/core");
 const chunk_1 = require("../chunk");
 class EndMethodTranspiler {
     transpile(node, traversal) {
-        var _a, _b;
         const token = node.getFirstToken();
         const scope = traversal.findCurrentScopeByToken(token);
         if (scope === undefined) {
             throw new Error("EndMethodTranspiler, scope not found");
         }
         let returning = "";
-        let methodName = undefined;
-        if ((scope === null || scope === void 0 ? void 0 : scope.getIdentifier().stype) === abaplint.ScopeType.Method) {
-            methodName = scope === null || scope === void 0 ? void 0 : scope.getIdentifier().sname;
-        }
-        let className = undefined;
-        if (((_a = scope === null || scope === void 0 ? void 0 : scope.getParent()) === null || _a === void 0 ? void 0 : _a.getIdentifier().stype) === abaplint.ScopeType.ClassImplementation) {
-            className = (_b = scope === null || scope === void 0 ? void 0 : scope.getParent()) === null || _b === void 0 ? void 0 : _b.getIdentifier().sname;
-        }
-        if (methodName && className) {
-            const classDef = scope.findClassDefinition(className);
-            const methodDef = classDef === null || classDef === void 0 ? void 0 : classDef.getMethodDefinitions().getByName(methodName);
-            if (methodDef && methodDef.getExceptions().length > 0) {
-                returning += "abap.builtin.sy.get().subrc.set(0);\n";
-            }
-        }
+        returning += this.setSubrc(scope, traversal);
         const vars = scope.getData().vars;
         for (const n in vars) {
             const identifier = vars[n];
@@ -40,6 +25,34 @@ class EndMethodTranspiler {
         }
         return new chunk_1.Chunk().append(returning + "}", node, traversal);
     }
+    setSubrc(scope, traversal) {
+        var _a, _b;
+        let methodName = undefined;
+        if ((scope === null || scope === void 0 ? void 0 : scope.getIdentifier().stype) === abaplint.ScopeType.Method) {
+            methodName = scope === null || scope === void 0 ? void 0 : scope.getIdentifier().sname;
+        }
+        let className = undefined;
+        if (((_a = scope === null || scope === void 0 ? void 0 : scope.getParent()) === null || _a === void 0 ? void 0 : _a.getIdentifier().stype) === abaplint.ScopeType.ClassImplementation) {
+            className = (_b = scope === null || scope === void 0 ? void 0 : scope.getParent()) === null || _b === void 0 ? void 0 : _b.getIdentifier().sname;
+        }
+        if (methodName === undefined || className === undefined) {
+            return "";
+        }
+        let methodDef = undefined;
+        if (methodName.includes("~")) {
+            const split = methodName.split("~");
+            const classDef = traversal.findInterfaceDefinition(split[0], scope);
+            methodDef = classDef === null || classDef === void 0 ? void 0 : classDef.getMethodDefinitions().getByName(split[1]);
+        }
+        else {
+            const classDef = traversal.findClassDefinition(className, scope);
+            methodDef = classDef === null || classDef === void 0 ? void 0 : classDef.getMethodDefinitions().getByName(methodName);
+        }
+        if (methodDef && methodDef.getExceptions().length > 0) {
+            return "abap.builtin.sy.get().subrc.set(0);\n";
+        }
+        return "";
+    }
 }
 exports.EndMethodTranspiler = EndMethodTranspiler;
 //# sourceMappingURL=end_method.js.map

package/build/src/statements/read_table.js

@@ -16,6 +16,11 @@ class ReadTableTranspiler {
             const s = new expressions_1.SourceTranspiler().transpile(index, traversal).getCode();
             extra.push("index: " + s);
         }
+        const from = node.findExpressionAfterToken("FROM");
+        if (from) {
+            const s = new expressions_1.SourceTranspiler().transpile(from, traversal).getCode();
+            extra.push("from: " + s);
+        }
         const rt = node.findDirectExpression(abaplint.Expressions.ReadTableTarget);
         const target = rt === null || rt === void 0 ? void 0 : rt.findDirectExpression(abaplint.Expressions.Target);
         const fs = rt === null || rt === void 0 ? void 0 : rt.findDirectExpression(abaplint.Expressions.FSTarget);

package/build/src/structures/interface.js

@@ -47,12 +47,14 @@ class InterfaceTranspiler {
             const valExpression = constantStatement === null || constantStatement === void 0 ? void 0 : constantStatement.findFirstExpression(abaplint.Expressions.Value);
             if ((valExpression === null || valExpression === void 0 ? void 0 : valExpression.getChildren()[1].get()) instanceof abaplint.Expressions.SimpleFieldChain) {
                 const s = new expressions_1.FieldChainTranspiler().transpile(valExpression.getChildren()[1], traversal, false).getCode();
-                ret += name + ".set(" + s + ");\n";
+                const e = new expressions_1.ConstantTranspiler().escape(s);
+                ret += name + ".set(" + e + ");\n";
                 continue;
             }
             const val = identifier.getValue();
             if (typeof val === "string") {
-                ret += name + ".set(" + identifier.getValue() + ");\n";
+                const e = new expressions_1.ConstantTranspiler().escape(val);
+                ret += name + ".set(" + e + ");\n";
             }
             else if (typeof val === "object") {
                 const a = val;

package/build/src/traversal.d.ts

@@ -31,6 +31,9 @@ export declare class Traversal {
     isInterfaceAttribute(token: abaplint.Token): string | undefined;
     private findReadOrWriteReference;
     buildConstructorContents(scope: abaplint.ISpaghettiScopeNode | undefined, def: abaplint.IClassDefinition, inputName: string): string;
+    findInterfaceDefinition(name: string, scope: abaplint.ISpaghettiScopeNode | undefined): abaplint.IInterfaceDefinition | undefined;
+    findClassDefinition(name: string, scope: abaplint.ISpaghettiScopeNode | undefined): abaplint.IClassDefinition | undefined;
+    private dataFromInterfaces;
     determineType(node: abaplint.Nodes.ExpressionNode | abaplint.Nodes.StatementNode, scope: abaplint.ISpaghettiScopeNode | undefined): abaplint.AbstractType | undefined;
     registerClassOrInterface(def: abaplint.IClassDefinition | abaplint.IInterfaceDefinition | undefined): string;
     lookupClassOrInterface(name: string | undefined, token: abaplint.Token | undefined): string;

package/build/src/traversal.js

@@ -246,7 +246,6 @@ class Traversal {
         return undefined;
     }
     buildConstructorContents(scope, def, inputName) {
-        var _a;
         /*
         const vars = scope?.getData().vars;
         if (vars === undefined || Object.keys(vars).length === 0) {
@@ -259,30 +258,6 @@ class Traversal {
             // for now just pass the same input
             ret += `await super.constructor_(${inputName});\n`;
         }
-        /*
-        for (const n in vars) {
-          const identifier = vars[n];
-          if (identifier.getMeta().includes(abaplint.IdentifierMeta.Static) === true) {
-            continue;
-          }
-          const name = n.toLowerCase().replace("~", "$");
-          if (name === "super") {
-            continue; // todo, https://github.com/abaplint/transpiler/issues/133
-          }
-    
-          // todo, better handling of variables from interfaces, it should only initialize those that are directly implemented
-          if (def.getAttributes().findByName(name) === undefined
-              && name.includes("$") === false
-              && name !== "me") {
-            continue;
-          }
-    
-          ret += "this." + name + " = " + new TranspileTypes().toType(identifier.getType()) + ";\n";
-          if (name === "me") {
-            ret += "this.me.set(this);\n";
-          }
-        }
-        */
         ret += "this.me = new abap.types.ABAPObject();\n";
         ret += "this.me.set(this);\n";
         for (const a of def.getAttributes().getAll()) {
@@ -294,13 +269,7 @@ class Traversal {
         }
         // attributes from directly implemented interfaces(not interfaces implemented in super classes)
         for (const i of def.getImplementing()) {
-            for (const a of ((_a = scope === null || scope === void 0 ? void 0 : scope.findInterfaceDefinition(i.name)) === null || _a === void 0 ? void 0 : _a.getAttributes().getAll()) || []) {
-                if (a.getMeta().includes("static" /* Static */) === true) {
-                    continue;
-                }
-                const name = i.name.toLowerCase() + "$" + a.getName().toLowerCase();
-                ret += "this." + name + " = " + new types_1.TranspileTypes().toType(a.getType()) + ";\n";
-            }
+            ret += this.dataFromInterfaces(i.name, scope);
         }
         // handle aliases after initialization of carrier variables
         for (const a of def.getAliases().getAll()) {
@@ -312,6 +281,37 @@ class Traversal {
         }
         return ret;
     }
+    findInterfaceDefinition(name, scope) {
+        let intf = scope === null || scope === void 0 ? void 0 : scope.findInterfaceDefinition(name);
+        if (intf === undefined) {
+            const iglobal = this.reg.getObject("INTF", name);
+            intf = iglobal === null || iglobal === void 0 ? void 0 : iglobal.getDefinition();
+        }
+        return intf;
+    }
+    findClassDefinition(name, scope) {
+        let clas = scope === null || scope === void 0 ? void 0 : scope.findClassDefinition(name);
+        if (clas === undefined) {
+            const iglobal = this.reg.getObject("CLAS", name);
+            clas = iglobal === null || iglobal === void 0 ? void 0 : iglobal.getDefinition();
+        }
+        return clas;
+    }
+    dataFromInterfaces(name, scope) {
+        let ret = "";
+        const intf = this.findInterfaceDefinition(name, scope);
+        for (const a of (intf === null || intf === void 0 ? void 0 : intf.getAttributes().getAll()) || []) {
+            if (a.getMeta().includes("static" /* Static */) === true) {
+                continue;
+            }
+            const n = name.toLowerCase() + "$" + a.getName().toLowerCase();
+            ret += "this." + n + " = " + new types_1.TranspileTypes().toType(a.getType()) + ";\n";
+        }
+        for (const i of (intf === null || intf === void 0 ? void 0 : intf.getImplementing()) || []) {
+            ret += this.dataFromInterfaces(i.name, scope);
+        }
+        return ret;
+    }
     determineType(node, scope) {
         var _a;
         if (scope === undefined) {

package/build/src/unit_test.js

@@ -4,28 +4,28 @@ exports.UnitTest = void 0;
 const abaplint = require("@abaplint/core");
 class UnitTest {
     initializationScript(reg, dbSetup) {
-        return `import runtime from "@abaplint/runtime";
-global.abap = new runtime.ABAP();
-${this.buildImports(reg)}
-export async function initDB() {
-  return global.abap.initDB(\`${dbSetup}\`);
+        return `import runtime from "@abaplint/runtime";
+global.abap = new runtime.ABAP();
+${this.buildImports(reg)}
+export async function initDB() {
+  return global.abap.initDB(\`${dbSetup}\`);
 }`;
     }
     unitTestScript(reg, skip) {
-        let ret = `import fs from "fs";
-import path from "path";
-import {dirname} from 'path';
-import {fileURLToPath} from 'url';
-const __dirname = dirname(fileURLToPath(import.meta.url));
-import {initDB} from "./init.mjs";
-import runtime from "@abaplint/runtime";
-
-async function run() {
-  await initDB();
-  const unit = new runtime.UnitTestResult();
-  let clas;
-  let locl;
-  let meth;
+        let ret = `import fs from "fs";
+import path from "path";
+import {dirname} from 'path';
+import {fileURLToPath} from 'url';
+const __dirname = dirname(fileURLToPath(import.meta.url));
+import {initDB} from "./init.mjs";
+import runtime from "@abaplint/runtime";
+
+async function run() {
+  await initDB();
+  const unit = new runtime.UnitTestResult();
+  let clas;
+  let locl;
+  let meth;
 try {\n`;
         for (const obj of reg.getObjects()) {
             if (reg.isDependency(obj) || !(obj instanceof abaplint.Objects.Class)) {
@@ -40,8 +40,8 @@ try {\n`;
                         || def.methods.length === 0) {
                         continue;
                     }
-                    ret += `{
-const {${def.name}} = await import("./${obj.getName().toLowerCase()}.${obj.getType().toLowerCase()}.testclasses.mjs");
+                    ret += `{
+const {${def.name}} = await import("./${obj.getName().toLowerCase()}.${obj.getType().toLowerCase()}.testclasses.mjs");
 locl = clas.addTestClass("${def.name}");\n`;
                     ret += `if (${def.name}.class_setup) await ${def.name}.class_setup();\n`;
                     for (const m of def.methods) {
@@ -69,24 +69,24 @@ locl = clas.addTestClass("${def.name}");\n`;
                 }
             }
         }
-        ret += `// -------------------END-------------------
-console.log(abap.console.get());
-fs.writeFileSync(__dirname + path.sep + "output.xml", unit.xUnitXML());
-} catch (e) {
-  if (meth) {
-    meth.fail();
-  }
-  console.log(abap.console.get());
-  fs.writeFileSync(__dirname + path.sep + "output.xml", unit.xUnitXML());
-  throw e;
-}
-}
-
-run().then(() => {
-  process.exit();
-}).catch((err) => {
-  console.log(err);
-  process.exit(1);
+        ret += `// -------------------END-------------------
+console.log(abap.console.get());
+fs.writeFileSync(__dirname + path.sep + "output.xml", unit.xUnitXML());
+} catch (e) {
+  if (meth) {
+    meth.fail();
+  }
+  console.log(abap.console.get());
+  fs.writeFileSync(__dirname + path.sep + "output.xml", unit.xUnitXML());
+  throw e;
+}
+}
+
+run().then(() => {
+  process.exit();
+}).catch((err) => {
+  console.log(err);
+  process.exit(1);
 });`;
         return ret;
     }

package/package.json

@@ -1,41 +1,41 @@
-{
-  "name": "@abaplint/transpiler",
-  "version": "1.7.0",
-  "description": "Transpiler",
-  "main": "build/src/index.js",
-  "typings": "build/src/index.d.ts",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/abaplint/transpiler.git"
-  },
-  "scripts": {
-    "compile": "tsc",
-    "publish:minor": "npm --no-git-tag-version version minor && rm -rf build && npm install && npm run test && npm publish --access public",
-    "publish:patch": "npm --no-git-tag-version version patch && rm -rf build && npm install && npm run test && npm publish --access public",
-    "test": "npm run compile && mocha"
-  },
-  "mocha": {
-    "recursive": true,
-    "reporter": "progress",
-    "spec": "build/test/**/*.js",
-    "require": "source-map-support/register"
-  },
-  "keywords": [
-    "ABAP",
-    "abaplint"
-  ],
-  "author": "abaplint",
-  "license": "MIT",
-  "dependencies": {
-    "@abaplint/core": "^2.83.16",
-    "source-map": "^0.7.3"
-  },
-  "devDependencies": {
-    "@types/chai": "^4.3.0",
-    "@types/mocha": "^9.0.0",
-    "chai": "^4.3.4",
-    "mocha": "^9.1.3",
-    "source-map-support": "^0.5.21",
-    "typescript": "^4.5.4"
-  }
-}
+{
+  "name": "@abaplint/transpiler",
+  "version": "1.7.4",
+  "description": "Transpiler",
+  "main": "build/src/index.js",
+  "typings": "build/src/index.d.ts",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/abaplint/transpiler.git"
+  },
+  "scripts": {
+    "compile": "tsc",
+    "publish:minor": "npm --no-git-tag-version version minor && rm -rf build && npm install && npm run test && npm publish --access public",
+    "publish:patch": "npm --no-git-tag-version version patch && rm -rf build && npm install && npm run test && npm publish --access public",
+    "test": "npm run compile && mocha"
+  },
+  "mocha": {
+    "recursive": true,
+    "reporter": "progress",
+    "spec": "build/test/**/*.js",
+    "require": "source-map-support/register"
+  },
+  "keywords": [
+    "ABAP",
+    "abaplint"
+  ],
+  "author": "abaplint",
+  "license": "MIT",
+  "dependencies": {
+    "@abaplint/core": "^2.83.18",
+    "source-map": "^0.7.3"
+  },
+  "devDependencies": {
+    "@types/chai": "^4.3.0",
+    "@types/mocha": "^9.0.0",
+    "chai": "^4.3.4",
+    "mocha": "^9.1.3",
+    "source-map-support": "^0.5.21",
+    "typescript": "^4.5.4"
+  }
+}