fix(Router) Fix detect active route in depth.

modules/angular2/src/router/router.ts

@@ -13,6 +13,7 @@ import {
 import {RouterOutlet} from './directives/router_outlet';
 import {getCanActivateHook} from './lifecycle/route_lifecycle_reflector';
 import {RouteDefinition} from './route_config/route_config_impl';
+import {DefaultInstruction} from "./instruction";
 
 let _resolveToTrue = PromiseWrapper.resolve(true);
 let _resolveToFalse = PromiseWrapper.resolve(false);
@@ -135,8 +136,9 @@ export class Router {
    */
   isRouteActive(instruction: Instruction): boolean {
     var router: Router = this;
+    var currentInstruction = this.currentInstruction;
 
-    if (isBlank(this.currentInstruction)) {
+    if (isBlank(currentInstruction)) {
       return false;
     }
 
@@ -146,22 +148,28 @@ export class Router {
       instruction = instruction.child;
     }
 
-    if (isBlank(instruction.component) || isBlank(this.currentInstruction.component) ||
-        this.currentInstruction.component.routeName != instruction.component.routeName) {
-      return false;
-    }
-
-    let paramEquals = true;
+    let reason = true;
 
-    if (isPresent(this.currentInstruction.component.params)) {
-      StringMapWrapper.forEach(instruction.component.params, (value, key) => {
-        if (this.currentInstruction.component.params[key] !== value) {
-          paramEquals = false;
-        }
-      });
-    }
+    // check the instructions in depth
+    do {
+      if (isBlank(instruction.component) || isBlank(currentInstruction.component) ||
+          currentInstruction.component.routeName != instruction.component.routeName) {
+        return false;
+      }
+      if (isPresent(instruction.component.params)) {
+        StringMapWrapper.forEach(instruction.component.params, (value, key) => {
+          if (currentInstruction.component.params[key] !== value) {
+            reason = false;
+          }
+        });
+      }
+      currentInstruction = currentInstruction.child;
+      instruction = instruction.child;
+    } while (isPresent(currentInstruction) && isPresent(instruction) &&
+             !(instruction instanceof DefaultInstruction) && reason);
 
-    return paramEquals;
+    // ignore DefaultInstruction
+    return reason && (isBlank(instruction) || instruction instanceof DefaultInstruction);
   }
 
 

modules/angular2/test/router/integration/router_link_spec.ts

@@ -303,6 +303,51 @@ export function main() {
                });
          }));
 
+      it('should not be added to links in other child routes',
+         inject([AsyncTestCompleter], (async) => {
+           router.config([
+                   new Route({path: '/child', component: HelloCmp, name: 'Child'}),
+                   new Route({
+                     path: '/child-with-grandchild/...',
+                     component: ParentCmp,
+                     name: 'ChildWithGrandchild'
+                   }),
+                   new Route({
+                     path: '/child-with-other-grandchild/...',
+                     component: ParentCmp,
+                     name: 'ChildWithOtherGrandchild'
+                   })
+                 ])
+               .then((_) => compile(`<a [routerLink]="['./Child']" class="child-link">Child</a>
+                                <a [routerLink]="['./ChildWithGrandchild/Grandchild']" class="child-with-grandchild-link">Better Child</a>
+                                <a [routerLink]="['./ChildWithOtherGrandchild/Grandchild']" class="child-with-other-grandchild-link">Better Child</a>
+                                <router-outlet></router-outlet>`))
+               .then((_) => {
+                 var element = fixture.debugElement.nativeElement;
+
+                 fixture.detectChanges();
+
+                 var link1 = DOM.querySelector(element, '.child-link');
+                 var link2 = DOM.querySelector(element, '.child-with-grandchild-link');
+                 var link3 = DOM.querySelector(element, '.child-with-other-grandchild-link');
+
+                 expect(link1).not.toHaveCssClass('router-link-active');
+                 expect(link2).not.toHaveCssClass('router-link-active');
+                 expect(link3).not.toHaveCssClass('router-link-active');
+
+                 router.subscribe((_) => {
+                   fixture.detectChanges();
+
+                   expect(link1).not.toHaveCssClass('router-link-active');
+                   expect(link2).toHaveCssClass('router-link-active');
+                   expect(link3).not.toHaveCssClass('router-link-active');
+
+                   async.done();
+                 });
+                 router.navigateByUrl('/child-with-grandchild/grandchild?extra=0');
+               });
+         }));
+
 
       describe('router link dsl', () => {
         it('should generate link hrefs with params', inject([AsyncTestCompleter], (async) => {