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

Timestamp:

04/19/2010 02:31:29 AM (16 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 );
                         }
                 }