Description: ------------ I've played around with MS Excel (yeah, I know, not the most reliable when it comes to standards) and it seems like commas and double quotes are handled thus: aaa,bbb,"cc,c",ddd,"ee""e" To support this, the output should be altered thus: if (strstr($text, '"')) { $text = '"'.str_replace('"', '""', $text).'"'; } elseif (strstr($text, ',')) { $text = '"'.$text.'"'; } Reproduce code: --------------- Ouput a CSV with a comma or double quote in the output Expected result: ---------------- Commas should stay in their field and double quotes should show up correctky Actual result: -------------- Commas in fields create an extra field when opening / processing the CSV file

I'll add that feature when I've time ... it will be something that will support the same as File_CSV ... will be it's own package, I might bundle it with SDG to start with I think, it's ready now I think so it might happen soon.

This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. I will implement Helgi's File_CSV implementation once it's ready. I added Justin's patch for now.

Actually it won't have anything to do with File_CSV :-) It's a package that doesn't have the file logic. At the moment Firman is giving my package as spin with the AddressBook package, haven't tho heard back from him in a while :-) Will probably do this after the Amsterdam conference in May.