WordPress.org
Get WordPress

Make WordPress Core

Ticket #15729: 15729-8.diff

File 15729-8.diff, 7.8 KB (added by kapeels, 14 years ago)

This has got to be commitable now :D

  • wp-admin/setup-config.php

     
    8888<link rel="stylesheet" href="css/install.css" type="text/css" />
    8989
    9090</head>
    91 <body>
     91<body onload="setFocus();">
    9292<h1 id="logo"><img alt="WordPress" src="images/wordpress-logo.png" /></h1>
    9393<?php
    9494}//end function display_header();
    … …  
    114114        break;
    115115
    116116        case 1:
    117                 display_header();
     117            display_header();
     118            if ( isset( $_POST['setup_error'] ) ) {
     119                switch( $_POST['setup_error'] ) {
     120                    case 'db_connect_fail' :
     121                        $error_msg = "Cannot connect to the database server with the provided username,password and host combination.";
     122                        $focus_element = "uname";
     123                    break;
     124                    case 'db_select_fail' :
     125                        $error_msg = "Cannot select the database.";
     126                        $focus_element = "dbname";
     127                    break;
     128                    case 'invalid_prefix' :
     129                        $error_msg = "The table prefix can contain only letters, numbers, and underscores.";
     130                        $focus_element = "prefix";
     131                    break;
     132                }               
     133                ?>
     134                <script type="text/javascript">
     135                    function setFocus() {
     136                        document.getElementById( '<?php echo $focus_element; ?>' ).focus();
     137                    }
     138                </script>
     139                <p style="color:red;"><?php echo $error_msg; ?></p>
     140                <?php               
     141            }
     142            $dbname =   !empty( $_POST['dbname'] ) ? trim( $_POST['dbname'] ) : 'wordpress';
     143            $uname  =   !empty( $_POST['uname'] ) ? trim( $_POST['uname'] ) : 'username';
     144            // password can be left blank
     145            $password=   isset( $_POST['pwd'] ) ? $_POST['pwd'] : 'password';
     146            $dbhost =   !empty( $_POST['dbhost'] ) ? trim( $_POST['dbhost'] ) : 'localhost';
     147            $prefix =   !empty( $_POST['prefix'] ) ? trim( $_POST['prefix'] ) : 'wp_';           
    118148        ?>
    119149<form method="post" action="setup-config.php?step=2">
    120150        <p>Below you should enter your database connection details. If you're not sure about these, contact your host. </p>
    121151        <table class="form-table">
    122152                <tr>
    123153                        <th scope="row"><label for="dbname">Database Name</label></th>
    124                         <td><input name="dbname" id="dbname" type="text" size="25" value="wordpress" /></td>
     154                        <td><input name="dbname" id="dbname" type="text" size="25" value="<?php echo htmlspecialchars( $dbname, ENT_QUOTES ); ?>" /></td>
    125155                        <td>The name of the database you want to run WP in. </td>
    126156                </tr>
    127157                <tr>
    128158                        <th scope="row"><label for="uname">User Name</label></th>
    129                         <td><input name="uname" id="uname" type="text" size="25" value="username" /></td>
     159                        <td><input name="uname" id="uname" type="text" size="25" value="<?php echo htmlspecialchars( $uname, ENT_QUOTES ); ?>" /></td>
    130160                        <td>Your MySQL username</td>
    131161                </tr>
    132162                <tr>
    133163                        <th scope="row"><label for="pwd">Password</label></th>
    134                         <td><input name="pwd" id="pwd" type="text" size="25" value="password" /></td>
     164                        <td><input name="pwd" id="pwd" type="text" size="25" value="<?php echo htmlspecialchars( $password, ENT_QUOTES ); ?>" /></td>
    135165                        <td>...and MySQL password.</td>
    136166                </tr>
    137167                <tr>
    138168                        <th scope="row"><label for="dbhost">Database Host</label></th>
    139                         <td><input name="dbhost" id="dbhost" type="text" size="25" value="localhost" /></td>
     169                        <td><input name="dbhost" id="dbhost" type="text" size="25" value="<?php echo htmlspecialchars( $dbhost, ENT_QUOTES ); ?>" /></td>
    140170                        <td>You should be able to get this info from your web host, if <code>localhost</code> does not work.</td>
    141171                </tr>
    142172                <tr>
    143173                        <th scope="row"><label for="prefix">Table Prefix</label></th>
    144                         <td><input name="prefix" id="prefix" type="text" id="prefix" value="wp_" size="25" /></td>
     174                        <td><input name="prefix" id="prefix" type="text" id="prefix" value="<?php echo htmlspecialchars( $prefix, ENT_QUOTES ); ?>" size="25" /></td>
    145175                        <td>If you want to run multiple WordPress installations in a single database, change this.</td>
    146176                </tr>
    147177        </table>
    … …  
    154184        case 2:
    155185        $dbname  = trim($_POST['dbname']);
    156186        $uname   = trim($_POST['uname']);
    157         $passwrd = trim($_POST['pwd']);
     187        $password = trim($_POST['pwd']);
    158188        $dbhost  = trim($_POST['dbhost']);
    159189        $prefix  = trim($_POST['prefix']);
    160190        if ( empty($prefix) )
    161                 $prefix = 'wp_';
    162 
     191                $prefix = 'wp_';       
     192        $setup_error = null;     
    163193        // Validate $prefix: it can only contain letters, numbers and underscores
    164         if ( preg_match( '|[^a-z0-9_]|i', $prefix ) )
    165                 wp_die( /*WP_I18N_BAD_PREFIX*/'<strong>ERROR</strong>: "Table Prefix" can only contain numbers, letters, and underscores.'/*/WP_I18N_BAD_PREFIX*/ );
     194        if ( preg_match( '|[^a-z0-9_]|i', $prefix ) ) {               
     195                $prefix =   htmlspecialchars( $prefix, ENT_QUOTES );
     196                $setup_error = new WP_Error( 'invalid_prefix', '<strong>ERROR</strong>: "Table Prefix" can only contain numbers, letters, and underscores.' );
     197        }       
     198        // prefix is good, let's continue
     199        else {
     200            // Test the db connection.
     201            /**#@+
     202             * @ignore
     203             */
     204            define('DB_NAME', $dbname);
     205            define('DB_USER', $uname);
     206            define('DB_PASSWORD', $password);
     207            define('DB_HOST', $dbhost);
     208            /**#@-*/
    166209
    167         // Test the db connection.
    168         /**#@+
    169          * @ignore
    170          */
    171         define('DB_NAME', $dbname);
    172         define('DB_USER', $uname);
    173         define('DB_PASSWORD', $passwrd);
    174         define('DB_HOST', $dbhost);
    175         /**#@-*/
    176 
    177         // We'll fail here if the values are no good.
    178         require_wp_db();
    179         if ( ! empty( $wpdb->error ) ) {
    180                 $back = '<p class="step"><a href="setup-config.php?step=1" onclick="javascript:history.go(-1);return false;" class="button">Try Again</a></p>';
    181                 wp_die( $wpdb->error->get_error_message() . $back );
    182         }
    183 
     210            // We'll fail here if the values are no good.
     211            require_wp_db();
     212            if ( ! empty( $wpdb->error ) ) {                                   
     213                $setup_error = $wpdb->error;
     214            }
     215        }
     216       
     217        // check if any error occured above       
     218        if ( is_wp_error( $setup_error ) ) {
     219            $try_again = '<p class="step">
     220            <form action="setup-config.php?step=1" method="post"> 
     221                <input name="setup_error" type="hidden" value="' . $setup_error->get_error_code() . '" />
     222                <input name="dbname" type="hidden" value="' . htmlspecialchars( $dbname, ENT_QUOTES ) . '" />           
     223                <input name="uname" type="hidden" value="' . htmlspecialchars( $uname, ENT_QUOTES ) . '" />             
     224                <input name="pwd" type="hidden" value="' . htmlspecialchars( $password, ENT_QUOTES ) . '" />                   
     225                <input name="dbhost" type="hidden" value="' . htmlspecialchars( $dbhost, ENT_QUOTES ) . '" />           
     226                <input name="prefix" type="hidden" id="prefix" value="' . $prefix . '" />   
     227                <input type="submit" class="button" value="Try Again" name="try-again" />
     228            </form>
     229                  </p>';
     230            wp_die( $setup_error->get_error_message() . $try_again );
     231        }       
    184232        // Fetch or generate keys and salts.
    185233        $no_api = isset( $_POST['noapi'] );
    186234        require_once( ABSPATH . WPINC . '/plugin.php' );
    … …  
    223271                                $configFile[$line_num] = str_replace("'username_here'", "'$uname'", $line);
    224272                                break;
    225273                        case "define('DB_PASSW":
    226                                 $configFile[$line_num] = str_replace("'password_here'", "'$passwrd'", $line);
     274                                $configFile[$line_num] = str_replace("'password_here'", "'$password'", $line);
    227275                                break;
    228276                        case "define('DB_HOST'":
    229277                                $configFile[$line_num] = str_replace("localhost", $dbhost, $line);