Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:100079
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain duncanc.co.uk from 209.85.216.170 cause and error)
MIME-Version: 1.0
In-Reply-To: <CAPhkiZxk7KVV6J4KW3N_bzeH3CxVgXnxjaSjthyX=x6ciP48Cw@mail.gmail.com>
References: <CAHE1Edu4gENzeC22wKFVuLSKV=JpE9Lymqa34b5buvTAoQHxug@mail.gmail.com>
 <CAF+90c9rj8pN7in0nQcVAtnEw00t4VUXr0k=CP2471W75t5BZQ@mail.gmail.com>
 <CAHE1EdufukEPkiJFhn49Y_wuayfRsuBrLA9aY4KiJEGPRucasg@mail.gmail.com>
 <CAEKnhAEw9z_LeqhaoXe0tcy045ScdNPqrbu-H43iPagS+VnJ0w@mail.gmail.com> <CAPhkiZxk7KVV6J4KW3N_bzeH3CxVgXnxjaSjthyX=x6ciP48Cw@mail.gmail.com>
Date: Fri, 28 Jul 2017 12:51:07 +0100
Message-ID: <CAHE1EdsbYJfgV1Ag8XJOHg7OChKX757ib=ZRSfe5r0s+cabfJQ@mail.gmail.com>
To: Andrey Andreev <narf@devilix.net>
Cc: Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="94eb2c072b68f68a4e05555f4b25"
Subject: Re: [PHP-DEV] [RFC] Raise warnings for json_encode() and
 json_decode() issues
From: php@duncanc.co.uk (Craig Duncan)

--94eb2c072b68f68a4e05555f4b25
Content-Type: text/plain; charset="UTF-8"

On 28 July 2017 at 12:46, Andrey Andreev <narf@devilix.net> wrote:

>
> This is a long shot, but here's an idea: an OOP API throwing exceptions.
>
> Would require more work both to implement and use, but it would surely
> be less error-prone and there'd be no BC break.
>

Hi Andrey, Yes that would resolve all BC issues, however that wouldn't help
with new developers picking up the `json_*` functions and using them
dangerously.
As pointed out in the original thread, there are plenty of wrappers that
provide this feature, the problem is that core functions have a huge gotcha.

If we introduced a new API then we'd need to deprecate the `json_*`
functions to make users aware, which I imagine would be the most
contentious thing suggested thus far

--94eb2c072b68f68a4e05555f4b25--