Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:97384
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain pthreads.org from 74.125.82.45 cause and error)
MIME-Version: 1.0
In-Reply-To: <a9677cdd-8c4c-26a4-50ed-6ba7f5dbf570@beccati.com>
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> <op.yru714ajhuszwh@tsukapiou.home>
 <CADxrgCMQCovAvNRw8hrr0fK9uUTm+jW8Ai_=pGjzt8OMVn6wzQ@mail.gmail.com>
 <CANaX5ndzt+58D87GTqfHy+gNNf9qhaJpdUhp1Q1iB660yqM9ww@mail.gmail.com>
 <CADxrgCORO-Tar9cSKUO=6j+0yuYW6VWigXMt+1zXtD9riHQdWw@mail.gmail.com>
 <933676c1-e828-9eb5-405e-13242fee39a7@lsces.co.uk> <CAFp73Xsba1aHTvZJStr+_w507zoijTqp3Q5chKum1jskSYzX+w@mail.gmail.com>
 <a9677cdd-8c4c-26a4-50ed-6ba7f5dbf570@beccati.com>
Date: Tue, 13 Dec 2016 11:26:40 +0000
Message-ID: <CAL=_i_mkoW6WZEZYFM9BSYegwb==OdD3p0FZ4h1Wmr7=v4XgYA@mail.gmail.com>
To: Matteo Beccati <php@beccati.com>
Cc: "guilhermeblanco@gmail.com" <guilhermeblanco@gmail.com>, Lester Caine <lester@lsces.co.uk>, 
	PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a114435668a4aa30543887e27
Subject: Re: [PHP-DEV] Re: [RFC] Debugging PDO Prepared Statement Emulation v2
From: pthreads@pthreads.org (Joe Watkins)

--001a114435668a4aa30543887e27
Content-Type: text/plain; charset=UTF-8

Morning,

Can you please update the RFC with the dates that voting started and will
end, so that anyone browsing knows when the deadline for decision is
without trawling through internals.

Cheers
Joe

On Fri, Dec 9, 2016 at 9:36 AM, Matteo Beccati <php@beccati.com> wrote:

> Hi Guilherme,
>
> it's not really a new method. Just some more output to the existing
> debugDumpParams method, which is indeed available after execute() when
> emulated prepares is supported by the driver and currently active. The
> method output already depends on its location: before/after binding
> parameters.
>
> To me, this is much better than v1 of the RFC that was adding a new
> debug method specifically for this.
>
> That said, I think we should prepare yet another RFC to have the debug
> information optionally returned as array.
>
>
> Cheers
>
> On 09/12/2016 05:27, guilhermeblanco@gmail.com wrote:
> > Hi Adam,
> >
> > Providing my input on why I voted "NO".
> > Your proposed new method is not pure. This means it does have distinct
> > responses depending on where it's located in the code.
> > That's a big no-no for me.
> >
> > Cheers,
> >
> > On Tue, Dec 6, 2016 at 12:15 PM, Lester Caine <lester@lsces.co.uk>
> wrote:
> >
> >> On 06/12/16 16:30, Adam Baratz wrote:
> >>>> This issue is not difficult to solve in userland, at least well enough
> >> for
> >>>> debugging in my experience. Here's an older example I wrote up on my
> >> blog
> >>>> that worked (but I'm told is broken right now):
> >> https://daveyshafik.com/
> >>>> archives/605-debugging-pdo-prepared-statements.html
> >>>>
> >>> I want to be able to write .phpt tests that validate how values are
> >> getting
> >>> interpolated into emulated prepares. Since this creates a separate code
> >>> path, it wouldn't let me validate what gets sent to the server.
> >>> PDO::quote() doesn't necessarily produce the same output as
> >>> pdo_parse_params().
> >>
> >> The whole point of testing is that it tests real results not testing
> >> 'test code' that tries to emulate the real world. The whole point of
> >> 'emulated prepare statements' is that it creates a SQL query that the
> >> target database engine can actually work with rather than one that a
> >> more capable database engine will process. Testing the innards of some
> >> driver is simply wrong, one tests the whole process is working! And
> >> IDEALLY the test suit for PDO should work which ever target is selected,
> >> but that is not lightly to happen any time soon.
> >>
> >> --
> >> Lester Caine - G8HFL
> >> -----------------------------
> >> Contact - http://lsces.co.uk/wiki/?page=contact
> >> L.S.Caine Electronic Services - http://lsces.co.uk
> >> EnquirySolve - http://enquirysolve.com/
> >> Model Engineers Digital Workshop - http://medw.co.uk
> >> Rainbow Digital Media - http://rainbowdigitalmedia.co.uk
> >>
> >> --
> >> PHP Internals - PHP Runtime Development Mailing List
> >> To unsubscribe, visit: http://www.php.net/unsub.php
> >>
> >>
> >
> >
>
>
> --
> Matteo Beccati
>
> Development & Consulting - http://www.beccati.com/
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--001a114435668a4aa30543887e27--