@sebavan I’ve checked the diff between the 2 versions: Comparing 5.34.0...5.35.0 · BabylonJS/Babylon.js · GitHub
And it looks like the root observable class has been modified with: Observable: Do not include deleted observers in hasObservers result
- Observable: Do not include deleted observers in hasObservers result by Popov72 · Pull Request #13283 · BabylonJS/Babylon.js · GitHub
- Observable: Do not include deleted observers in hasObservers result by Popov72 · Pull Request #13283 · BabylonJS/Babylon.js · GitHub
I’ve just updated to v5.35.1 to reproduce the bug… but when I go to node_modules and manually comment this:
/**
* Gets a boolean indicating if the observable has at least one observer
* @returns true is the Observable has at least one Observer registered
*/
hasObservers() {
return this._observers.length/* - this._numObserversMarkedAsDeleted*/ > 0;
}
Everything is working fine
So… looks like there is a bug with your PR @Evgeni_Popov could you check please?
Edit: Related to Observable.hasObservers() is still true after removing all the observers - #4 by sharp