Arsharth/hydrogen-web
1
0
Fork 0
forked from mystiq/hydrogen-web
Code Issues Pull requests Projects Releases Packages Wiki Activity

dispose existing view model when changing status, otherwise we leak!

Browse source
This commit is contained in:
Bruno Windels 2021-05-11 16:39:33 +02:00
parent 8b8214cd1b
commit a0e3848cf6
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/domain/session/RoomViewModelObservable.js
View file

@ -51,6 +51,8 @@ export class RoomViewModelObservable extends ObservableValue {
this._statusObservable = await session.observeRoomStatus(this.id); this._statusObservable = await session.observeRoomStatus(this.id);
this.set(await this._statusToViewModel(this._statusObservable.get())); this.set(await this._statusToViewModel(this._statusObservable.get()));
this._statusObservable.subscribe(async status => { this._statusObservable.subscribe(async status => {
// first dispose existing VM, if any
this.get()?.dispose();
this.set(await this._statusToViewModel(status)); this.set(await this._statusToViewModel(status));
}); });
} }