fix(cdk/tree): remove leaking subscription

crisbeto · crisbeto · commit 782305df4bb4 · 2025-06-27T09:10:19.000+02:00
Fixes that we weren't unsubscribing from one observable in the tree. Fixes #31454.
diff --git a/src/cdk/tree/tree.ts b/src/cdk/tree/tree.ts
@@ -1400,6 +1400,7 @@ export class CdkTreeNode<T, K = T> implements OnDestroy, OnInit, TreeKeyManagerI
       .changed.pipe(
         map(() => this.isExpanded),
         distinctUntilChanged(),
+        takeUntil(this._destroyed),
       )
       .subscribe(() => this._changeDetectorRef.markForCheck());
     this._tree._setNodeTypeIfUnset(this._type);

Original file line number	Diff line number	Diff line change
`@@ -1400,6 +1400,7 @@ export class CdkTreeNode<T, K = T> implements OnDestroy, OnInit, TreeKeyManagerI`
`1400`	`1400`	`.changed.pipe(`
`1401`	`1401`	`map(() => this.isExpanded),`
`1402`	`1402`	`distinctUntilChanged(),`
	`1403`	`+ takeUntil(this._destroyed),`
`1403`	`1404`	`)`
`1404`	`1405`	`.subscribe(() => this._changeDetectorRef.markForCheck());`
`1405`	`1406`	`this._tree._setNodeTypeIfUnset(this._type);`