trans_status() misdocumented

[eluser]chriskl[/eluser]
The user guide says:

Code:

if ($this->db->trans_status() === FALSE)
{
    // generate an error... or use the log_message() function to log your error
}

However, the code for trans_status() does this:

Code:

return $this->_trans_failure;

But _trans_failure is set like this:

Code:

// Run the Query
if (FALSE === ($this->result_id = $this->simple_query($sql)))
{
    // This will trigger a rollback if transactions are being used
    $this->_trans_failure = TRUE;
}

Hence, trans_status() returns TRUE on failure, not false...

[eluser]Derek Allard[/eluser]
Could you file a bug report for this please?