h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2024-12-16 00:52:59 +08:00
Code Issues Releases Wiki Activity

Permitted nulls in ChatOverlay.load() so that in testing, no "OsuGame" instance is required. Also added null checks to the links' on click actions

Browse Source
This commit is contained in:
FreezyLemon 2017-12-07 10:31:02 +01:00
parent 006ac44e11
commit ec8b5c2465
2 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
osu.Game/Graphics/Sprites/OsuLinkSpriteText.cs
View File

@ -99,7 +99,7 @@ namespace osu.Game.Graphics.Sprites
break; break;
case "edit": case "edit":
chat.Game.LoadEditorTimestamp(); chat.Game?.LoadEditorTimestamp();
break; break;
case "b": case "b":
if (args.Length > 1 && int.TryParse(args[1], out int mapId)) if (args.Length > 1 && int.TryParse(args[1], out int mapId))
@ -122,11 +122,9 @@ namespace osu.Game.Graphics.Sprites
req.Success += user => req.Success += user =>
{ {
// TODO: Open spectator screen and start spectating chat.Game?.LoadSpectatorScreen();
}; };
// api.Queue(req); api.Queue(req);
chat.Game.LoadSpectatorScreen();
break; break;
default: default:
@ -139,7 +137,7 @@ namespace osu.Game.Graphics.Sprites
if (!int.TryParse(url.Split('/').ElementAtOrDefault(1), out int multiId)) if (!int.TryParse(url.Split('/').ElementAtOrDefault(1), out int multiId))
return; return;
chat.Game.LoadMultiplayerLobby(); chat.Game?.LoadMultiplayerLobby();
} }
else if (url.StartsWith("http://") || url.StartsWith("https://")) else if (url.StartsWith("http://") || url.StartsWith("https://"))
{ {

6
osu.Game/Overlays/ChatOverlay.cs
View File

@ -41,6 +41,9 @@ namespace osu.Game.Overlays
private readonly FocusedTextBox textbox; private readonly FocusedTextBox textbox;
/// <summary>
/// The current OsuGame instance. Will be null for Tests.
/// </summary>
public OsuGame Game; public OsuGame Game;
private APIAccess api; private APIAccess api;
@ -272,9 +275,10 @@ namespace osu.Game.Overlays
base.PopOut(); base.PopOut();
} }
[BackgroundDependencyLoader] [BackgroundDependencyLoader(true)]
private void load(APIAccess api, OsuConfigManager config, OsuColour colours, OsuGame game) private void load(APIAccess api, OsuConfigManager config, OsuColour colours, OsuGame game)
{ {
// game will be null in testing, so some links will not work
Game = game; Game = game;
this.api = api; this.api = api;
api.Register(this); api.Register(this);