Re: bug: sybase.php for DB common package

On Tuesday, August 12, 2003 14:24, nicos@php.net wrote: > ----- Original Message ----- > From: "Tomas V.V.Cox" <cox@idecnet.com> > To: <nicos@php.net> > Cc: <pear-dev@lists.php.net> > Sent: Tuesday, August 12, 2003 2:05 PM > Subject: Re: [PEAR-DEV] bug: sybase.php for DB common package >> >> >> 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. > All this for nothing :-p Not really as the PEAR DB implementation was testing a NULL, so it has to be fixed :-) -- Tomas V.V.Cox mailto:cox@idecnet.com