@abaplint/transpiler-cli 2.7.76 → 2.7.78

package/build/bundle.js

@@ -6360,7 +6360,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 +13058,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 +23225,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 +27341,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 +28964,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 +47791,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.25";
+        return "2.102.27";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -52685,7 +52690,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 +52874,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 +53418,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 +53457,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();
@@ -70171,10 +70182,13 @@ class CompareTranspiler {
                 return new chunk_1.Chunk().appendString(pre + "INPUT && INPUT." + concat.replace(" IS NOT SUPPLIED", "").toLowerCase() + " === undefined");
             }
             if (concat.endsWith(" IS REQUESTED")) {
-                return new chunk_1.Chunk().appendString(pre + "INPUT && INPUT." + concat.replace(" IS REQUESTED", "").toLowerCase());
+                const field = concat.replace(" IS REQUESTED", "").toLowerCase();
+                // yea, for function modules the naming is revereed, using "importing"
+                return new chunk_1.Chunk().appendString(pre + "INPUT && (INPUT." + field + " || INPUT.importing." + field + ")");
             }
             else if (concat.endsWith(" IS NOT REQUESTED")) {
-                return new chunk_1.Chunk().appendString(pre + "INPUT && INPUT." + concat.replace(" IS NOT REQUESTED", "").toLowerCase() + " === undefined");
+                const field = concat.replace(" IS NOT REQUESTED", "").toLowerCase();
+                return new chunk_1.Chunk().appendString(pre + "INPUT && INPUT." + field + " === undefined && INPUT.importing." + field + " === undefined");
             }
             if (concat.startsWith("NOT ") || concat.includes(" IS NOT INSTANCE OF ")) {
                 const cname = (_a = node.findDirectExpression(core_1.Expressions.ClassName)) === null || _a === void 0 ? void 0 : _a.concatTokens();
@@ -77295,9 +77309,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])

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abaplint/transpiler-cli",
-  "version": "2.7.76",
+  "version": "2.7.78",
   "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.76",
+    "@abaplint/transpiler": "^2.7.78",
     "@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.3",
+    "@abaplint/core": "^2.102.27",
     "progress": "^2.0.3",
     "webpack": "^5.88.2",
     "webpack-cli": "^5.1.4",