From 9f38f71a4cff43084e0487f6ba2cb984753f03a0 Mon Sep 17 00:00:00 2001 From: RogueException Date: Wed, 14 Oct 2015 19:56:42 -0300 Subject: [PATCH] Split SetStatus into SetStatus and SetGame --- src/Discord.Net/DiscordClient.API.cs | 31 ++++++++++++---------------- 1 file changed, 13 insertions(+), 18 deletions(-) diff --git a/src/Discord.Net/DiscordClient.API.cs b/src/Discord.Net/DiscordClient.API.cs index 3e981ceb7..b47e0b956 100644 --- a/src/Discord.Net/DiscordClient.API.cs +++ b/src/Discord.Net/DiscordClient.API.cs @@ -639,27 +639,22 @@ namespace Discord return _api.EditProfile(currentPassword: currentPassword, username: username, email: email ?? _currentUser?.Email, password: password, avatarType: avatarType, avatar: avatar); } - public Task SetStatus(string status = null, int? gameId = null) + public Task SetStatus(string status) { - if (status == null && gameId == null) - throw new ArgumentNullException("Either status or gameId must be non-null"); - - if (status != null) + switch (status) { - switch (status) - { - case UserStatus.Online: - case UserStatus.Away: - _status = status; - break; - default: - throw new ArgumentException($"Invalid status, must be {UserStatus.Online} or {UserStatus.Away}"); - } + case UserStatus.Online: + case UserStatus.Away: + _status = status; + break; + default: + throw new ArgumentException($"Invalid status, must be {UserStatus.Online} or {UserStatus.Away}"); } - - if (gameId != null) - _gameId = gameId; - + return SendStatus(); + } + public Task SetGame(int? gameId) + { + _gameId = gameId; return SendStatus(); } private Task SendStatus()