Changeset 2932 for branches/1.2/bp-core/bp-core-signup.php

Timestamp:

04/19/2010 02:31:29 AM (15 years ago)

Author:

johnjamesjacoby

Message:

Fixes #2290

File:

• branches/1.2/bp-core/bp-core-signup.php (modified) (1 diff)

branches/1.2/bp-core/bp-core-signup.php

@@ -54 +54 @@
         $bp->signup->email = $_POST['signup_email'];
 
-        if ( !empty( $_POST['signup_profile_field_ids'] ) && function_exists( 'xprofile_check_is_required_field' ) ) {
-            /* Now we've checked account details, we can check profile information */
-            $profile_field_ids = explode( ',', $_POST['signup_profile_field_ids'] );
-
-            /* Loop through the posted fields formatting any datebox values then validate the field */
-            foreach ( (array) $profile_field_ids as $field_id ) {
-                if ( !isset( $_POST['field_' . $field_id] ) ) {
-                    if ( isset( $_POST['field_' . $field_id . '_day'] ) )
-                        $_POST['field_' . $field_id] = strtotime( $_POST['field_' . $field_id . '_day'] . $_POST['field_' . $field_id . '_month'] . $_POST['field_' . $field_id . '_year'] );
+        /* Now we've checked account details, we can check profile information */
+        if ( function_exists( 'xprofile_check_is_required_field' ) ) {
+
+            /* Make sure hidden field is passed and populated */
+            if ( isset( $_POST['signup_profile_field_ids'] ) && !empty( $_POST['signup_profile_field_ids'] ) ) {
+
+                /* Let's compact any profile field info into an array */
+                $profile_field_ids = explode( ',', $_POST['signup_profile_field_ids'] );
+
+                /* Loop through the posted fields formatting any datebox values then validate the field */
+                foreach ( (array) $profile_field_ids as $field_id ) {
+                    if ( !isset( $_POST['field_' . $field_id] ) ) {
+                        if ( isset( $_POST['field_' . $field_id . '_day'] ) )
+                            $_POST['field_' . $field_id] = strtotime( $_POST['field_' . $field_id . '_day'] . $_POST['field_' . $field_id . '_month'] . $_POST['field_' . $field_id . '_year'] );
+                    }
+
+                    /* Create errors for required fields without values */
+                    if ( xprofile_check_is_required_field( $field_id ) && empty( $_POST['field_' . $field_id] ) )
+                        $bp->signup->errors['field_' . $field_id] = __( 'This is a required field', 'buddypress' );
                 }
 
-                if ( xprofile_check_is_required_field( $field_id ) && empty( $_POST['field_' . $field_id] ) )
-                    $bp->signup->errors['field_' . $field_id] = __( 'This is a required field', 'buddypress' );
+            /* This situation doesn't naturally occur so bounce to website root */
+            } else {
+                bp_core_redirect( $bp->root_domain );
             }
         }