Changeset 8691

Timestamp:

07/24/2014 08:49:06 AM (5 years ago)

Author:

djpaul

Message:

xProfile: when a user's profile is updated, correctly pass the fields' previous values to the xprofile_updated_profile action.

Fixes #5755 (branch), props Viper007Bond

File:

• branches/2.0/bp-xprofile/bp-xprofile-screens.php (modified) (2 diffs)

branches/2.0/bp-xprofile/bp-xprofile-screens.php

@@ -109 +109 @@
                 $value = isset( $_POST['field_' . $field_id] ) ? $_POST['field_' . $field_id] : '';
 
-                if ( !xprofile_set_field_data( $field_id, bp_displayed_user_id(), $value, $is_required[$field_id] ) ) {
-                    $errors = true;
-                } else {
-                    do_action( 'xprofile_profile_field_data_updated', $field_id, $value );
-                }
-
-                // Save the visibility level
                 $visibility_level = !empty( $_POST['field_' . $field_id . '_visibility'] ) ? $_POST['field_' . $field_id . '_visibility'] : 'public';
-                xprofile_set_field_visibility_level( $field_id, bp_displayed_user_id(), $visibility_level );
 
                 // Save the old and new values. They will be
@@ -130 +122 @@
                     'visibility' => $visibility_level,
                 );
+
+                if ( !xprofile_set_field_data( $field_id, bp_displayed_user_id(), $value, $is_required[$field_id] ) ) {
+                    $errors = true;
+                } else {
+                    do_action( 'xprofile_profile_field_data_updated', $field_id, $value );
+                }
+
+                // Save the visibility level
+                xprofile_set_field_visibility_level( $field_id, bp_displayed_user_id(), $visibility_level );
             }