Elgato_dark/angular
1
0
Fork 0
mirror of https://github.com/angular/angular synced 2026-05-24 09:28:37 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(router): Fix outlet serialization and parsing with no primary children (#64505)

Browse source 
this fixes tree creation, serialization, and parsing of trees created with
children outlets and no primary path

fixes #62384

PR Close #64505
This commit is contained in:
Andrew Scott 2025-10-17 12:48:28 -07:00 committed by Jessica Janiuk
parent de8af17951
commit cfd8ed3fff
2 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/router/src/url_tree.ts
View file

@ -728,7 +728,7 @@ class UrlParser {
const children = this.parseChildren();
segments[outletName] =
Object.keys(children).length === 1
Object.keys(children).length === 1 && children[PRIMARY_OUTLET]
? children[PRIMARY_OUTLET]
: new UrlSegmentGroup([], children);
this.consumeOptional('//');

11
packages/router/test/url_tree.spec.ts
View file

@ -6,7 +6,8 @@
* found in the LICENSE file at https://angular.dev/license
*/
import {exactMatchOptions, subsetMatchOptions} from '../src/router';
import {TestBed} from '@angular/core/testing';
import {exactMatchOptions, Router, subsetMatchOptions} from '../src/router';
import {containsTree, DefaultUrlSerializer} from '../src/url_tree';
describe('UrlTree', () => {
@ -31,6 +32,14 @@ describe('UrlTree', () => {
const tree = serializer.parse('/path/to?first=http://foo/bar?baz=true&second=123');
expect(tree.queryParams).toEqual({'first': 'http://foo/bar?baz=true', 'second': '123'});
});
it('create, serialize, parse, serialize results in same serialized tree with outlet and no primary children', () => {
const router = TestBed.inject(Router);
const th = router.createUrlTree(['/', {outlets: {a: ['a'], b: [{outlets: {a: ['b1']}}]}}]);
const serialized = router.serializeUrl(th);
const p = router.parseUrl(serialized);
expect(router.serializeUrl(p)).toBe(serialized);
});
});
describe('containsTree', () => {