AngularJS scopes inherit directly from their parent, so overwriting $emit/$broadcast on the $rootScope changes it for everyone. Which is a problem if the overwritten function uses $rootScope directly to broadcast, and not the given scope. ('$destroy' on a child scope for example can destroy many scopes it shouldn't).
Do we know what this breaks for the user? Generally we could merge it as long as we don't break anything else with it.
For it to be in 4.6.2 we would need a PR against
This caused at least the problem in ActivityLog, not sure if anything else is still undiscovered
Updated the base branch
@diosmosis some UI tests are now failing, not sure if this is expected?
@tsteur most failures are random, the rest are due to database version being different in the screenshot. I updated those.
👍 good to merge then
it's probably not needed to update screenshots in
next_release as it might be fixed then in 4.x-dev I would assume. Not sure what best flow is there
@tsteur ok, I won't update (I actually pushed them to the wrong branch)