mirror of
https://github.com/ppy/osu
synced 2025-01-25 07:13:22 +00:00
Fix MultiplayerMatchSubScreen
mutating mods outside of bindable lease
As seen at https://github.com/peppy/osu/runs/4674501626?check_suite_focus=true.
This commit is contained in:
parent
f656e7d2d2
commit
675bdd3213
@ -381,7 +381,7 @@ namespace osu.Game.Screens.OnlinePlay.Match
|
||||
|
||||
protected virtual void UpdateMods()
|
||||
{
|
||||
if (SelectedItem.Value == null)
|
||||
if (SelectedItem.Value == null || !this.IsCurrentScreen())
|
||||
return;
|
||||
|
||||
Mods.Value = UserMods.Value.Concat(SelectedItem.Value.RequiredMods).ToList();
|
||||
|
@ -241,7 +241,7 @@ namespace osu.Game.Screens.OnlinePlay.Multiplayer
|
||||
|
||||
protected override void UpdateMods()
|
||||
{
|
||||
if (SelectedItem.Value == null || client.LocalUser == null)
|
||||
if (SelectedItem.Value == null || client.LocalUser == null || !this.IsCurrentScreen())
|
||||
return;
|
||||
|
||||
// update local mods based on room's reported status for the local user (omitting the base call implementation).
|
||||
|
Loading…
Reference in New Issue
Block a user