6.0.0-beta1
7/18/25

[#9397] CSV import is not working
Summary CSV import is not working
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Resolved
Priority 2. Medium
Owners jan (at) horde (dot) org
Requester goncalo.queiros (at) portugalmail (dot) net
Created 11/19/2010 (5355 days ago)
Due
Updated 07/02/2011 (5130 days ago)
Assigned 11/24/2010 (5350 days ago)
Resolved 07/02/2011 (5130 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
07/02/2011 04:41:18 PM Jan Schneider State ⇒ Resolved
 
07/02/2011 04:41:13 PM Git Commit Comment #5 Reply to this comment
Changes have been made in Git for this ticket:

Use detected number of fields from Horde_Data and hardcode default 
separator and quote characters (Bug #9397).

  1 files changed, 3 insertions(+), 5 deletions(-)
http://git.horde.org/horde-git/-/commit/23edc79dafc270a2cde3dfe157680d668567782c
07/02/2011 04:41:09 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git for this ticket:

[jan] Try to detect number of fields again when importing CSV data 
(Bug #9397).

  2 files changed, 12 insertions(+), 2 deletions(-)
http://git.horde.org/horde-git/-/commit/4bd4073bc3c2a324995747748f8a80bc44d738ce
11/24/2010 04:56:55 PM Michael Slusarz Comment #3
Assigned to Jan Schneider
Priority ⇒ 2. Medium
State ⇒ Assigned
Reply to this comment
Eh, ignore the commit... figured out where the length determination 
actually disappeared:

commit fa182cd88e4cbaf3bdd09888913dabeed2c515ee
Author: Jan Schneider <jan@horde.org>
Date:   Tue Oct 26 23:09:47 2010 +0200

     Drop Horde_File_Csv. fgetcsv() is sufficiently capable these days 
that a whole library is no longer justified. Add a light wrapper to 
Horde_Util instead.

Specifically: we no longer call Horde_File_Csv::discoverFormat() so we 
never automatically determine the length - without the length, 
importFile() fails.
11/19/2010 05:51:20 PM goncalo (dot) queiros (at) portugalmail (dot) net Comment #1
Priority ⇒ 1. Low
State ⇒ Unconfirmed
Patch ⇒ No
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ CSV import is not working
Type ⇒ Bug
Reply to this comment
Importing CSV data is not working, because we no longer know how many 
fields the file header contains, and are expecting this to have a 
value on Horde_Data_Csv::importFile

Saved Queries