Skip to content

Commit 45f15cf

Browse files
Log error to console and do not send message to player when there is an error giving a permission group reward to player
1 parent db8af37 commit 45f15cf

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

PlayerStats/Components/PlayerStatsComponent.cs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
using RestoreMonarchy.PlayerStats.Helpers;
22
using RestoreMonarchy.PlayerStats.Models;
3+
using Rocket.API;
34
using Rocket.API.Serialisation;
45
using Rocket.Core;
56
using Rocket.Unturned.Player;
@@ -228,7 +229,13 @@ internal void CheckGiveReward()
228229
List<RocketPermissionsGroup> groups = R.Permissions.GetGroups(unturnedPlayer, false);
229230
if (!groups.Exists(x => x.Id.Equals(reward.PermissionGroup, StringComparison.OrdinalIgnoreCase)))
230231
{
231-
R.Permissions.AddPlayerToGroup(reward.PermissionGroup, unturnedPlayer);
232+
RocketPermissionsProviderResult result = R.Permissions.AddPlayerToGroup(reward.PermissionGroup, unturnedPlayer);
233+
if (result != RocketPermissionsProviderResult.Success)
234+
{
235+
Logger.Log($"Warning >> Failed to give {reward.PermissionGroup} permission group as reward to {unturnedPlayer.DisplayName}. Reason: {result}", ConsoleColor.Yellow);
236+
return;
237+
}
238+
232239
string treshold = reward.Treshold.ToString("N0");
233240
if (configuration.ActualStatsMode == StatsMode.Both || configuration.ActualStatsMode == StatsMode.PVP)
234241
{

PlayerStats/PlayerStats.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<TargetFramework>net48</TargetFramework>
55
<LangVersion>latest</LangVersion>
66
<RootNamespace>RestoreMonarchy.PlayerStats</RootNamespace>
7-
<Version>1.1.0</Version>
7+
<Version>1.1.1</Version>
88
</PropertyGroup>
99

1010
<ItemGroup>

0 commit comments

Comments
 (0)