@abaplint/transpiler 2.13.93 → 2.13.95

package/build/src/statements/do.js

@@ -46,11 +46,14 @@ class DoTranspiler {
     transpile(node, traversal) {
         const found = node.findFirstExpression(abaplint.Expressions.Source);
         if (found) {
-            const source = new expressions_1.SourceTranspiler(true).transpile(found, traversal).getCode();
+            // the count is converted to i, like any numeric expression position: an
+            // f count read with get( ) gives its text 3,0000000000000000E+00, which
+            // no loop counter compares with, and a p count of 2.4 means 2 passes
+            const source = new expressions_1.SourceTranspiler(false).transpile(found, traversal).getCode();
             const idSource = unique_identifier_1.UniqueIdentifier.get();
             const id = unique_identifier_1.UniqueIdentifier.get();
             return new chunk_1.Chunk(`const ${this.syIndexBackup} = abap.builtin.sy.get().index.get();
-const ${idSource} = ${source};
+const ${idSource} = new abap.types.Integer().set(${source}).get();
 for (let ${id} = 0; ${id} < ${idSource}; ${id}++) {
 abap.builtin.sy.get().index.set(${id} + 1);`);
         }

package/build/src/statements/loop.js

@@ -148,7 +148,10 @@ class LoopTranspiler {
             extra.push(`atLast: (last) => { ${this.atLast} = last; }`);
         }
         const topEquals = {};
-        for (const compare of whereNode?.findDirectExpressions(abaplint.Expressions.ComponentCompare) || []) {
+        // only a pure conjunction makes each top-level `=` a condition every row
+        // has to meet: `a = 1 OR b = 2` narrows by neither
+        const onlyAnd = whereNode?.getChildren().every(c => !(c instanceof abaplint.Nodes.TokenNode) || c.concatTokens().toUpperCase() !== "OR") ?? true;
+        for (const compare of onlyAnd ? whereNode?.findDirectExpressions(abaplint.Expressions.ComponentCompare) || [] : []) {
             const op = compare.findDirectExpression(abaplint.Expressions.CompareOperator)?.concatTokens().toUpperCase();
             if (op !== "=" && op !== "EQ") {
                 continue;

package/build/src/statements/update_database.js

@@ -58,8 +58,12 @@ class UpdateDatabaseTranspiler {
             }
             options.push(`"set": [${s.join(",")}]`);
         }
+        const fromTable = node.findExpressionAfterToken("TABLE");
+        if (fromTable && fromTable.get() instanceof abaplint.Expressions.SQLSource) {
+            options.push(`"table": ` + traversal.traverse(fromTable).getCode());
+        }
         const sourceExpression = node.findDirectExpression(abaplint.Expressions.SQLSource)?.findDirectExpression(abaplint.Expressions.SimpleSource3);
-        if (sourceExpression) {
+        if (sourceExpression && fromTable === undefined) {
             const sqlSource = new expressions_1.SourceTranspiler(true).transpile(sourceExpression, traversal).getCode();
             const tableName = node.findDirectExpression(abaplint.Expressions.DatabaseTable)?.concatTokens();
             const tabl = traversal.findTable(tableName);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler",
-  "version": "2.13.93",
+  "version": "2.13.95",
   "description": "Transpiler",
   "main": "build/src/index.js",
   "typings": "build/src/index.d.ts",
@@ -29,14 +29,14 @@
   "author": "abaplint",
   "license": "MIT",
   "dependencies": {
-    "@abaplint/core": "^2.120.59",
+    "@abaplint/core": "^2.120.63",
     "source-map": "^0.8.0"
   },
   "devDependencies": {
     "@types/chai": "^5.2.3",
     "@types/mocha": "^10.0.10",
     "chai": "^6.2.2",
-    "mocha": "^12.0.2",
+    "mocha": "^12.0.3",
     "source-map-support": "^0.5.21",
     "typescript": "^6.0.3"
   }