Changeset 13584 for trunk/wp-signup.php

Timestamp:

03/04/2010 06:38:39 PM (15 years ago)

Author:

westi

Message:

Fix some notices in wp-signup.php

File:

• trunk/wp-signup.php (modified) (4 diffs)

trunk/wp-signup.php

@@ -9 +9 @@
 require_once( ABSPATH . WPINC . '/registration.php' );
 
-if ( is_array( get_site_option( 'illegal_names' )) && $_GET[ 'new' ] != '' && in_array( $_GET[ 'new' ], get_site_option( 'illegal_names' ) ) == true ) {
+if ( is_array( get_site_option( 'illegal_names' )) && isset( $_GET[ 'new' ] ) && in_array( $_GET[ 'new' ], get_site_option( 'illegal_names' ) ) == true ) {
     wp_redirect( "http://{$current_site->domain}{$current_site->path}" );
     die();
@@ -234 +234 @@
     else
         $signup[ 'blog' ] = 'checked="checked"';
+        
+    //TODO - This doesn't seem to do anything do we really need it?
+    $signup['user'] = isset( $signup['user'] ) ? $signup['user'] : '';
 
     // allow definition of default variables
@@ -394 +397 @@
     echo sprintf( __( "You must first <a href=\"%s\">login</a>, and then you can create a new site."), $login_url );
 } else {
-    switch ($_POST['stage']) {
+    $stage = isset( $_POST['stage'] ) ?  $_POST['stage'] : 'default';
+    switch ( $stage ) {
         case 'validate-user-signup' :
             if ( $active_signup == 'all' || $_POST[ 'signup_for' ] == 'blog' && $active_signup == 'blog' || $_POST[ 'signup_for' ] == 'user' && $active_signup == 'user' )
@@ -410 +414 @@
             validate_another_blog_signup();
             break;
+        case 'default':
         default :
-            $user_email = $_POST[ 'user_email' ];
+            $user_email = isset( $_POST[ 'user_email' ] ) ? $_POST[ 'user_email' ] : '';
             do_action( "preprocess_signup_form" ); // populate the form from invites, elsewhere?
             if ( is_user_logged_in() && ( $active_signup == 'all' || $active_signup == 'blog' ) )