Re: cvs: pear /DB/DB common.php
| From: | MARTY Fabien | Date: | Tue, 11 Jun 2002 12:18:24 +0000 |
| Subject: | Re: cvs: pear /DB/DB common.php | ||
| References: | 1 2 3 | Groups: | php.pear.cvs |
| Request: | Send a blank email to pear-cvs+get-4548@lists.php.net to get a copy of this message | ||
> How sounds:
>
> - buildManipSQL($table, $table_fields, $mode, $where = false)
> - autoPrepare($table, $table_fields, $mode, $where = false)
> - autoExecute($table, $fields_values, $mode, $where = false)
>
> $table_fields -> ordered array
> $fields_values -> assoc array
It sounds fine to me ! :-)
> Why you pass an assoc to the actual autoPrepare()? I guess that normally
> one has the table names in one side and the values in another.
I use an assoc to perform the autoPrepare() then execute() by using a single
array. It's true that autoPrepare doesn't need values of the assoc.
But if I write autoExecute(), I don't need at all values in autoPrepare().
So, it's perfect. I will commit this.
And if it is ok for all, I will write the xml documentation for autoPrepare()
and autoExecute() and maybe a little auto*() tuturial.
Just one question for buildManipSQL : private or public method ?
Thanks,
Fabien