SoDOff-Project/sodoff
1
0
Fork 1
mirror of https://github.com/SoDOff-Project/sodoff.git synced 2026-01-13 13:41:52 -08:00
Code Issues Packages Projects Releases Wiki Activity

clans: disallow lower roles from removing higher role members

Browse Source
This commit is contained in:
Spirtix 2025-12-30 14:22:22 +01:00
parent 20308d55e6
commit 9f923e69d8
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Controllers/Common/GroupController.cs
View File

@ -370,7 +370,10 @@ public class GroupController : Controller {
GroupMember? targetRole = vikingRole.Group.Vikings.FirstOrDefault(gv => gv.Viking.Uid == Guid.Parse(request.RemoveUserID)); GroupMember? targetRole = vikingRole.Group.Vikings.FirstOrDefault(gv => gv.Viking.Uid == Guid.Parse(request.RemoveUserID));
if (targetRole == null) if (targetRole == null)
return Ok(new RemoveMemberResult { Success = false, Status = RemoveMemberStatus.UserNotAMemberOfTheGroup }); return Ok(new RemoveMemberResult { Success = false, Status = RemoveMemberStatus.UserNotAMemberOfTheGroup });
if (targetRole.UserRole >= vikingRole.UserRole)
return Ok(new RemoveMemberResult { Success = false, Status = RemoveMemberStatus.InvalidParameters });
vikingRole.Group.Vikings.Remove(targetRole); vikingRole.Group.Vikings.Remove(targetRole);
if (!vikingRole.Group.Vikings.Any()) if (!vikingRole.Group.Vikings.Any())
ctx.Groups.Remove(vikingRole.Group); ctx.Groups.Remove(vikingRole.Group);