importX icon indicating copy to clipboard operation
importX copied to clipboard

Published 20 hours ago • verified publisher icon modmore

Doesn't support CSV

Open stefan-scholl opened this issue 10 years ago • 1 comments

The current code doesn't support CSV. See RFC 4180 and http://en.wikipedia.org/wiki/Comma-separated_values

process() in components/importx/processors/prepare/csv.php reads and treats every line in isolation and splits it with explode(). Please use fgetcsv() instead.

This bug is related to issue #23.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

stefan-scholl avatar Jul 10 '14 11:07 stefan-scholl

Don't suppose you have a patch ready for this? :P

Sounds like a good improvement.

Mark-H avatar Jul 13 '14 10:07 Mark-H