Re: Package Proposal: Text_TSV

On Tuesday, Jul 15, 2003, at 14:59 US/Central, Tomas V.V.Cox wrote:

On Tuesday, July 15, 2003 20:49, Paul M Jones wrote:

Given the discussion yesterday, I have added a parse() method to File_CSV that is very heavily based on Cox's original readQuoted() method, except that parse() handles text (not a file) and I have commented it very liberally.

I have tested File_CSV::parse() against File_CSV::readQuoted() using the same test file, and the results appear to be identical, with the exception that it returns all rows of data (not just one row). No changes to the rest of the class.

Please see the attached candidate File_CSV.php. If it's good enough, can somebody with proper ownership and karma commit it? If not, of course, I look forward to comments and criticism. Thanks.

Hi Paul, I'll take a look a it, thanks! You saw that was a feasible thing :-)

Yes, well, sometimes I need a little convincing. ;-) I'll want to add a ::readAll() style method or something similar later, if that's OK. However, it looks like the File::readAll() method does not close the file once it is opened, so subsequent attempts to read the same .csv file in the same script are ignored (returns no data). Is there something I'm missing there? -- pmj