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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alex_b’s picture

alex_b CreditAttribution: alex_b commented
Status: Active » Needs review
FileSize
850652-1_column_names.patch2.1 KB

Running tests ATM. Straightforward patch, will commit ASAP.

alex_b’s picture

alex_b CreditAttribution: 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.