cal/paper-dynasty-database
1
0
Fork 0
Code Issues 7 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

Merge pull request 'fix: consolidate redundant double-query in get_one_play (#14)' (#52) from ai/paper-dynasty-database#14 into next-release

Browse Source 
Reviewed-on: #52
This commit is contained in:
cal 2026-03-05 03:36:25 +00:00
commit 8283425b84
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/routers_v2/stratplays.py
View File

@ -1390,10 +1390,10 @@ async def get_game_summary(
@router.get("/{play_id}") @router.get("/{play_id}")
async def get_one_play(play_id: int): async def get_one_play(play_id: int):
if StratPlay.get_or_none(StratPlay.id == play_id) is None: play = StratPlay.get_or_none(StratPlay.id == play_id)
if play is None:
raise HTTPException(status_code=404, detail=f"Play ID {play_id} not found") raise HTTPException(status_code=404, detail=f"Play ID {play_id} not found")
r_play = model_to_dict(StratPlay.get_by_id(play_id)) return model_to_dict(play)
return r_play
@router.patch("/{play_id}") @router.patch("/{play_id}")