@abaplint/transpiler-cli 2.5.45 → 2.5.47

package/build/bundle.js

@@ -46211,7 +46211,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.95.44";
+        return "2.95.47";
     }
     getDDICReferences() {
         return this.references;
@@ -50994,14 +50994,16 @@ Only one transformation is applied to a statement at a time, so multiple steps m
                 // actively editing the files
                 continue;
             }
+            if (highSyntax === undefined) {
+                highSyntax = new syntax_1.SyntaxLogic(this.highReg, highSyntaxObj).run();
+            }
+            let containsUnknown = false;
             for (let i = 0; i < lowStatements.length; i++) {
                 const low = lowStatements[i];
                 const high = highStatements[i];
                 if ((low.get() instanceof _statement_1.Unknown && !(high.get() instanceof _statement_1.Unknown))
                     || high.findFirstExpression(Expressions.InlineData)) {
-                    if (highSyntax === undefined) {
-                        highSyntax = new syntax_1.SyntaxLogic(this.highReg, highSyntaxObj).run();
-                    }
+                    containsUnknown = true;
                     try {
                         const issue = this.checkStatement(low, high, lowFile, highSyntax, highFile);
                         if (issue) {
@@ -51019,6 +51021,18 @@ Only one transformation is applied to a statement at a time, so multiple steps m
                     }
                 }
             }
+            if (ret.length === 0 && containsUnknown) {
+                // this is a hack in order not to change too many unit tests
+                for (let i = 0; i < lowStatements.length; i++) {
+                    const high = highStatements[i];
+                    if (high.get() instanceof Statements.Data) {
+                        const issue = this.anonymousTableType(high, lowFile, highSyntax);
+                        if (issue) {
+                            ret.push(issue);
+                        }
+                    }
+                }
+            }
         }
         return ret;
     }
@@ -51049,7 +51063,12 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         if (low.getFirstToken().getStart() instanceof position_1.VirtualPosition) {
             return undefined;
         }
-        let found = this.downportEnum(low, high, lowFile, highSyntax, highFile);
+        // downport XSDBOOL() early, as it is valid 702 syntax
+        let found = this.replaceXsdBool(high, lowFile, highSyntax);
+        if (found) {
+            return found;
+        }
+        found = this.downportEnum(low, high, lowFile, highSyntax, highFile);
         if (found) {
             return found;
         }
@@ -51191,10 +51210,6 @@ Only one transformation is applied to a statement at a time, so multiple steps m
         if (found) {
             return found;
         }
