Re: bug: sybase.php for DB common package
| From: | Tomas V.V.Cox | Date: | Tue, 12 Aug 2003 12:05:55 +0000 |
| Subject: | Re: bug: sybase.php for DB common package | ||
| References: | 1 2 3 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-19556@lists.php.net to get a copy of this message | ||
On Tuesday, August 12, 2003 1:38, nicos@php.net wrote:
> "Tomas V.V.Cox" <cox@idecnet.com> a écrit dans le message de
> news:191052873.20030811120001@idecnet.com...
>>
>>
>> On Monday, August 11, 2003 6:06, Spider wrote:
>>
>> > I've found out sybase.php don't provide a function for
>> > DB_Result::numRows(). I have to add the following
>> > script to sybase.php to make DB_Result::numRows() to
>> > work.
>>
>> > function numRows($result)
>> > {
>> > $rows = @sybase_num_rows($result);
>> > if ($rows === null) {
>> > return $this->raiseError();
>> > }
>> > return $rows;
>> > }
>>
>> I'll add it, thanks. Just one question, does sybase_num_rows() return
>> "null" on errors? Are you sure on this?
>>
> According to the source, it returns FALSE.
Thanks nicos. Anyways, seeing the sources, the sybase numrows()
functionality is already implemented, so just upgrade your PEAR DB
Spider.
--
Tomas V.V.Cox mailto:cox@idecnet.com