cal/claude-memory
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

store: Fix: get_one_player CSV missing data row — to_csv() called before append()

Browse Source
This commit is contained in:
Cal Corum 2026-03-03 20:32:17 -06:00
parent 7072459995
commit 262449abe3
1 changed files with 25 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
graph/fixes/fix-get-one-player-csv-missing-data-row-to-csv-called-before-1fca0a.md Normal file
View File

@ -0,0 +1,25 @@
---
id: 1fca0ae0-681e-4755-b16c-e5661639c6ef
type: fix
title: "Fix: get_one_player CSV missing data row — to_csv() called before append()"
tags: [paper-dynasty, python, fastapi, pandas, csv, bug, fix]
importance: 0.6
confidence: 0.8
created: "2026-03-04T02:32:17.214021+00:00"
updated: "2026-03-04T02:32:17.214021+00:00"
---
## Problem
`get_one_player` in `app/routers_v2/players.py` returned a CSV response containing only the header row — player data was never included.
## Root Cause
`return_val = DataFrame(data_list).to_csv(header=False, index=False)` was called on line 625 when `data_list` contained only the header list. The player data row was appended to `data_list` on line 626 — after the CSV was already computed.
## Solution
Moved the `to_csv()` call to after `data_list.append(player_row)`. Single-line swap, no logic changes.
## Files Changed
- `app/routers_v2/players.py` (~line 625-651)
## PR
https://git.manticorum.com/cal/paper-dynasty-database/pulls/51 (issue #12)