Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:78990
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.171 as permitted sender)
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Message-ID: <82139FDD-8D8B-43D9-B811-ECC1FFE6E8A6@gmail.com>
Date: Wed, 19 Nov 2014 20:20:01 +0800
To: PHP Internals <internals@lists.php.net>
Mime-Version: 1.0 (Mac OS X Mail 8.0 \(1990.1\))
Subject: Re: fgetcsv incompatible with fputcsv
From: tjerk.meesters@gmail.com (Tjerk Meesters)

Hi list,

As I was fiddling with CSV data reading and writing I noticed that =
fgetcsv() is inherently incompatible with fputcsv() when it comes to the =
enclosure escape character that=E2=80=99s used.

Example: http://3v4l.org/LHEZj

The above example code demonstrates how, by default, fputcsv() encodes a =
backslash as is but fgetcsv() will treat that same backslash as the =
enclosure escape character as well as the enclosure character itself; =
this is rather surprising behaviour and imho unnecessarily complicated.

I would suggest changing the behaviour in such a way that:
a) the default enclosure escape character for fgetcsv() is a double =
quote.
b) fgetcsv() only treats the escape character as =E2=80=A6 an escape =
character.

Due to the kind of change BC can=E2=80=99t be maintained, so I=E2=80=99d =
propose this change for PHP 7.

If anyone has violent objections to the above, or thinks that an RFC =
should be drawn up, do let me know =E2=80=A6 otherwise I=E2=80=99ll =
commit the change into master by next week or so.


Best,
 Tjerk=