@abaplint/transpiler-cli 2.7.77 → 2.7.79

package/build/bundle.js

@@ -6007,7 +6007,8 @@ class SimpleTarget extends combi_1.Expression {
         const clas = (0, combi_1.seq)(_1.ClassName, (0, combi_1.tok)(tokens_1.StaticArrow), attribute_name_1.AttributeName);
         const start = (0, combi_1.altPrio)(cast, clas, _1.TargetField, _1.TargetFieldSymbol);
         const fields = (0, combi_1.seq)((0, combi_1.optPrio)(_1.FieldOffset), (0, combi_1.optPrio)(_1.FieldLength));
-        return (0, combi_1.seq)(start, something, fields);
+        const optional = (0, combi_1.altPrio)(_1.TableBody, fields);
+        return (0, combi_1.seq)(start, something, optional);
     }
 }
 exports.SimpleTarget = SimpleTarget;
@@ -6360,7 +6361,7 @@ exports.SQLCompareOperator = void 0;
 const combi_1 = __webpack_require__(/*! ../combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
 class SQLCompareOperator extends combi_1.Expression {
     getRunnable() {
-        const operator = (0, combi_1.altPrio)("=", "<>", "<", ">", "<=", ">=", "EQ", "NE", "GE", "GT", "LT", "LE", "><", "=>");
+        const operator = (0, combi_1.altPrio)("=", "<>", "<", ">", "<=", ">=", "EQ", "NE", "GE", "GT", "LT", "LE", "><", "=>", "=<");
         return operator;
     }
 }
@@ -13058,7 +13059,9 @@ class MoveCorresponding {
     getMatcher() {
         const keeping = (0, combi_1.ver)(version_1.Version.v740sp05, "KEEPING TARGET LINES");
         const expanding = (0, combi_1.ver)(version_1.Version.v740sp05, "EXPANDING NESTED TABLES");
-        const move = (0, combi_1.seq)("MOVE-CORRESPONDING", (0, combi_1.optPrio)("EXACT"), expressions_1.Source, "TO", expressions_1.Target, (0, combi_1.optPrio)(expanding), (0, combi_1.optPrio)(keeping));
+        const move = (0, combi_1.seq)("MOVE-CORRESPONDING", (0, combi_1.optPrio)("EXACT"), expressions_1.Source, "TO", 
+        // inline defintions not possible in this position,
+        expressions_1.SimpleTarget, (0, combi_1.optPrio)(expanding), (0, combi_1.optPrio)(keeping));
         return move;
     }
 }
@@ -23223,6 +23226,9 @@ class InlineData {
         var _a;
         const token = (_a = node.findFirstExpression(Expressions.TargetField)) === null || _a === void 0 ? void 0 : _a.getFirstToken();
         if (token && type) {
+            if (type instanceof basic_1.CSequenceType) {
+                type = basic_1.StringType.get();
+            }
             const identifier = new _typed_identifier_1.TypedIdentifier(token, filename, type, ["inline" /* IdentifierMeta.InlineDefinition */]);
             scope.addIdentifier(identifier);
             scope.addReference(token, identifier, _reference_1.ReferenceType.DataWriteReference, filename);
@@ -27336,7 +27342,7 @@ class Data {
             const id = new data_definition_1.DataDefinition().runSyntax(dd, scope, filename);
             if ((id === null || id === void 0 ? void 0 : id.getType().isGeneric()) === true
                 && (id === null || id === void 0 ? void 0 : id.getType().containsVoid()) === false) {
-                throw new Error("DATA definition cannot be generic, " + name);
+                throw new Error("DATA definition cannot be generic, " + (name === null || name === void 0 ? void 0 : name.concatTokens()));
             }
             return id;
         }
@@ -28959,7 +28965,7 @@ const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@ab
 class MoveCorresponding {
     runSyntax(node, scope, filename) {
         const s = node.findDirectExpression(Expressions.Source);
-        const t = node.findDirectExpression(Expressions.Target);
+        const t = node.findDirectExpression(Expressions.SimpleTarget);
         if (s === undefined || t === undefined) {
             throw new Error("MoveCorresponding, source or target not found");
         }
@@ -47786,7 +47792,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.25";
+        return "2.102.28";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -52685,7 +52691,7 @@ Make sure to test the downported code, it might not always be completely correct
                     }
                 }
             }
