Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:113649
Return-Path: <php@beccati.com>
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 67681 invoked from network); 21 Mar 2021 16:45:38 -0000
Received: from unknown (HELO php-smtp4.php.net) (45.112.84.5)
  by pb1.pair.com with SMTP; 21 Mar 2021 16:45:38 -0000
Received: from php-smtp4.php.net (localhost [127.0.0.1])
	by php-smtp4.php.net (Postfix) with ESMTP id B6130180505
	for <internals@lists.php.net>; Sun, 21 Mar 2021 09:40:40 -0700 (PDT)
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on php-smtp4.php.net
X-Spam-Level: 
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS
	autolearn=no autolearn_force=no version=3.4.2
X-Spam-Virus: No
X-Envelope-From: <php@beccati.com>
Received: from box.beccati.com (box.beccati.com [176.9.123.236])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(No client certificate requested)
	by php-smtp4.php.net (Postfix) with ESMTPS
	for <internals@lists.php.net>; Sun, 21 Mar 2021 09:40:39 -0700 (PDT)
Received: from authenticated-user (box.beccati.com [176.9.123.236])
	(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))
	(No client certificate requested)
	by box.beccati.com (Postfix) with ESMTPSA id 2AF762000AE;
	Sun, 21 Mar 2021 17:40:38 +0100 (CET)
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=beccati.com; s=mail;
	t=1616344838; bh=FZ5n+MqX+A5B8FqbdFnz9z77wVPY2hlUECuL0Dg+zAg=;
	h=Date:Subject:To:References:From:Cc:In-Reply-To:From;
	b=jP+ZAsxG1tVqSRHky/qpdZ//nzfTeLdCVkBFkXhGrQ1MJ25zfOFYKcYsc6ydwxngU
	 aT04zQ3w3pa1Uf0n84gKHKU1qFqPuXOiYAbJYv4oF3+XvXNQrAGintkbsOd4mgtlwM
	 PDGkAVMtQn3bX1qx727e1Dq5tSOZfnjPkYt12ZVcIM19EfORiU+gIOu0cjvy+7oY5m
	 dXgw8bexXIey48y0PH7xHRuWWlTulyoX5FTaIF47yiRC3jPpeKkioMBUfeU/wd5W1F
	 BzfGKQjdXxL+YnLy6DvIPJfPkBSl5RLak74R+voBhaZl76YfFBj2J30NL8shHl0wLf
	 3Bb2NavM3rThw==
Message-ID: <f00d149f-b34a-2ba8-d7b8-8619a46be8f9@beccati.com>
Date: Sun, 21 Mar 2021 17:40:36 +0100
MIME-Version: 1.0
Content-Language: en-GB
To: Benjamin Morel <benjamin.morel@gmail.com>,
 "Christoph M. Becker" <cmbecker69@gmx.de>
References: <CAG9XoMS5QySfvO5eZv=vTiGefcynkOL3mGwq8K5Knq1OQRqWSQ@mail.gmail.com>
 <163c35c6-90ba-c343-b64d-d17f2e2b7766@beccati.com>
 <f96b6e6f-0993-4711-2b72-c657ca1b34ac@gmx.de>
 <CAG9XoMSu_6U=U4k-NzzqtKodsHembaPYHvZBuUMC2C_Q84sWfA@mail.gmail.com>
Cc: PHP Internals <internals@lists.php.net>
In-Reply-To: <CAG9XoMSu_6U=U4k-NzzqtKodsHembaPYHvZBuUMC2C_Q84sWfA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] PDO::PARAM_INT and pgsql driver
From: php@beccati.com (Matteo Beccati)

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/