diff options
| author | Niklas Nilsson <splitfeed@gmail.com> | 2011-08-30 13:39:42 +0200 |
|---|---|---|
| committer | Niklas Nilsson <splitfeed@gmail.com> | 2011-08-30 13:39:42 +0200 |
| commit | d2018ee8ae967cafbd8315ba9ea45f58150bcffe (patch) | |
| tree | 82f78b4ba1cd58f71169e8bf702bf1926808c5bf /system | |
| parent | b50df5f018176c0cd0ad498e9c710a2b0b016a80 (diff) | |
Fixed issue #105 SQL log errors
Enabled logging database query errors even if $db_debug is not enabled.
Diffstat (limited to 'system')
| -rw-r--r-- | system/database/DB_driver.php | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/system/database/DB_driver.php b/system/database/DB_driver.php index f3e824daa..300ca2977 100644 --- a/system/database/DB_driver.php +++ b/system/database/DB_driver.php @@ -251,9 +251,10 @@ class CI_DB_driver { { if ($sql == '') { + log_message('error', 'Invalid query: '.$sql); + if ($this->db_debug) { - log_message('error', 'Invalid query: '.$sql); return $this->display_error('db_invalid_query'); } return FALSE; @@ -306,21 +307,23 @@ class CI_DB_driver { // This will trigger a rollback if transactions are being used $this->_trans_status = FALSE; + // Grab the error number and message now, as we might run some + // additional queries before displaying the error + $error_no = $this->_error_number(); + $error_msg = $this->_error_message(); + + // Log errors + log_message('error', 'Query error: '.$error_msg); + if ($this->db_debug) { - // grab the error number and message now, as we might run some - // additional queries before displaying the error - $error_no = $this->_error_number(); - $error_msg = $this->_error_message(); - // We call this function in order to roll-back queries // if transactions are enabled. If we don't call this here // the error message will trigger an exit, causing the // transactions to remain in limbo. $this->trans_complete(); - // Log and display errors - log_message('error', 'Query error: '.$error_msg); + // Display errors return $this->display_error( array( 'Error Number: '.$error_no, |