Closed devmach closed 12 years ago
The inline docs are incorrect, the connection property may it may not be a PDO instance.
It would be more accurate to remove the PDO (and pgsql) specifics since this class is database agnostic.
(...but I would leave the usage example at L201->L205)
Build Scheduled
Why not just add a prepare_statement()
method to Database_PDO
?
That is, I would prefer PDO
functionality be added to Database_PDO
.
I'm sorry, may bad . If you think too giving access to a connection handler/link is a logical way, i'll fix inline docs and make another pull request.
useless ps: this patch was intended use in-house and other than PDO is kind of no-go, that's why it's PDO ( and postgresql centric )
@cbandy hmm, prepare_statement() method could be better
I've implemented prepare_statement()
and last_insert_id()
methods to Database_PDO.
Build Scheduled
Kohana's DB module hasn't got any real support for prepared statements. If we're using PDO driver, while we're enjoying Kohana goodies, we can also have "real" prepared statements etc.
This patch basicly gives user access to connection object.