Toutsu/GmRelayBot
1
1
Fork 0
Code Issues 18 Pull Requests Actions 1 Packages Projects Releases 85 Wiki Activity

fix(bot): trim misleading comment in IsComplete
PR Checks / test-and-build (pull_request) Successful in 12m9s
Details

Browse Source 
The previous comment claimed a 0 MaxPlayers would also be blocked, but
the code never actually enforced that. The wizard's text-input path
already guarantees cap >= MinCapacity, so 0 is unreachable. Drop the
inaccurate half-sentence to keep the comment faithful to the code.
This commit is contained in:
Toutsu
2026-06-09 13:33:53 +03:00
parent 320ec18ab0
commit 37ed697696
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/GmRelay.Bot/Features/Sessions/CreateSession/CreateSessionHandler.cs
+1 -4
View File
@@ -230,10 +230,7 @@ public sealed class CreateSessionHandler
{ {
if (p.Single?.ScheduledAt is null) missingFields.Add("дата/время"); if (p.Single?.ScheduledAt is null) missingFields.Add("дата/время");
// MaxPlayers = null is a valid "♾ Без лимита" choice // MaxPlayers = null is a valid "♾ Без лимита" choice
// (see GameCreationWizard.ApplyCapacityChoice "no_limit"). Only a // (see GameCreationWizard.ApplyCapacityChoice "no_limit").
// missing wizard step on the numeric-input path is a real defect;
// a non-null 0 also blocks (zero players is never what the user
// asked for). Treat null as "no limit" and accept it.
} }
else else
{ {