not including notContext scopes in correct paths (fixes #166)

canjs · Jun 1, 2018 · bdef021 · bdef021
1 parent aef1a03
commit bdef021
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/can-view-scope.js b/can-view-scope.js
@@ -446,11 +446,13 @@ assign(Scope.prototype, {
 			// `notContext` and `special` contexts can't be read using `../`
 			var canBeRead = !scope._meta.special &&  !scope._meta.notContext;
 
-			if (canBeRead && typeof scope._context === "object" && canReflect.hasKey(scope._context, normalizedKey)) {
-				paths[cur + normalizedKey] = scope._context;
-			}
+			if (canBeRead) {
+				if (typeof scope._context === "object" && canReflect.hasKey(scope._context, normalizedKey)) {
+					paths[cur + normalizedKey] = scope._context;
+				}
 
-			cur += "../";
+				cur += "../";
+			}
 
 			// walk entire scope tree
 			return false;

diff --git a/test/scope-test.js b/test/scope-test.js
@@ -1305,8 +1305,8 @@ QUnit.test("scope.getPathsForKey", function() {
 		"scope.vm.name": vm,
 		"scope.top.name": top,
 		"name": nonVm,
-		"../../../name": vm,
-		"../../../../../name": top
+		"../../name": vm,
+		"../../../name": top
 	});
 });