TODO BaseConnection needs better error handling

[jim-parry]

Database/BaseCOnnection::query(...) has a todo note...
deal with errors (they are currently ignored)

[jim-parry]

@lonnieezell Cleaning up todo tags :-/

[lonnieezell]

I saw that! Though in this case you were a bit pre-mature. You just returned instead of handling the error. I need to look back at the original driver, but I believe the intent was to capture any errors that happened during a transaction and display them to the user (depending on CI_DEBUG setting?) after the transaction has been closed.