honeymoose/angular-cn
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(UrlParser) stop setting default value 'true' (matrix params) (#10946)

Browse Source 
This was already fixed recently for query params in #10399.
This commit is contained in:
James Blacklock 2016-08-26 18:41:32 -04:00 committed by Victor Berchet
parent 27539c8b80
commit 4a44832114
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
modules/@angular/router/src/url_tree.ts
View File

@ -391,7 +391,7 @@ class UrlParser {
return; return;
} }
this.capture(key); this.capture(key);
let value: any = 'true'; let value: any = '';
if (this.peekStartsWith('=')) { if (this.peekStartsWith('=')) {
this.capture('='); this.capture('=');
const valueMatch = matchSegments(this.remaining); const valueMatch = matchSegments(this.remaining);

4
modules/@angular/router/test/url_serializer.spec.ts
View File

@ -113,9 +113,9 @@ describe('url serializer', () => {
it('should parse key only matrix params', () => { it('should parse key only matrix params', () => {
const tree = url.parse('/one;a'); 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)', () => { it('should parse query params (root)', () => {