summary refs log tree commit diff
diff options
context:
space:
mode:
authorRory& <root@rory.gay>2026-06-24 06:59:24 +0200
committerRory& <root@rory.gay>2026-06-24 06:59:24 +0200
commitb9c608de962c12885402b8773b30656c485b9903 (patch)
treecb53a46634f89326e67bbb8758a8d7911b05b9c5
parentop14: deduplicate group adding code (diff)
downloadserver-ts-b9c608de962c12885402b8773b30656c485b9903.tar.xz
op14: dont account for roles when including offline members
-rw-r--r--extra/admin-api/Spacebar.Offload/Controllers/Op14Controller.cs5
1 files changed, 2 insertions, 3 deletions
diff --git a/extra/admin-api/Spacebar.Offload/Controllers/Op14Controller.cs b/extra/admin-api/Spacebar.Offload/Controllers/Op14Controller.cs

index 9b676bad..ab18a969 100644 --- a/extra/admin-api/Spacebar.Offload/Controllers/Op14Controller.cs +++ b/extra/admin-api/Spacebar.Offload/Controllers/Op14Controller.cs
@@ -169,9 +169,8 @@ public class Op14Controller(ILogger<Op12Controller> logger, SpacebarAspNetAuthen logger.LogInformation("Less than 2000 members, including offline members..."); var offlineMembers = await baseMembersQry .Where(x => - !x.Roles.Any(r => handledRoles.Contains(r.Id)) - // and finally, filter by online - && (x.IdNavigation.Sessions.All(s => s.Status == "offline" || s.Status == "invisible" || s.Status == "unknown") || !x.IdNavigation.Sessions.Any()) + x.IdNavigation.Sessions.All(s => s.Status == "offline" || s.Status == "invisible" || s.Status == "unknown") + || !x.IdNavigation.Sessions.Any() ).ToListAsync(); AddGroupIfNonEmpty("offline", "offline", offlineMembers);