From c1d1b4be51024b53be689195106a670151b40cc6 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Fri, 27 Mar 2026 04:03:55 -0500 Subject: [PATCH] fix: replace integer comparisons on boolean fields with True/False (#69) Closes #69 Co-Authored-By: Claude Sonnet 4.6 --- app/db_engine.py | 12 ++++++------ app/routers_v3/transactions.py | 8 ++++---- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/app/db_engine.py b/app/db_engine.py index c1979cf..8f02e2c 100644 --- a/app/db_engine.py +++ b/app/db_engine.py @@ -435,10 +435,10 @@ class Team(BaseModel): active_roster[pos] += 1 all_drops = Transaction.select_season(Current.latest().season).where( - (Transaction.oldteam == self) & (Transaction.week == current.week + 1) & (Transaction.cancelled == 0) + (Transaction.oldteam == self) & (Transaction.week == current.week + 1) & (Transaction.cancelled == False) ) all_adds = Transaction.select_season(Current.latest().season).where( - (Transaction.newteam == self) & (Transaction.week == current.week + 1) & (Transaction.cancelled == 0) + (Transaction.newteam == self) & (Transaction.week == current.week + 1) & (Transaction.cancelled == False) ) for move in all_drops: @@ -496,10 +496,10 @@ class Team(BaseModel): sil_team = Team.get_season(f'{self.abbrev}SIL', current.season) all_drops = Transaction.select_season(Current.latest().season).where( - (Transaction.oldteam == sil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == 0) + (Transaction.oldteam == sil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == False) ) all_adds = Transaction.select_season(Current.latest().season).where( - (Transaction.newteam == sil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == 0) + (Transaction.newteam == sil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == False) ) for move in all_drops: @@ -557,10 +557,10 @@ class Team(BaseModel): lil_team = Team.get_season(f'{self.abbrev}LIL', current.season) all_drops = Transaction.select_season(Current.latest().season).where( - (Transaction.oldteam == lil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == 0) + (Transaction.oldteam == lil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == False) ) all_adds = Transaction.select_season(Current.latest().season).where( - (Transaction.newteam == lil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == 0) + (Transaction.newteam == lil_team) & (Transaction.week == current.week + 1) & (Transaction.cancelled == False) ) for move in all_drops: diff --git a/app/routers_v3/transactions.py b/app/routers_v3/transactions.py index 1880dcc..47e0bce 100644 --- a/app/routers_v3/transactions.py +++ b/app/routers_v3/transactions.py @@ -75,14 +75,14 @@ async def get_transactions( transactions = transactions.where(Transaction.player << these_players) if cancelled: - transactions = transactions.where(Transaction.cancelled == 1) + transactions = transactions.where(Transaction.cancelled == True) else: - transactions = transactions.where(Transaction.cancelled == 0) + transactions = transactions.where(Transaction.cancelled == False) if frozen: - transactions = transactions.where(Transaction.frozen == 1) + transactions = transactions.where(Transaction.frozen == True) else: - transactions = transactions.where(Transaction.frozen == 0) + transactions = transactions.where(Transaction.frozen == False) if is_trade is not None: raise HTTPException(