Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112958
To: Kamil Tekiela <tekiela246@gmail.com>
References: <CAGBsUrddzJQu3JtqCdLjyHOBf71gj5LpNKuV=OmYGm0Vd3n9Yw@mail.gmail.com>
Cc: PHP internals <internals@lists.php.net>
Message-ID: <0edafce4-c9a5-c483-65f1-72e49614135a@telia.com>
Date: Thu, 21 Jan 2021 22:51:24 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.6.1
MIME-Version: 1.0
In-Reply-To: <CAGBsUrddzJQu3JtqCdLjyHOBf71gj5LpNKuV=OmYGm0Vd3n9Yw@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-GB
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] [RFC]: Change Default mysqli Error Mode
From: bjorn.x.larsson@telia.com (=?UTF-8?Q?Bj=c3=b6rn_Larsson?=)

Den 2021-01-21 kl. 00:04, skrev Kamil Tekiela:
> Hi Internals,
> 
> Following my earlier announcement in https://wiki.php.net/rfc/improve_mysqli
> I decided to submit a dedicated RFC for the first proposal.
> 
> This RFC proposes to change the default mysqli error reporting mode to
> exception mode. This change follows the same change in PDO. The RFC can be
> found here https://wiki.php.net/rfc/mysqli_default_errmode
> 
> Kind Regards,
> Kamil Tekiela
> 

If I understand this correctly, does it imply that a lot of
legacy code that works today, will not work any longer and
needs to change? Like:
if ("mysqlierror") {
	"error handling"
}

If so I think that like in other RFC's an investigation should
be done on how much code that is affected to assess BC impact.

Btw, I liked your proposal on how to improve prepared statements
in mysqli a lot!

Regards //Björn Larsson