Re: cvs: php-src(PHP_5_1) /ext/pgsql pgsql.c
| From: | Lukas Kahwe Smith | Date: | Mon, 27 Mar 2006 07:42:18 +0000 |
| Subject: | Re: cvs: php-src(PHP_5_1) /ext/pgsql pgsql.c | ||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-37520@lists.php.net to get a copy of this message | ||
Yasuo Ohgaki wrote:
yohgaki Mon Mar 27 02:59:55 2006 UTC
Modified files: (Branch: PHP_5_1)
/php-src/ext/pgsql pgsql.c Log:
remove pg_execute() E_WARNING error when query plan is not defined
Sorry but I do not understand the logic behind this change. Nor do I think this should go into 5.1.3. I think it makes perfect sense to get and E_WARNING for this. Of course there are other ways to check if an error did ocurr. Maybe I am misunderstanding the scope of the change, but from your comments on internals it just seems you need to beef up your customer error handler to ignore supressed errors.
regards,
Lukas