From 9a4371a81748b821ae452f93c52763043d670097 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bart=C5=82omiej=20Dach?= Date: Thu, 3 Apr 2025 10:03:06 +0200 Subject: [PATCH] Add `[JsonIgnore]` to `MultiplayerRoom.CurrentPlaylistItem` The lack of this is currently failing a unit test on `osu-server-spectator` current master: https://github.com/ppy/osu-server-spectator/actions/runs/14193158383/job/39762243965#step:4:28 I don't think the failure actually matters because I don't think we're using json serialisation on spectator server side anywhere (used to for iOS at least, but I don't think we do anymore?), but probably better to be safe than sorry. --- osu.Game/Online/Multiplayer/MultiplayerRoom.cs | 1 + 1 file changed, 1 insertion(+) diff --git a/osu.Game/Online/Multiplayer/MultiplayerRoom.cs b/osu.Game/Online/Multiplayer/MultiplayerRoom.cs index db1722af8c..3c02565fa1 100644 --- a/osu.Game/Online/Multiplayer/MultiplayerRoom.cs +++ b/osu.Game/Online/Multiplayer/MultiplayerRoom.cs @@ -85,6 +85,7 @@ namespace osu.Game.Online.Multiplayer /// Retrieves the active as determined by the room's current settings. /// [IgnoreMember] + [JsonIgnore] public MultiplayerPlaylistItem CurrentPlaylistItem => Playlist.Single(item => item.ID == Settings.PlaylistItemId); ///