@abaplint/cli 2.102.11 → 2.102.13

package/build/cli.js

@@ -48557,7 +48557,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.11";
+        return "2.102.13";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -53194,6 +53194,7 @@ const objects_1 = __webpack_require__(/*! ../objects */ "./node_modules/@abaplin
 const _builtin_1 = __webpack_require__(/*! ../abap/5_syntax/_builtin */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_builtin.js");
 const _scope_type_1 = __webpack_require__(/*! ../abap/5_syntax/_scope_type */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_scope_type.js");
 const statements_1 = __webpack_require__(/*! ../abap/2_statements/statements */ "./node_modules/@abaplint/core/build/src/abap/2_statements/statements/index.js");
+const crypto = __webpack_require__(/*! node:crypto */ "node:crypto");
 // todo: refactor each sub-rule to new classes?
 // todo: add configuration
 class DownportConf extends _basic_rule_config_1.BasicRuleConfig {
@@ -53960,7 +53961,7 @@ ${indentation}`);
         if (tt === undefined) {
             return undefined;
         }
-        const uniqueName = this.uniqueName(high.getFirstToken().getStart(), lowFile.getFilename(), highSyntax, "ty_");
+        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);
@@ -55705,15 +55706,21 @@ ${indentation}    output = ${uniqueName}.\n`;
         }
         return undefined;
     }
-    uniqueName(position, filename, highSyntax, prefix = "") {
+    uniqueName(position, filename, highSyntax) {
         const spag = highSyntax.spaghetti.lookupPosition(position, filename);
         if (spag === undefined) {
-            const name = prefix + "temprr" + this.counter;
+            const name = "temprr" + this.counter;
             this.counter++;
             return name;
         }
+        let postfix = "";
+        if (spag.getIdentifier().stype === _scope_type_1.ScopeType.ClassDefinition) {
+            // try making sure this name is not used in subclasses
+            const hash = crypto.createHash("sha1").update(spag.getIdentifier().sname).digest("hex");
+            postfix = "_" + hash.substring(0, 10);
+        }
         while (true) {
-            const name = prefix + "temp" + this.counter;
+            const name = "temp" + this.counter + postfix;
             const exists = this.existsRecursive(spag, name);
             this.counter++;
             if (exists === false) {
@@ -55721,13 +55728,14 @@ ${indentation}    output = ${uniqueName}.\n`;
             }
         }
     }
+    // todo, optimize, the findVariable() and findType() does a lot of redundant checks
     existsRecursive(spag, name) {
-        const existsDirect = spag.findVariable(name);
+        const existsDirect = spag.findVariable(name) || spag.findType(name);
         if (existsDirect) {
             return true;
         }
         for (const child of spag.getChildren()) {
-            if (child.findVariable(name) || this.existsRecursive(child, name)) {
+            if (child.findVariable(name) || child.findType(name) || this.existsRecursive(child, name)) {
                 return true;
             }
         }
@@ -80948,6 +80956,17 @@ module.exports = require("fs");
 
 /***/ }),
 
+/***/ "node:crypto":
+/*!******************************!*\
+  !*** external "node:crypto" ***!
+  \******************************/
+/***/ ((module) => {
+
+"use strict";
+module.exports = require("node:crypto");
+
+/***/ }),
+
 /***/ "node:os":
 /*!**************************!*\
   !*** external "node:os" ***!

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.11",
+    "version": "2.102.13",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,7 +38,7 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.11",
+        "@abaplint/core": "^2.102.13",
         "@types/chai": "^4.3.5",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.2",