cargdev/angular
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(router): handle empty path with query params

Browse Source
This commit is contained in:
vsavkin 2016-06-25 12:07:47 -07:00
parent 90295e3252
commit 3f44377f2f
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -277,7 +277,11 @@ class UrlParser {
} }
parseRootSegment(): UrlSegment { parseRootSegment(): UrlSegment {
if (this.remaining === '' || this.remaining === '/') { if (this.remaining.startsWith('/')) {
this.capture('/');
}
if (this.remaining === '' || this.remaining.startsWith('?')) {
return new UrlSegment([], {}); return new UrlSegment([], {});
} else { } else {
return new UrlSegment([], this.parseSegmentChildren()); return new UrlSegment([], this.parseSegmentChildren());

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

@ -91,6 +91,13 @@ describe('url serializer', () => {
expect(url.serialize(tree)).toEqual('/one;a=true'); expect(url.serialize(tree)).toEqual('/one;a=true');
}); });
it('should parse query params (root)', () => {
const tree = url.parse('/?a=1&b=2');
expect(tree.root.children).toEqual({});
expect(tree.queryParams).toEqual({a: '1', b: '2'});
expect(url.serialize(tree)).toEqual('/?a=1&b=2');
});
it('should parse query params', () => { it('should parse query params', () => {
const tree = url.parse('/one?a=1&b=2'); const tree = url.parse('/one?a=1&b=2');
expect(tree.queryParams).toEqual({a: '1', b: '2'}); expect(tree.queryParams).toEqual({a: '1', b: '2'});