From 4a448321149343837da10e08fcd4fdb57f9f7545 Mon Sep 17 00:00:00 2001 From: James Blacklock Date: Fri, 26 Aug 2016 18:41:32 -0400 Subject: [PATCH] fix(UrlParser) stop setting default value 'true' (matrix params) (#10946) This was already fixed recently for query params in #10399. --- modules/@angular/router/src/url_tree.ts | 2 +- modules/@angular/router/test/url_serializer.spec.ts | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/modules/@angular/router/src/url_tree.ts b/modules/@angular/router/src/url_tree.ts index 5b7358d31f..14554051ba 100644 --- a/modules/@angular/router/src/url_tree.ts +++ b/modules/@angular/router/src/url_tree.ts @@ -391,7 +391,7 @@ class UrlParser { return; } this.capture(key); - let value: any = 'true'; + let value: any = ''; if (this.peekStartsWith('=')) { this.capture('='); const valueMatch = matchSegments(this.remaining); diff --git a/modules/@angular/router/test/url_serializer.spec.ts b/modules/@angular/router/test/url_serializer.spec.ts index 61312571b4..463b3c43da 100644 --- a/modules/@angular/router/test/url_serializer.spec.ts +++ b/modules/@angular/router/test/url_serializer.spec.ts @@ -113,9 +113,9 @@ describe('url serializer', () => { it('should parse key only matrix params', () => { const tree = url.parse('/one;a'); - expectSegment(tree.root.children[PRIMARY_OUTLET], 'one;a=true'); + expectSegment(tree.root.children[PRIMARY_OUTLET], 'one;a='); - expect(url.serialize(tree)).toEqual('/one;a=true'); + expect(url.serialize(tree)).toEqual('/one;a='); }); it('should parse query params (root)', () => {