Skip to:
Content

BuddyPress.org

Changeset 14124


Ignore:
Timestamp:
04/19/2025 01:12:19 PM (16 months ago)
Author:
espellcaste
Message:

WPCS - miscellaneous fixes for the BP Settings component.

See #9174
See #9173

Location:
trunk
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • trunk/phpcs.xml.dist

    r14026 r14124  
    5252                <exclude name="Generic.WhiteSpace.ScopeIndent.IncorrectExact" />
    5353
     54                <!-- Usually harmless, let's skip them. -->
     55                <exclude name="Squiz.Commenting.InlineComment.InvalidEndChar" />
     56                <exclude name="Squiz.PHP.CommentedOutCode.Found" />
     57
    5458                <exclude name="WordPress.Security.ValidatedSanitizedInput.InputNotValidated" />
    5559                <exclude name="WordPress.Security.ValidatedSanitizedInput.InputNotSanitized" />

  • trunk/src/bp-settings/actions/capabilities.php

    r14077 r14124  
    5454        $is_spammer = ! empty( $_POST['user-spammer'] ) ? true : false;
    5555
    56         if ( bp_is_user_spammer( bp_displayed_user_id() ) != $is_spammer ) {
    57                 $status = ( true == $is_spammer ) ? 'spam' : 'ham';
     56        if ( bp_is_user_spammer( bp_displayed_user_id() ) !== $is_spammer ) {
     57                $status = ( true === $is_spammer ) ? 'spam' : 'ham';
    5858                bp_core_process_spammer_status( bp_displayed_user_id(), $status );
    5959

  • trunk/src/bp-settings/actions/general.php

    r14077 r14124  
    156156                                if ( ! $pass_error->get_error_message() ) {
    157157                                        // Password change attempt is successful.
    158                                         if ( ( ! empty( $_POST['pwd'] ) && wp_unslash( $_POST['pwd'] ) !== $pass ) || is_super_admin() )  {
     158                                        if ( ( ! empty( $_POST['pwd'] ) && wp_unslash( $_POST['pwd'] ) !== $pass ) || is_super_admin() ) {
    159159                                                $update_user['user_pass'] = $_POST['pass1'];
    160160                                                $pass_error               = false;
    … …  
    196196        // Email feedback.
    197197        switch ( $email_error ) {
    198                 case 'invalid' :
    199                         $feedback['email_invalid']  = __( 'That email address is invalid. Check the formatting and try again.', 'buddypress' );
    200                         break;
    201                 case 'blocked' :
    202                         $feedback['email_blocked']  = __( 'That email address is currently unavailable for use.', 'buddypress' );
    203                         break;
    204                 case 'taken' :
    205                         $feedback['email_taken']    = __( 'That email address is already taken.', 'buddypress' );
    206                         break;
    207                 case 'empty' :
    208                         $feedback['email_empty']    = __( 'Email address cannot be empty.', 'buddypress' );
    209                         break;
    210                 case false :
     198                case 'invalid':
     199                        $feedback['email_invalid'] = __( 'That email address is invalid. Check the formatting and try again.', 'buddypress' );
     200                        break;
     201                case 'blocked':
     202                        $feedback['email_blocked'] = __( 'That email address is currently unavailable for use.', 'buddypress' );
     203                        break;
     204                case 'taken':
     205                        $feedback['email_taken'] = __( 'That email address is already taken.', 'buddypress' );
     206                        break;
     207                case 'empty':
     208                        $feedback['email_empty'] = __( 'Email address cannot be empty.', 'buddypress' );
     209                        break;
     210                case false:
    211211                        // No change.
    212212                        break;
    … …  
    218218
    219219        // No errors so show a simple success message.
    220         if ( ( ( false === $email_error ) || ( false == $pass_error ) ) && ( ( true === $pass_changed ) || ( true === $email_changed ) ) ) {
     220        if ( ( ( false === $email_error ) || ( false === $pass_error ) ) && ( ( true === $pass_changed ) || ( true === $email_changed ) ) ) {
    221221                $feedback[]    = __( 'Your settings have been saved.', 'buddypress' );
    222222                $feedback_type = 'success';
    … …  
    297297                }
    298298
    299                 $email_changed = wp_update_user( array(
    300                         'ID'         => $user_id,
    301                         'user_email' => trim( $pending_email['newemail'] ),
    302                 ) );
     299                $email_changed = wp_update_user(
     300                        array(
     301                                'ID'         => $user_id,
     302                                'user_email' => trim( $pending_email['newemail'] ),
     303                        )
     304                );
    303305
    304306                if ( $email_changed ) {

  • trunk/src/bp-settings/actions/notifications.php

    r14077 r14124  
    4040        // Switch feedback for super admins.
    4141        if ( bp_is_my_profile() ) {
    42                 bp_core_add_message( __( 'Your notification settings have been saved.',        'buddypress' ), 'success' );
     42                bp_core_add_message( __( 'Your notification settings have been saved.', 'buddypress' ), 'success' );
    4343        } else {
    4444                bp_core_add_message( __( "This user's notification settings have been saved.", 'buddypress' ), 'success' );

  • trunk/src/bp-settings/bp-settings-filters.php

    r14077 r14124  
    88 */
    99
    10 // Personal data export.
    11 add_filter( 'wp_privacy_personal_data_exporters', 'bp_settings_register_personal_data_exporter' );
     10// Exit if accessed directly.
     11defined( 'ABSPATH' ) || exit;
    1212
    1313/**
    … …  
    2929        return $exporters;
    3030}
     31add_filter( 'wp_privacy_personal_data_exporters', 'bp_settings_register_personal_data_exporter' );

  • trunk/src/bp-settings/bp-settings-functions.php

    r14077 r14124  
    8989 */
    9090function bp_settings_get_registered_notification_keys() {
    91 
    9291        ob_start();
     92
    9393        /**
    9494         * Fires at the start of the building of the notification keys allowed list.
    … …  
    9797         */
    9898        do_action( 'bp_notification_settings' );
     99
    99100        $screen = ob_get_clean();
    100101
    … …  
    114115 *
    115116 * @since 4.0.0
    116  *
    117  * @param string $email_address  The user's email address.
    118  * @param int    $page           Batch number.
     117 * @since 15.0.0 The `$page` parameter was removed since it was unused.
     118 *
     119 * @param string $email_address The user's email address.
    119120 * @return array An array of personal data.
    120121 */
    121 function bp_settings_personal_data_exporter( $email_address, $page ) {
     122function bp_settings_personal_data_exporter( $email_address ) {
    122123        $email_address = trim( $email_address );
    123124
    … …  
    239240        }
    240241
    241         $query = new WP_Query( array(
    242                 'author'        => (int) $user_id,
    243                 'post_type'     => 'user_request',
    244                 'post_status'   => 'any',
    245                 'post_name__in' => array(
    246                         'export_personal_data',
    247                 ),
    248         ) );
     242        $query = new WP_Query(
     243                array(
     244                        'author'        => (int) $user_id,
     245                        'post_type'     => 'user_request',
     246                        'post_status'   => 'any',
     247                        'post_name__in' => array(
     248                                'export_personal_data',
     249                        ),
     250                )
     251        );
    249252
    250253        if ( ! empty( $query->post ) ) {
    251254                return wp_get_user_request( $query->post->ID );
    252         } else {
    253                 return false;
    254         }
     255        }
     256
     257        return false;
    255258}
    256259
    … …  
    328331        /** This filter is documented in /wp-admin/includes/ajax-actions.php */
    329332        $exporters             = apply_filters( 'wp_privacy_personal_data_exporters', array() );
    330         $custom_friendly_names = apply_filters( 'bp_settings_data_custom_friendly_names', array(
    331                 'wordpress-comments' => _x( 'Comments', 'WP Comments data exporter friendly name', 'buddypress' ),
    332                 'wordpress-media'    => _x( 'Media', 'WP Media data exporter friendly name', 'buddypress' ),
    333                 'wordpress-user'     => _x( 'Personal information', 'WP Media data exporter friendly name', 'buddypress' ),
    334         ) );
     333        $custom_friendly_names = apply_filters(
     334                'bp_settings_data_custom_friendly_names',
     335                array(
     336                        'wordpress-comments' => _x( 'Comments', 'WP Comments data exporter friendly name', 'buddypress' ),
     337                        'wordpress-media'    => _x( 'Media', 'WP Media data exporter friendly name', 'buddypress' ),
     338                        'wordpress-user'     => _x( 'Personal information', 'WP Media data exporter friendly name', 'buddypress' ),
     339                )
     340        );
    335341
    336342?>
    337343        <ul>
    338         <?php foreach ( $exporters as $exporter => $data ) :
     344        <?php
     345        foreach ( $exporters as $exporter => $data ) :
    339346                // Use the exporter friendly name by default.
    340347                $friendly_name = $data['exporter_friendly_name'];

  • trunk/src/bp-settings/bp-settings-template.php

    r14077 r14124  
    7979        }
    8080
    81         if ( bp_get_displayed_user_email() == $pending_email['newemail'] ) {
     81        if ( bp_get_displayed_user_email() === $pending_email['newemail'] ) {
    8282                return;
    8383        }

  • trunk/src/bp-settings/classes/class-bp-settings-component.php

    r14077 r14124  
    4343         */
    4444        public function includes( $includes = array() ) {
    45                 parent::includes( array(
    46                         'template',
    47                         'filters',
    48                         'functions',
    49                 ) );
     45                parent::includes(
     46                        array(
     47                                'template',
     48                                'filters',
     49                                'functions',
     50                        )
     51                );
    5052        }
    5153
    … …  
    113115
    114116                // All globals for settings component.
    115                 parent::setup_globals( array(
    116                         'slug'          => $default_slug,
    117                         'has_directory' => false,
    118                 ) );
     117                parent::setup_globals(
     118                        array(
     119                                'slug'          => $default_slug,
     120                                'has_directory' => false,
     121                        )
     122                );
    119123        }
    120124
    … …  
    132136         */
    133137        public function register_nav( $main_nav = array(), $sub_nav = array() ) {
    134                 $slug   = bp_get_settings_slug();
     138                $slug = bp_get_settings_slug();
    135139
    136140                // Add the settings navigation item.
Note: See TracChangeset for help on using the changeset viewer.