@abaplint/core 2.93.80 → 2.93.81

package/build/src/registry.js

@@ -63,7 +63,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.93.80";
+        return "2.93.81";
     }
     getDDICReferences() {
         return this.references;

package/build/src/rules/downport.js

@@ -24,6 +24,7 @@ const tokens_1 = require("../abap/1_lexer/tokens");
 const include_graph_1 = require("../utils/include_graph");
 const objects_1 = require("../objects");
 const _builtin_1 = require("../abap/5_syntax/_builtin");
+const _scope_type_1 = require("../abap/5_syntax/_scope_type");
 // todo: refactor each sub-rule to new classes?
 // todo: add configuration
 class DownportConf extends _basic_rule_config_1.BasicRuleConfig {
@@ -1967,16 +1968,29 @@ ${indentation}    output = ${topTarget}.`;
         const issue = issue_1.Issue.atPosition(lowFile, pos, "Rename before outline", this.getMetadata().key, this.conf.severity, fix);
         throw new SkipToNextFile(issue);
     }
-    isDuplicateName(spag, name, pos) {
-        var _a;
-        let isDuplicate = false;
-        for (const child of ((_a = spag.getParent()) === null || _a === void 0 ? void 0 : _a.getChildren()) || []) {
-            const found = child.findVariable(name);
-            if ((found === null || found === void 0 ? void 0 : found.getStart().equals(pos)) === false) {
-                isDuplicate = true;
+    findRecursiveDuplicate(spag, name, skip) {
+        const found = spag.findVariable(name);
+        if ((found === null || found === void 0 ? void 0 : found.getStart().equals(skip)) === false) {
+            return found;
+        }
+        for (const child of (spag === null || spag === void 0 ? void 0 : spag.getChildren()) || []) {
+            const sub = this.findRecursiveDuplicate(child, name, skip);
+            if (sub) {
+                return sub;
             }
         }
-        return isDuplicate;
+        return undefined;
+    }
+    isDuplicateName(spag, name, pos) {
+        let parent = spag.getParent();
+        while ((parent === null || parent === void 0 ? void 0 : parent.getIdentifier().stype) === _scope_type_1.ScopeType.Let
+            || (parent === null || parent === void 0 ? void 0 : parent.getIdentifier().stype) === _scope_type_1.ScopeType.For) {
+            parent = parent.getParent();
+        }
+        if (parent === undefined) {
+            return undefined;
+        }
+        return this.findRecursiveDuplicate(parent, name, pos) !== undefined;
     }
     findType(i, lowFile, highSyntax, ref = false) {
         var _a;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.93.80",
+    "version": "2.93.81",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",