chrisp/brackt
1
1
Fork 0
Code Issues 42 Pull requests Projects Releases Packages Wiki Activity Actions 1

Fix no-non-null-assertion lint errors in isScoring Map lookups

Browse source 
Replace Map.has(key) ? Map.get(key)! : fallback pattern with
Map.get(key) ?? fallback to satisfy oxlint no-non-null-assertion rule.

https://claude.ai/code/session_01D3NbnVQdaH82Fk7Jm8y3sB
This commit is contained in:
Claude 2026-04-15 18:22:41 +00:00
parent 8d4b1eb238
commit 309381e43b
No known key found for this signature in database
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/routes/admin.sports-seasons.$id.events.$eventId.bracket.server.ts
View file

@ -418,7 +418,7 @@ export async function action({ request, params }: Route.ActionArgs) {
round: match.round,
winnerId,
loserId,
isScoring: roundIsScoring.has(match.round) ? roundIsScoring.get(match.round)! : (match.isScoring ?? true),
isScoring: roundIsScoring.get(match.round) ?? (match.isScoring ?? true),
sportsSeasonId: event.sportsSeasonId,
bracketTemplateId: event.bracketTemplateId,
eventName: event.name ?? undefined,
@ -619,9 +619,7 @@ export async function action({ request, params }: Route.ActionArgs) {
// defaults to true, so legacy play-in rows that predate the column are
// incorrectly marked as scoring and would assign finalPosition=0 to losers
// who still have a second game (e.g. NBA 7v8 play-in loser → Round 2).
const isScoring = templateRoundIsScoring.has(match.round)
? templateRoundIsScoring.get(match.round)!
: (match.isScoring ?? true);
const isScoring = templateRoundIsScoring.get(match.round) ?? (match.isScoring ?? true);
await processMatchResult(
{
round: match.round,