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

Timestamp:

01/06/2023 11:35:55 PM (3 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 );
         }