@abaplint/cli 2.102.34 → 2.102.36

package/build/cli.js

@@ -1020,7 +1020,7 @@ class Rename {
                 }
                 const newStr = o.getName().replace(regex, p.newName);
                 if (quiet !== true) {
-                    console.log("Renaming " + o.getName().padEnd(30, " ") + " -> " + newStr);
+                    console.log("Renaming " + o.getType() + " " + o.getName().padEnd(30, " ") + " -> " + newStr);
                 }
                 const result = renamer.rename(o.getType(), o.getName(), newStr);
                 result.updatedFiles.forEach(f => { this.updatedFiles.add(f); });
@@ -13651,10 +13651,11 @@ class Loop {
     getMatcher() {
         const where = (0, combi_1.seq)("WHERE", (0, combi_1.alt)(expressions_1.ComponentCond, expressions_1.Dynamic));
         const group = (0, combi_1.ver)(version_1.Version.v740sp08, (0, combi_1.seq)("GROUP BY", loop_group_by_1.LoopGroupBy));
+        const step = (0, combi_1.ver)(version_1.Version.v757, (0, combi_1.seq)("STEP", expressions_1.Source));
         const from = (0, combi_1.seq)("FROM", expressions_1.Source);
         const to = (0, combi_1.seq)("TO", expressions_1.Source);
         const usingKey = (0, combi_1.seq)("USING KEY", (0, combi_1.altPrio)(expressions_1.SimpleName, expressions_1.Dynamic));
-        const options = (0, combi_1.per)(expressions_1.LoopTarget, from, to, where, usingKey, group);
+        const options = (0, combi_1.per)(expressions_1.LoopTarget, from, to, where, usingKey, group, step);
         const at = (0, combi_1.seq)("AT", (0, combi_1.opt)((0, combi_1.seq)("SCREEN", (0, combi_1.failCombinator)())), (0, combi_1.opt)((0, combi_1.ver)(version_1.Version.v740sp08, "GROUP")), (0, combi_1.alt)(simple_source2_1.SimpleSource2, (0, combi_1.ver)(version_1.Version.v740sp02, expressions_1.Source)), (0, combi_1.opt)(options));
         return (0, combi_1.seq)("LOOP", (0, combi_1.opt)(at));
     }
@@ -14758,7 +14759,7 @@ const version_1 = __webpack_require__(/*! ../../../version */ "./node_modules/@a
 class ReadTextpool {
     getMatcher() {
         const language = (0, combi_1.seq)("LANGUAGE", expressions_1.Source);
-        const into = (0, combi_1.seq)("INTO", expressions_1.Target);
+        const into = (0, combi_1.seq)("INTO", expressions_1.SimpleTarget);
         const state = (0, combi_1.seq)("STATE", expressions_1.Source);
         const ret = (0, combi_1.seq)("READ TEXTPOOL", expressions_1.Source, (0, combi_1.per)(into, language, state));
         return (0, combi_1.verNot)(version_1.Version.Cloud, ret);
@@ -21900,6 +21901,9 @@ class TypeUtils {
             else if (target.containsVoid() === true) {
                 return true;
             }
+            else if (source instanceof basic_1.StringType && this.structureContainsString(target)) {
+                return false;
+            }
             else if (source instanceof basic_1.IntegerType) {
                 return false;
             }
@@ -30513,7 +30517,7 @@ class ReadTextpool {
         for (const s of node.findDirectExpressions(Expressions.Source)) {
             new source_1.Source().runSyntax(s, scope, filename);
         }
-        for (const t of node.findDirectExpressions(Expressions.Target)) {
+        for (const t of node.findDirectExpressions(Expressions.SimpleTarget)) {
             new target_1.Target().runSyntax(t, scope, filename);
         }
     }
@@ -46770,9 +46774,10 @@ class RenamerHelper {
     renameFiles(obj, oldName, name) {
         const list = [];
         const newName = name.toLowerCase().replace(/\//g, "#");
+        oldName = oldName.toLowerCase().replace(/\//g, "#");
         for (const f of obj.getFiles()) {
             // todo, this is not completely correct, ie. if the URI contains the same directory name as the object name
-            const newFilename = f.getFilename().replace(oldName.toLowerCase(), newName.toLowerCase());
+            const newFilename = f.getFilename().replace(oldName, newName);
             list.push(vscode_languageserver_types_1.RenameFile.create(f.getFilename(), newFilename));
         }
         return list;
@@ -48777,7 +48782,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.34";
+        return "2.102.36";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -60881,7 +60886,7 @@ class MessageExistsRule {
                         const msag = this.reg.getObject("MSAG", ref.messageClass);
                         if (msag === undefined) {
                             if (new ddic_1.DDIC(this.reg).inErrorNamespace(ref.messageClass) === true) {
-                                const message = "Message class \"" + ref.token.getStr() + "\" not found";
+                                const message = "Message class \"" + ref.messageClass + "\" not found";
                                 issues.push(issue_1.Issue.atToken(file, ref.token, message, this.getMetadata().key, this.conf.severity));
                             }
                             continue;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.34",
+    "version": "2.102.36",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.34",
-        "@types/chai": "^4.3.5",
+        "@abaplint/core": "^2.102.36",
+        "@types/chai": "^4.3.6",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",
         "@types/mocha": "^10.0.1",
-        "@types/node": "^20.5.7",
+        "@types/node": "^20.6.0",
         "@types/progress": "^2.0.5",
         "chai": "^4.3.8",
         "chalk": "^5.3.0",
-        "eslint": "^8.48.0",
+        "eslint": "^8.49.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
         "memfs": "^4.2.1",