Changeset 13392 for trunk/src/bp-xprofile/classes/class-bp-xprofile-field-type-selectbox.php

Timestamp:

01/06/2023 11:35:55 PM (4 years ago)

Author:

espellcaste

Message:

PHPCS: adding a little bit of space between brackets.

Closes ​https://github.com/buddypress/buddypress/pull/44
See #7228

File:

• trunk/src/bp-xprofile/classes/class-bp-xprofile-field-type-selectbox.php (modified) (5 diffs)

trunk/src/bp-xprofile/classes/class-bp-xprofile-field-type-selectbox.php

@@ -107 +107 @@
                 $html    = '<option value="">' . /* translators: no option picked in select box */ esc_html__( '----', 'buddypress' ) . '</option>';
 
-                if ( empty( $original_option_values ) && !empty( $_POST['field_' . $this->field_obj->id] ) ) {
-                        $original_option_values = sanitize_text_field(  $_POST['field_' . $this->field_obj->id] );
+                if ( empty( $original_option_values ) && !empty( $_POST[ 'field_' . $this->field_obj->id ] ) ) {
+                        $original_option_values = sanitize_text_field(  $_POST[ 'field_' . $this->field_obj->id ] );
                 }
 
@@ -118 +118 @@
                         // being saved first time.
                         foreach( $option_values as $i => $option_value ) {
-                                if ( isset( $_POST['field_' . $this->field_obj->id] ) && $_POST['field_' . $this->field_obj->id] != $option_value ) {
-                                        if ( ! empty( $_POST['field_' . $this->field_obj->id] ) ) {
-                                                $option_values[$i] = sanitize_text_field( $_POST['field_' . $this->field_obj->id] );
+                                if ( isset( $_POST[ 'field_' . $this->field_obj->id ] ) && $_POST[ 'field_' . $this->field_obj->id ] != $option_value ) {
+                                        if ( ! empty( $_POST[ 'field_' . $this->field_obj->id ] ) ) {
+                                                $option_values[ $i ] = sanitize_text_field( $_POST[ 'field_' . $this->field_obj->id ] );
                                         }
                                 }
@@ -127 +127 @@
                         // Run the allowed option name through the before_save filter, so
                         // we'll be sure to get a match.
-                        $allowed_options = xprofile_sanitize_data_value_before_save( $options[$k]->name, false, false );
+                        $allowed_options = xprofile_sanitize_data_value_before_save( $options[ $k ]->name, false, false );
 
                         // First, check to see whether the user-entered value matches.
@@ -135 +135 @@
 
                         // Then, if the user has not provided a value, check for defaults.
-                        if ( ! is_array( $original_option_values ) && empty( $option_values ) && $options[$k]->is_default_option ) {
+                        if ( ! is_array( $original_option_values ) && empty( $option_values ) && $options[ $k ]->is_default_option ) {
                                 $selected = ' selected="selected"';
                         }
@@ -150 +150 @@
                          * @param string $k        Current index in the foreach loop.
                          */
-                        $html .= apply_filters( 'bp_get_the_profile_field_options_select', '<option' . $selected . ' value="' . esc_attr( stripslashes( $options[$k]->name ) ) . '">' . esc_html( stripslashes( $options[$k]->name ) ) . '</option>', $options[$k], $this->field_obj->id, $selected, $k );
+                        $html .= apply_filters( 'bp_get_the_profile_field_options_select', '<option' . $selected . ' value="' . esc_attr( stripslashes( $options[ $k ]->name ) ) . '">' . esc_html( stripslashes( $options[ $k ]->name ) ) . '</option>', $options[ $k ], $this->field_obj->id, $selected, $k );
                 }