cal/major-domo-database
1
0
Fork 0
Code Issues 25 Pull Requests 25 Actions Packages Projects Releases Wiki Activity

fix: replace integer comparisons on boolean fields with True/False (#69) #94

Open
Claude wants to merge 1 commits from issue/69-boolean-fields-compared-as-integers-sqlite-pattern into main
pull from: issue/69-boolean-fields-compared-as-integers-sqlite-pattern
merge into: cal:main
Conversation 0 Commits 1 Files Changed 2 +10 -10
2 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
app/db_engine.py
View File

@ -435,10 +435,10 @@ class Team(BaseModel):
active_roster[pos] += 1 active_roster[pos] += 1
all_drops = Transaction.select_season(Current.latest().season).where( 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( 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: for move in all_drops:
@ -496,10 +496,10 @@ class Team(BaseModel):
sil_team = Team.get_season(f'{self.abbrev}SIL', current.season) sil_team = Team.get_season(f'{self.abbrev}SIL', current.season)
all_drops = Transaction.select_season(Current.latest().season).where( 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( 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: for move in all_drops:
@ -557,10 +557,10 @@ class Team(BaseModel):
lil_team = Team.get_season(f'{self.abbrev}LIL', current.season) lil_team = Team.get_season(f'{self.abbrev}LIL', current.season)
all_drops = Transaction.select_season(Current.latest().season).where( 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( 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: for move in all_drops:

8
app/routers_v3/transactions.py
View File

@ -75,14 +75,14 @@ async def get_transactions(
transactions = transactions.where(Transaction.player << these_players) transactions = transactions.where(Transaction.player << these_players)
if cancelled: if cancelled:
transactions = transactions.where(Transaction.cancelled == 1) transactions = transactions.where(Transaction.cancelled == True)
else: else:
transactions = transactions.where(Transaction.cancelled == 0) transactions = transactions.where(Transaction.cancelled == False)
if frozen: if frozen:
transactions = transactions.where(Transaction.frozen == 1) transactions = transactions.where(Transaction.frozen == True)
else: else:
transactions = transactions.where(Transaction.frozen == 0) transactions = transactions.where(Transaction.frozen == False)
if is_trade is not None: if is_trade is not None:
raise HTTPException( raise HTTPException(