Ticket #1485509 (closed Bugs: fixed)

Opened 17 months ago

Last modified 14 months ago

SQL query execution errors are not handled

Reported by: alec Owned by:
Priority: 5 Milestone: 0.2.1
Component: PHP backend Version: 0.2-beta
Severity: normal Keywords:
Cc: db query error

Description

SQL query execution errors aren't handled. Only prepare errors sets db_error flag and db_error_msg. By example see how rcube_user::update_identity() result is handled. Such function calls $this->db->affected_rows() which (called without arguments) returns result of last _SUCCESSFUL_ query. Then returned value is checked in save_identity.inc (line 57) where if() is always true no matter if query fails or not.

Change History

Changed 14 months ago by alec

  • status changed from new to closed
  • resolution set to fixed

Fixed in r2233.

Note: See TracTickets for help on using tickets.