Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103162
To: internals@lists.php.net
References: <7cf5adb8-0738-259e-6d1e-f966722fdae2@gmx.de>
 <CA+kxMuQRHq8HfuicqSFm8_Xcxw7VD6Aaqoi5TZa7m7Tofh1YKA@mail.gmail.com>
 <c2b8b875-f98e-1745-2ba8-ef7289a9c15f@gmx.de>
 <CY4PR18MB10489E50B61CC480A83B5853DE060@CY4PR18MB1048.namprd18.prod.outlook.com>
 <5ca1b295-6f43-794c-f2a9-f672155cc84c@gmx.de>
 <CY4PR18MB1048546563A670F0D2567C21DE1A0@CY4PR18MB1048.namprd18.prod.outlook.com>
Message-ID: <988920f5-2fea-5f3f-f521-ac8693e70020@gmail.com>
Date: Thu, 13 Sep 2018 20:24:35 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101
 Thunderbird/60.0
MIME-Version: 1.0
In-Reply-To: <CY4PR18MB1048546563A670F0D2567C21DE1A0@CY4PR18MB1048.namprd18.prod.outlook.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-GB
Subject: Re: [PHP-DEV] fputcsv() and $escape character
From: rowan.collins@gmail.com (Rowan Collins)

On 13/09/2018 16:55, Theodore Brown wrote:
> If the patch was also applied to PHP 7.2 and 7.1, we wouldn't need to add
> fallback code in the League/Csv library for these versions. I understand
> if this isn't possible, but I'm trying to figure out how best to approach
> fixing the issue in League/Csv.


Even if the fix was added to those branches, most people using those 
versions of PHP would not upgrade to it immediately, or at all, so the 
library would still need to work with and without the feature in some form.

Regards,

-- 
Rowan Collins
[IMSoP]