-            if (ret.length === 0 && lowFile.getRaw().includes(" xsdbool( ")) {
+            if (ret.length === 0 && lowFile.getRaw().includes(" xsdbool(")) {
                 for (let i = 0; i < lowStatements.length; i++) {
                     const high = highStatements[i];
                     const issue = this.replaceXsdBool(high, lowFile, highSyntax);
@@ -52869,7 +52875,7 @@ Make sure to test the downported code, it might not always be completely correct
         if (found) {
             return found;
         }
-        found = this.outlineDataSimple(high, lowFile);
+        found = this.outlineDataSimple(high, lowFile, highSyntax);
         if (found) {
             return found;
         }
@@ -53413,8 +53419,7 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
         const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
         return issue_1.Issue.atToken(lowFile, node.getFirstToken(), "Outline DATA", this.getMetadata().key, this.conf.severity, fix);
     }
-    outlineDataSimple(node, lowFile) {
-        // outlines "DATA(ls_msg) = temp1.", note that this does not need to look at types
+    outlineDataSimple(node, lowFile, highSyntax) {
         var _a, _b, _c;
         if (!(node.get() instanceof Statements.Move)) {
             return undefined;
@@ -53453,6 +53458,13 @@ ${indentation}CATCH ${className} INTO ${targetName}.`;
             }
         }
         else {
+            const spag = highSyntax.spaghetti.lookupPosition(source.getFirstToken().getStart(), lowFile.getFilename());
+            if (spag) {
+                const found = spag.findVariable(source.concatTokens());
+                if (found && found.getType().isGeneric() === true) {
+                    return undefined;
+                }
+            }
             type = source.concatTokens();
         }
         const targetName = (_c = target.findFirstExpression(Expressions.TargetField)) === null || _c === void 0 ? void 0 : _c.concatTokens();
@@ -53627,7 +53639,7 @@ ${indentation}CREATE OBJECT ${uniqueName2} EXPORTING textid = ${uniqueName1}.\n`
             return undefined;
         }
         const sourceRef = (_a = high.findFirstExpression(Expressions.Source)) === null || _a === void 0 ? void 0 : _a.findFirstExpression(Expressions.CorrespondingBody);
-        if ((sourceRef === null || sourceRef === void 0 ? void 0 : sourceRef.getChildren().length) === 1) {
+        if ((sourceRef === null || sourceRef === void 0 ? void 0 : sourceRef.getChildren().length) === 1 && target.concatTokens().toUpperCase().startsWith("DATA(") === false) {
             const code = `MOVE-CORRESPONDING ${sourceRef.concatTokens()} TO ${target.concatTokens()}`;
             const start = high.getFirstToken().getStart();
             const end = high.getLastToken().getStart();
@@ -77298,9 +77310,10 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.MoveCorrespondingTranspiler = void 0;
 const abaplint = __webpack_require__(/*! @abaplint/core */ "./node_modules/@abaplint/core/build/src/index.js");
 const chunk_1 = __webpack_require__(/*! ../chunk */ "./node_modules/@abaplint/transpiler/build/src/chunk.js");
+const expressions_1 = __webpack_require__(/*! ../expressions */ "./node_modules/@abaplint/transpiler/build/src/expressions/index.js");
 class MoveCorrespondingTranspiler {
     transpile(node, traversal) {
-        const target = traversal.traverse(node.findDirectExpression(abaplint.Expressions.Target));
+        const target = new expressions_1.TargetTranspiler().transpile(node.findDirectExpression(abaplint.Expressions.SimpleTarget), traversal);
         const source = traversal.traverse(node.findDirectExpression(abaplint.Expressions.Source));
         return new chunk_1.Chunk().append("abap.statements.moveCorresponding(", node, traversal)
             .join([source, target])
@@ -78097,7 +78110,7 @@ class SelectTranspiler {
   for await (const ${unique} of abap.statements.loop(${faeTranspiled})) {
     await abap.statements.select(${target}, {select: "${select.trim()}"${extra}}, {appending: true});
   }
-  if (!(${target} instanceof abap.types.HashedTable)) {
+  if (!(${target} instanceof abap.types.HashedTable) && ${target}.getOptions()?.primaryKey?.type !== "SORTED") {
     abap.statements.sort(${target}, {by: ${by}.map(k => { return {component: k}; })});
     await abap.statements.deleteInternal(${target}, {adjacent: true, by: ${by}});
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.77",
+  "version": "2.7.79",
   "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.77",
+    "@abaplint/transpiler": "^2.7.79",
     "@types/glob": "^7.2.0",
     "glob": "=7.2.0",
     "@types/progress": "^2.0.5",
-    "@types/node": "^20.5.1",
-    "@abaplint/core": "^2.102.25",
+    "@types/node": "^20.5.4",
+    "@abaplint/core": "^2.102.28",
     "progress": "^2.0.3",
     "webpack": "^5.88.2",
     "webpack-cli": "^5.1.4",