Ticket #10883 (new enhancement)

Opened 3 years ago

Last modified 3 days ago

db-error.php not used for all DB failures

Reported by: markjaquith Owned by: ryan
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

  • Keywords needs-patch added
  • Milestone changed from Unassigned to 3.0
  • Owner changed from westi to ryan
  • Component changed from Cron to Database

comment:3   dd322 years ago

  • 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..

  • Milestone changed from Awaiting Triage to Future Release
  • Cc carstenbach added
Note: See TracTickets for help on using tickets.