-        found = this.replaceXsdBool(high, lowFile, highSyntax);
-        if (found) {
-            return found;
-        }
         found = this.replaceLineFunctions(high, lowFile, highSyntax, highFile);
         if (found) {
             return found;
@@ -51432,6 +51447,22 @@ ${indentation}`);
         const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
         return issue_1.Issue.atToken(lowFile, inlineData.getFirstToken(), "Outline SELECT @DATA", this.getMetadata().key, this.conf.severity, fix);
     }
+    // the anonymous type minght be used in inferred type statements, define it so it can be referred
+    anonymousTableType(high, lowFile, highSyntax) {
+        if (!(high.get() instanceof Statements.Data)) {
+            return undefined;
+        }
+        const tt = high.findFirstExpression(Expressions.TypeTable);
+        if (tt === undefined) {
+            return undefined;
+        }
+        const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
+        const code = `TYPES ${uniqueName} ${tt.concatTokens()}.\n`;
+        const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, high.getStart(), code);
+        const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, tt.getFirstToken().getStart(), tt.getLastToken().getEnd(), "TYPE " + uniqueName);
+        const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
+        return issue_1.Issue.atToken(lowFile, high.getFirstToken(), "Add type for table definition", this.getMetadata().key, this.conf.severity, fix);
+    }
     downportMessage(high, lowFile, highSyntax) {
         var _a, _b;
         if (!(high.get() instanceof Statements.Message)) {
@@ -52624,7 +52655,12 @@ ${indentation}    output = ${topTarget}.`;
                 const s = (_a = init.findFirstExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.concatTokens();
                 const t = (_b = init.findFirstExpression(Expressions.TypeName)) === null || _b === void 0 ? void 0 : _b.concatTokens();
                 if (s) {
-                    body += indentation + `DATA(${name}) = ${s}.\n`;
+                    if (s.toUpperCase().startsWith("VALUE #")) {
+                        body += indentation + `DATA(${name}) = ${s.replace("#", type)}.\n`;
+                    }
+                    else {
+                        body += indentation + `DATA(${name}) = ${s}.\n`;
+                    }
                 }
                 else {
                     body += indentation + `DATA ${name} TYPE ${t}.\n`;
@@ -52679,6 +52715,7 @@ ${indentation}    output = ${topTarget}.`;
             if (firstToken.getStr().toUpperCase() !== "VALUE") {
                 continue;
             }
+            const valueBody = s.findDirectExpression(Expressions.ValueBody);
             let type = this.findType(s, lowFile, highSyntax);
             if (type === undefined) {
                 if (high.get() instanceof Statements.Move && high.findDirectExpression(Expressions.Source) === s) {
@@ -52691,7 +52728,6 @@ ${indentation}    output = ${topTarget}.`;
             else {
                 type = "TYPE " + type;
             }
-            const valueBody = s.findDirectExpression(Expressions.ValueBody);
             const uniqueName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
             let indentation = " ".repeat(high.getFirstToken().getStart().getCol() - 1);
             let body = "";
@@ -52704,6 +52740,11 @@ ${indentation}    output = ${topTarget}.`;
             let added = false;
             let data = "";
             let previous = undefined;
+            if ((valueBody === null || valueBody === void 0 ? void 0 : valueBody.findDirectExpression(Expressions.ValueBodyLine)) !== undefined) {
+                structureName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
+                const extra = (valueBody === null || valueBody === void 0 ? void 0 : valueBody.findDirectExpression(Expressions.For)) ? "  " : "";
+                data = indentation + extra + `DATA ${structureName} LIKE LINE OF ${uniqueName}.\n`;
+            }
             for (const a of (valueBody === null || valueBody === void 0 ? void 0 : valueBody.getChildren()) || []) {
                 if (a.get() instanceof Expressions.FieldAssignment) {
                     if (added === false) {
@@ -52730,10 +52771,6 @@ ${indentation}    output = ${topTarget}.`;
                 if (a instanceof nodes_1.ExpressionNode && a.get() instanceof Expressions.ValueBodyLine) {
                     let skip = false;
                     for (const b of (a === null || a === void 0 ? void 0 : a.getChildren()) || []) {
-                        if (b.concatTokens() === "(" && added === false) {
-                            structureName = this.uniqueName(firstToken.getStart(), lowFile.getFilename(), highSyntax);
-                            data = indentation + `DATA ${structureName} LIKE LINE OF ${uniqueName}.\n`;
-                        }
                         if (b.get() instanceof Expressions.FieldAssignment) {
                             if (added === false) {
                                 body += data;
@@ -69140,7 +69177,13 @@ class SQLCondTranspiler {
             }
         }
         else {
-            ret += source.concatTokens();
+            const concat = source.concatTokens();
+            if (concat.startsWith("`")) {
+                ret += "'" + concat.substring(1, concat.length - 1) + "'";
+            }
+            else {
+                ret += concat;
+            }
         }
         return ret;
     }
@@ -74202,8 +74245,9 @@ class ReadTableTranspiler {
         if (keyName && node.findDirectTokenByText("COMPONENTS")) {
             extra.push("keyName: \"" + keyName.concatTokens() + "\"");
         }
+        const withTableKey = node.findTokenSequencePosition("WITH", "TABLE");
         const binary = node.findTokenSequencePosition("BINARY", "SEARCH");
-        if (binary) {
+        if (binary || withTableKey) {
             extra.push("binarySearch: true");
         }
         const rt = node.findDirectExpression(abaplint.Expressions.ReadTableTarget);
@@ -76511,6 +76555,7 @@ class SelectTranspiler {
         if (selectStatement === undefined) {
             throw "Structure, select loop not found";
         }
+        const concat = selectStatement.concatTokens().toUpperCase();
         const from = (_a = selectStatement.findFirstExpression(abaplint.Expressions.SQLFromSource)) === null || _a === void 0 ? void 0 : _a.concatTokens().toUpperCase();
         const intoName = new expressions_1.SQLTargetTranspiler().transpile(selectStatement.findFirstExpression(abaplint.Expressions.SQLTarget), traversal).getCode();
         // note: this implementation SELECTs everything into memory, which might be bad, and sometimes not correct
@@ -76519,7 +76564,12 @@ class SelectTranspiler {
         ret.appendString(`let ${targetName} = new abap.types.Table(abap.DDIC["${from}"].type);\n`);
         ret.appendChunk(new select_1.SelectTranspiler().transpile(selectStatement, traversal, targetName));
         ret.appendString(`\nfor (const ${loopName} of ${targetName}.array()) {\n`);
-        ret.appendString(`${intoName}.set(${loopName});\n`);
+        if (concat.includes(" INTO CORRESPONDING FIELDS OF ")) {
+            ret.appendString(`abap.statements.moveCorresponding(${loopName}, ${intoName});\n`);
+        }
+        else {
+            ret.appendString(`${intoName}.set(${loopName});\n`);
+        }
         const body = node.findDirectStructure(abaplint.Structures.Body);
         if (body) {
             ret.appendChunk(traversal.traverse(body));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.5.45",
+  "version": "2.5.47",
   "description": "Transpiler - Command Line Interface",
   "bin": {
     "abap_transpile": "./abap_transpile"
@@ -25,12 +25,12 @@
   "author": "abaplint",
   "license": "MIT",
   "devDependencies": {
-    "@abaplint/transpiler": "^2.5.45",
+    "@abaplint/transpiler": "^2.5.47",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
-    "@types/node": "^18.15.3",
-    "@abaplint/core": "^2.95.44",
+    "@types/node": "^18.15.5",
+    "@abaplint/core": "^2.95.47",
     "progress": "^2.0.3",
     "webpack": "^5.76.2",
     "webpack-cli": "^5.0.1",