Opened 5 years ago

Closed 4 years ago

#1485509 closed Bugs (fixed)

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 (1)

comment:1 Changed 4 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [9c5bee69].

Note: See TracTickets for help on using tickets.