Re: cvs: pear /DB/DB dbase.php fbsql.php ibase.php ifx.php msql.php mssql.php mysql.php mysql4.php oci8.php odbc.php pgsql.php sqlite.php sybase.php
| From: | Daniel Convissor | Date: | Sat, 03 Jan 2004 03:09:09 +0000 |
| Subject: | Re: cvs: pear /DB/DB dbase.php fbsql.php ibase.php ifx.php msql.php mssql.php mysql.php mysql4.php oci8.php odbc.php pgsql.php sqlite.php sybase.php | ||
| References: | 1 2 | Groups: | php.pear.cvs php.pear.qa |
| Request: | Send a blank email to pear-qa+get-326@lists.php.net to get a copy of this message | ||
Hi Lukas, et al:
On Sat, Jan 03, 2004 at 01:58:20AM +0100, Lukas Smith wrote:
>
> - 'portability' will remain as it is now (lowercasing across all drivers),
> maybe evening adding the rtrim call (I think in MDB the ibase driver does
> that too)
> - add an option to get the old behaviour
> - add a smartquote()
> - remove magic stuff from quote()
> - add an option to keep the old magic in quote()
Adding all of these options would create a whole lot of extra quirks for
everyone to learn and remember, plus several if statements that have to be
executed over and over and over.
The point of quote() is to make a variable safe for insertion into a
query. It's not about putting "'" around everything. If it were, there'd
be no reason for the method, since it'd be easier for programmers to just
do that themselves.
Thanks,
--Dan
--
T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y
data intensive web and database programming
http://www.AnalysisAndSolutions.com/
4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409