Re: [PATCH] DB::oci8::fetchRow() should use fetchInto()
| From: | Tomas V.V.Cox | Date: | Tue, 10 Jun 2003 16:42:58 +0000 |
| Subject: | Re: [PATCH] DB::oci8::fetchRow() should use fetchInto() | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-17265@lists.php.net to get a copy of this message | ||
As I said in my previous email, oci::fetchRow() should just be dropped. The
DB_result::fetchRow() calls internally fetchInto(), so this func gets never
called.
Tomas V.V.Cox
----- Original Message -----
From: "Roman Neuhauser" <neuhauser@bellavista.cz>
To: "pear-dev" <pear-dev@lists.php.net>
Cc: "Tomas V.V.Cox" <cox@idecnet.com>
Sent: Tuesday, June 10, 2003 1:57 PM
Subject: [PEAR-DEV] [PATCH] DB::oci8::fetchRow() should use fetchInto()
> see attached patch
>
> Tomas, since you're listed as a maintainer, could you please review,
> test, and eventually commit this patch? It so happens that it fixes a
> minor issue: fetchInto() lowercases the column names if told so by the
> $options array, but fetchRow() never does.
>
> The line numbers might be slightly off (I have other changes in this
> driver not submitted yet), but should apply cleanly.
>
> Thanks!
>
> --
> If you cc me or remove the list(s) completely I'll most likely ignore
> your message. see
> http://www.eyrie.org./~eagle/faqs/questions.html
>
----------------------------------------------------------------------------
----
> --
> PEAR Development Mailing List (http://pear.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php