Make WordPress Core


Ignore:
Timestamp:
07/25/2019 12:47:53 AM (6 years ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Rename $r variable used with wp_parse_args() to $parsed_args for clarity.

Props freewebmentor.
Fixes #45059.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/user.php

    r45639 r45667  
    11241124    $defaults['selected'] = is_author() ? get_query_var( 'author' ) : 0;
    11251125
    1126     $r = wp_parse_args( $args, $defaults );
    1127 
    1128     $query_args = wp_array_slice_assoc( $r, array( 'blog_id', 'include', 'exclude', 'orderby', 'order', 'who', 'role', 'role__in', 'role__not_in' ) );
     1126    $parsed_args = wp_parse_args( $args, $defaults );
     1127
     1128    $query_args = wp_array_slice_assoc( $parsed_args, array( 'blog_id', 'include', 'exclude', 'orderby', 'order', 'who', 'role', 'role__in', 'role__not_in' ) );
    11291129
    11301130    $fields = array( 'ID', 'user_login' );
    11311131
    1132     $show = ! empty( $r['show'] ) ? $r['show'] : 'display_name';
     1132    $show = ! empty( $parsed_args['show'] ) ? $parsed_args['show'] : 'display_name';
    11331133    if ( 'display_name_with_login' === $show ) {
    11341134        $fields[] = 'display_name';
     
    11391139    $query_args['fields'] = $fields;
    11401140
    1141     $show_option_all   = $r['show_option_all'];
    1142     $show_option_none  = $r['show_option_none'];
    1143     $option_none_value = $r['option_none_value'];
     1141    $show_option_all   = $parsed_args['show_option_all'];
     1142    $show_option_none  = $parsed_args['show_option_none'];
     1143    $option_none_value = $parsed_args['option_none_value'];
    11441144
    11451145    /**
     
    11491149     *
    11501150     * @param array $query_args The query arguments for get_users().
    1151      * @param array $r          The arguments passed to wp_dropdown_users() combined with the defaults.
    1152      */
    1153     $query_args = apply_filters( 'wp_dropdown_users_args', $query_args, $r );
     1151     * @param array $parsed_args          The arguments passed to wp_dropdown_users() combined with the defaults.
     1152     */
     1153    $query_args = apply_filters( 'wp_dropdown_users_args', $query_args, $parsed_args );
    11541154
    11551155    $users = get_users( $query_args );
    11561156
    11571157    $output = '';
    1158     if ( ! empty( $users ) && ( empty( $r['hide_if_only_one_author'] ) || count( $users ) > 1 ) ) {
    1159         $name = esc_attr( $r['name'] );
    1160         if ( $r['multi'] && ! $r['id'] ) {
     1158    if ( ! empty( $users ) && ( empty( $parsed_args['hide_if_only_one_author'] ) || count( $users ) > 1 ) ) {
     1159        $name = esc_attr( $parsed_args['name'] );
     1160        if ( $parsed_args['multi'] && ! $parsed_args['id'] ) {
    11611161            $id = '';
    11621162        } else {
    1163             $id = $r['id'] ? " id='" . esc_attr( $r['id'] ) . "'" : " id='$name'";
    1164         }
    1165         $output = "<select name='{$name}'{$id} class='" . $r['class'] . "'>\n";
     1163            $id = $parsed_args['id'] ? " id='" . esc_attr( $parsed_args['id'] ) . "'" : " id='$name'";
     1164        }
     1165        $output = "<select name='{$name}'{$id} class='" . $parsed_args['class'] . "'>\n";
    11661166
    11671167        if ( $show_option_all ) {
     
    11701170
    11711171        if ( $show_option_none ) {
    1172             $_selected = selected( $option_none_value, $r['selected'], false );
     1172            $_selected = selected( $option_none_value, $parsed_args['selected'], false );
    11731173            $output   .= "\t<option value='" . esc_attr( $option_none_value ) . "'$_selected>$show_option_none</option>\n";
    11741174        }
    11751175
    1176         if ( $r['include_selected'] && ( $r['selected'] > 0 ) ) {
    1177             $found_selected = false;
    1178             $r['selected']  = (int) $r['selected'];
     1176        if ( $parsed_args['include_selected'] && ( $parsed_args['selected'] > 0 ) ) {
     1177            $found_selected          = false;
     1178            $parsed_args['selected'] = (int) $parsed_args['selected'];
    11791179            foreach ( (array) $users as $user ) {
    11801180                $user->ID = (int) $user->ID;
    1181                 if ( $user->ID === $r['selected'] ) {
     1181                if ( $user->ID === $parsed_args['selected'] ) {
    11821182                    $found_selected = true;
    11831183                }
     
    11851185
    11861186            if ( ! $found_selected ) {
    1187                 $users[] = get_userdata( $r['selected'] );
     1187                $users[] = get_userdata( $parsed_args['selected'] );
    11881188            }
    11891189        }
     
    11991199            }
    12001200
    1201             $_selected = selected( $user->ID, $r['selected'], false );
     1201            $_selected = selected( $user->ID, $parsed_args['selected'], false );
    12021202            $output   .= "\t<option value='$user->ID'$_selected>" . esc_html( $display ) . "</option>\n";
    12031203        }
     
    12151215    $html = apply_filters( 'wp_dropdown_users', $output );
    12161216
    1217     if ( $r['echo'] ) {
     1217    if ( $parsed_args['echo'] ) {
    12181218        echo $html;
    12191219    }
Note: See TracChangeset for help on using the changeset viewer.