Re: PDO::PARAM_INT and pgsql driver

From: Date: Sun, 21 Mar 2021 16:40:36 +0000
Subject: Re: PDO::PARAM_INT and pgsql driver
References: 1 2 3 4  Groups: php.internals 
Request: Send a blank email to internals+get-113649@lists.php.net to get a copy of this message 
Hi Benjamin,

thanks, in fact I was planning to ask you to open a bug report in order to reference it in the test files and commit message.

I have the fix ready and it doesn't seem to have side effects on the existing tests. I'd expect to push it in the coming week, thanks for the reminder ;-)


Cheers

On 21/03/2021 15:13, Benjamin Morel wrote:

Is there any reason why the pgsql driver doesn't respect

    PDO::PARAM_STR?


Looks like a bug to me. I thought this had been fixed a while

    back, but

apparently it wasn't. I'll try to find some time to investigate

    in the

next few days.


    Looks like the problem is in
    <https://github.com/php/php-src/blob/php-7.4.16/ext/pdo_pgsql/pgsql_statement.c#L377-L391
    <https://github.com/php/php-src/blob/php-7.4.16/ext/pdo_pgsql/pgsql_statement.c#L377-L391>>



I've opened a bug report to keep track of it: https://bugs.php.net/bug.php?id=80892 <https://bugs.php.net/bug.php?id=80892>

Thank you,
Benjamin



-- 
Matteo Beccati

Development & Consulting - http://www.beccati.com/

Thread (6 messages)

« previous php.internals (#113649) next »