Context Navigation

Changeset 6444

Timestamp:

12/21/2007 01:35:26 AM (18 years ago)

Author:

ryan

Message:

Set error if there is a problem connecting to the DB. Props skeltoac and Potter_System. fixes #5495 for 2.3

Location:

branches/2.3

Files:

-                      r6140
+                      r6444
 else
     $step = 0;
+function display_header(){
 header( 'Content-Type: text/html; charset=utf-8' );
 ?>
 …
 <body>
 <h1 id="logo"><img alt="WordPress" src="images/wordpress-logo.png" /></h1>
 <?php
+}//end function display_header();
 // Let's check to make sure WP isn't already installed.
 if ( is_blog_installed() ) die('<h1>'.__('Already Installed').'</h1><p>'.__('You appear to have already installed WordPress. To reinstall please clear your old database tables first.').'</p></body></html>');
+if ( is_blog_installed() ) {display_header(); die('<h1>'.__('Already Installed').'</h1><p>'.__('You appear to have already installed WordPress. To reinstall please clear your old database tables first.').'</p></body></html>');}
 switch($step) {
     case 0:
     case 1: // in case people are directly linking to this
+      display_header();
 ?>
 <h1><?php _e('Welcome'); ?></h1>
 …
         break;
     case 2:
+      if ( !empty($wpdb->error) )
+        wp_die($wpdb->error->get_error_message());
+    display_header();
         // Fill in the data we gathered
         $weblog_title = stripslashes($_POST['weblog_title']);

-                      r6137
+                      r6444
 require_once('../wp-includes/compat.php');
 require_once('../wp-includes/functions.php');
+require_once('../wp-includes/classes.php');
 if (!file_exists('../wp-config-sample.php'))
 …
     // We'll fail here if the values are no good.
     require_once('../wp-includes/wp-db.php');
+    if ( !empty($wpdb->error) )
+        wp_die($wpdb->error->get_error_message());
     $handle = fopen('../wp-config.php', 'w');

-                      r6443
+                      r6444
 <ul>
 <li>Are you sure it exists?</li>
+<li>Does the user <code>".DB_USER."</code> have permission to use the <code>$db</code> database?</li>
 <li>On some systems the name of your database is prefixed with your username, so it would be like username_wordpress. Could that be the problem?</li>
 </ul>
 …
      */
     function bail($message) { // Just wraps errors in a nice header and footer
+        if ( !$this->show_errors )
+        if ( !$this->show_errors ) {
+            if ( class_exists('WP_Error') )
+                $this->error = new WP_Error('500', $message);
+            else
+                $this->error = $message;
             return false;
+        }
         wp_die($message);
+    }

Note: See TracChangeset for help on using the changeset viewer.