Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:97256
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain beccati.com designates 176.9.123.236 as permitted sender)
To: Thomas Nunninger <thomas@nunninger.info>,
 Benoit Schildknecht <bensor987@neuf.fr>, internals@lists.php.net
References: <CADxrgCOPNDSKhptKqa-CYF9mOscJHHN+fx19pWyQiqJVdGigCQ@mail.gmail.com>
 <op.yrtb5yadhuszwh@tsukapiou.home>
 <758680d9-cfe2-4a93-3831-7aa03b4bc9db@nunninger.info>
 <ae327680-59a2-bf5f-bbc9-3afcfa6a6248@beccati.com>
 <3dbd220f-4d69-cac5-e45b-1f949f6943f9@nunninger.info>
Message-ID: <0f3432db-537f-2711-12e9-33e43857af98@beccati.com>
Date: Fri, 2 Dec 2016 15:51:20 +0100
MIME-Version: 1.0
In-Reply-To: <3dbd220f-4d69-cac5-e45b-1f949f6943f9@nunninger.info>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Re: [RFC] Debugging PDO Prepared Statement Emulation v2
From: php@beccati.com (Matteo Beccati)

Hi,

On 02/12/2016 15:20, Thomas Nunninger wrote:
>> Could you please explain what's the use case? As far as I'm concerned
>> such information would only be useful during PDO driver development,
>> phpt files and bug reporting / fixing.
> 
> Sometimes I'd like to log what SQL is really used that I can
> analyze/modify a query without the need to replace all bound parameters
> manually before applying it to the command line of my database.
> 
> Additionally, it's a matter of good style in my opinion. E.g. if you
> return an array you can extend the structure later. If some test checks
> a string output you have to change this. If it checks the values of some
> array keys, the test is stable as long as you just add new keys.

Your planned usage only applies if you use emulated prepared statements,
so it's not really something one should depend on.

That said, alas the only debug facility in PDO prints directly to
output. Version 1 of this RFC added something completely different and
v2 fixes it re-using what PDO already has.

If you ask me, what PDO offers sucks, but that's what we've got and
improving it is out of scope for this RFC. Anyone could create a new RFC
to add array output to debugDumpParams, that would be most welcome.


Cheers
-- 
Matteo Beccati

Development & Consulting - http://www.beccati.com/