Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:113153
Content-Type: text/plain;
	charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.60.0.2.21\))
In-Reply-To: <CAJvSBVPUT7fUC8OEAwzQdrByiTbV0cSnqUKNxkgjrynQ8JeZ8g@mail.gmail.com>
Date: Fri, 12 Feb 2021 14:30:52 +0000
Cc: Christian Schneider <cschneid@cschneid.com>,
 PHP internals <internals@lists.php.net>
Content-Transfer-Encoding: quoted-printable
Message-ID: <BDE6D497-965D-477E-9D6F-16F9C9E45F52@craigfrancis.co.uk>
References: <CAGBsUreDc1CgpBawb6LNXMzpjK_9Y3LQW9n4jYB9AaDjmD9n-g@mail.gmail.com>
 <5D0AC588-7BBC-419D-BFEA-06E2E112D3F8@cschneid.com>
 <CAJvSBVPUT7fUC8OEAwzQdrByiTbV0cSnqUKNxkgjrynQ8JeZ8g@mail.gmail.com>
To: Peter Bowyer <phpmailinglists@gmail.com>
Subject: Re: [PHP-DEV] [VOTE]: Change Default mysqli Error Mode
From: craig@craigfrancis.co.uk (Craig Francis)

On 12 Feb 2021, at 08:45, Peter Bowyer <phpmailinglists@gmail.com> =
wrote:
> On Thu, 11 Feb 2021 at 13:31, Christian Schneider =
<cschneid@cschneid.com> wrote:
>=20
>> [...] If there would have been an intermediate PHP version with =
default
>> MYSQLI_REPORT_ERROR I would have voted "Yes", but in the current form =
I
>> have to say "No".
>>=20
>=20
> I voted "No" for the same reasons.


If you want an intermediate PHP version, can it be done via Nikita's =
solution.

As in, for 8.1, issue a deprecation warning if mysqli_report() hasn't =
been called before opening a connection, then change the default to =
exceptions in 8.2.

This would get everyone (who upgrades to every 0.x) to notice that they =
haven't specified how they want reporting to work, and effectively gives =
them a year to notice and specify that choice.

Switching the default from `OFF` to `ERROR` to `ERROR | STRICT` isn't an =
obvious step, as that still means you're still having a single step to =
change from error reporting to exceptions.

Thanks,
Craig=