Ticket #10883 (new enhancement)
db-error.php not used for all DB failures
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | Future Release |
| Component: | Database | Version: | 2.8.4 |
| Severity: | normal | Keywords: | needs-patch 2nd-opinion |
| Cc: | carstenbach |
Description
db-error.php (the optional custom DB error message file to be placed in wp-content) does not get included all the time. Sometimes wp-db.php will use its bail() method to spit out its own message. This code needs to be there too:
if ( file_exists( WP_CONTENT_DIR . '/db-error.php' ) ) {
require_once( WP_CONTENT_DIR . '/db-error.php' );
die();
}
Change History
- Owner changed from westi to ryan
- Component changed from Cron to Database
- Keywords 2nd-opinion added
- Type changed from defect (bug) to enhancement
- Milestone changed from 3.0 to 3.1
The purpose of the db-error.php was for prettying up the front-end database connection error screen, or giving some other form of feedback. It has never affected the back-end.
I'm not sure using it for the backend failure cases is a good idea, it'll mean that anyone currently using the file will have to modify it in order to see the DB errors.
Shifting to 3.1 pending patch & feedback from others..
Note: See
TracTickets for help on using
tickets.
