Currently there is a loop in FeedsCSVParser replacing integer column keys with the names of the first row. The ParserCSV library supports exactly that, we should use it.

Patch coming.

CommentFileSizeAuthor
#1 850652-1_column_names.patch2.1 KBalex_b

Comments

alex_b’s picture

alex_b commented
Status: Active » Needs review
StatusFileSize
new 850652-1_column_names.patch2.1 KB

Running tests ATM. Straightforward patch, will commit ASAP.

alex_b’s picture

alex_b commented
Status: Needs review » Fixed

All tests passing. Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.