RE: MDB patch
| From: | Lukas Smith | Date: | Mon, 17 Jun 2002 15:13:55 +0000 |
| Subject: | RE: MDB patch | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-7133@lists.php.net to get a copy of this message | ||
Thx,
those are the types of errors my current tests dont really catch and
that have been introduced by switching from Metabases error handling to
pear error handling ...
anyways thx for the patch
Best regards,
Lukas Smith
smith@dybnet.de
_______________________________
DybNet Internet Solutions GbR
Reuchlinstr. 10-11
Gebäude 4 1.OG Raum 6 (4.1.6)
10553 Berlin
Germany
Tel. : +49 30 83 22 50 00
Fax : +49 30 83 22 50 07
www.dybnet.de info@dybnet.de
> -----Original Message-----
> From: Paul Cooper [mailto:pgc@ucecom.com]
> Sent: Monday, June 17, 2002 5:06 PM
> To: pear-dev@lists.php.net
> Cc: smith@dybnet.de
> Subject: MDB patch
>
> in the fetch* functions the input parameter $result (a result
> identifier) is tested with
>
> if (!$result) { ... }
>
> However $db->query() will return an mdb_error object when the query
> fails, so the test should be MDB::isError($result). See attached diff,
>
> Paul
>
> --
> -----------------------------------------------------------------
> Paul Cooper | Tel: 0121 331 7858
> Senior Programmer and Database Engineer | Fax: 0121 331 7859
> UCEcom |
> mailto:pgc@ucecom.com
> University of Central England |
> http://www.ucecom.com
> Birmingham, B4 7DX |
> -----------------------------------------------------------------