major-domo-database/app
Cal Corum 39bc99b0dd
Some checks failed
Build Docker Image / build (pull_request) Failing after 15s
fix: Respect short_output parameter in player search endpoint
The search_players() method was hardcoding short_output=True when
converting query results to dicts, ignoring the function parameter.
This caused the /api/v3/players/search endpoint to always return
team_id as an integer instead of nested team objects, even when
short_output=False was specified.

Impact:
- Discord bot's /injury clear command was crashing because
  player.team was None (only team_id was populated)
- Any code using search endpoint couldn't get full team data

Fix:
- Changed line 386 to use the short_output parameter value
- Now respects short_output parameter: False returns full team
  objects, True returns just team IDs

Root cause analysis from production logs:
- Error: AttributeError: 'NoneType' object has no attribute 'roster_type'
- Location: commands/injuries/management.py:647
- Cause: player.team was None after search_players() call

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2026-02-05 19:43:50 -06:00
..
routers_v3 fix: Prevent locals() from polluting player PATCH data dict 2026-02-04 14:57:32 -06:00
services fix: Respect short_output parameter in player search endpoint 2026-02-05 19:43:50 -06:00
__init__.py Initial commit 2023-03-21 16:09:46 -05:00
db_engine.py Add salary_cap column to Team model (v2.2.0) 2025-12-10 07:28:16 -06:00
dependencies.py Add CACHE_ENABLED env var to toggle Redis caching (v2.2.1) 2025-12-10 07:59:54 -06:00
main.py Added HelpCommands 2025-10-17 16:36:40 -05:00