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

fix: invalidate cache after PlayerService write operations (#32)

Browse Source 
Add finally blocks to update_player, patch_player, create_players, and
delete_player in PlayerService to call invalidate_related_cache() using
the existing cache_patterns. Matches the pattern already used in
TeamService.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Cal Corum 2026-03-05 13:03:53 -06:00
parent 9558da6ace
commit d92bb263f1
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/services/player_service.py
View File

@ -526,6 +526,8 @@ class PlayerService(BaseService):
raise HTTPException( raise HTTPException(
status_code=500, detail=f"Error updating player {player_id}: {str(e)}" status_code=500, detail=f"Error updating player {player_id}: {str(e)}"
) )
finally:
temp_service.invalidate_related_cache(cls.cache_patterns)
@classmethod @classmethod
def patch_player( def patch_player(
@ -553,6 +555,8 @@ class PlayerService(BaseService):
raise HTTPException( raise HTTPException(
status_code=500, detail=f"Error patching player {player_id}: {str(e)}" status_code=500, detail=f"Error patching player {player_id}: {str(e)}"
) )
finally:
temp_service.invalidate_related_cache(cls.cache_patterns)
@classmethod @classmethod
def create_players( def create_players(
@ -585,6 +589,8 @@ class PlayerService(BaseService):
raise HTTPException( raise HTTPException(
status_code=500, detail=f"Error creating players: {str(e)}" status_code=500, detail=f"Error creating players: {str(e)}"
) )
finally:
temp_service.invalidate_related_cache(cls.cache_patterns)
@classmethod @classmethod
def delete_player(cls, player_id: int, token: str) -> Dict[str, str]: def delete_player(cls, player_id: int, token: str) -> Dict[str, str]:
@ -608,6 +614,8 @@ class PlayerService(BaseService):
raise HTTPException( raise HTTPException(
status_code=500, detail=f"Error deleting player {player_id}: {str(e)}" status_code=500, detail=f"Error deleting player {player_id}: {str(e)}"
) )
finally:
temp_service.invalidate_related_cache(cls.cache_patterns)
@classmethod @classmethod
def _format_player_csv(cls, players: List[Dict]) -> str: def _format_player_csv(cls, players: List[Dict]) -> str: