Database: Avoid internal use of ignoreErrors()
authorKevin Israel <pleasestand@live.com>
Tue, 27 Dec 2016 10:20:30 +0000 (05:20 -0500)
committerKevin Israel <pleasestand@live.com>
Tue, 27 Dec 2016 10:20:30 +0000 (05:20 -0500)
commitcc495efff47ab351093f408ad18a46b551b9f9cd
tree57a018eff2093009de733ac747be1d15dd0347fe
parent2c4a6cc7e0a8dc393633d098e0863397da325f3a
Database: Avoid internal use of ignoreErrors()

* Use $tempIgnore in DatabaseBase::tableExists() instead of ignoreErrors().
* DatabasePostgres::reportQueryError(), since 419221e4a71b, calls
  ignoreErrors() before and after rollback(). This is probably not effective
  as a recursion guard (reportQueryError() would be called regardless), and
  for suppressing any log entry or exception, it is redundant (doRollback()
  already sets $tempIgnore when calling query()).
* ignoreErrors(), now unused, can be removed in a separate change set.

Change-Id: Iee768abf996ece213146ab8f64a15a2b407a2e5e
includes/libs/rdbms/database/Database.php
includes/libs/rdbms/database/DatabasePostgres.php