Re: [PATCH] DB::oci8::fetchRow() should use fetchInto()

From: 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

« previous php.pear.dev (#17265) next »