WordPress.org

Make WordPress Core


Ignore:
Timestamp:
07/25/2019 12:47:53 AM (10 months 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/post-template.php

    r45627 r45667  
    7979 */
    8080function the_title_attribute( $args = '' ) {
    81     $defaults = array(
     81    $defaults    = array(
    8282        'before' => '',
    8383        'after'  => '',
     
    8585        'post'   => get_post(),
    8686    );
    87     $r        = wp_parse_args( $args, $defaults );
    88 
    89     $title = get_the_title( $r['post'] );
     87    $parsed_args = wp_parse_args( $args, $defaults );
     88
     89    $title = get_the_title( $parsed_args['post'] );
    9090
    9191    if ( strlen( $title ) == 0 ) {
     
    9393    }
    9494
    95     $title = $r['before'] . $title . $r['after'];
     95    $title = $parsed_args['before'] . $title . $parsed_args['after'];
    9696    $title = esc_attr( strip_tags( $title ) );
    9797
    98     if ( $r['echo'] ) {
     98    if ( $parsed_args['echo'] ) {
    9999        echo $title;
    100100    } else {
     
    947947    );
    948948
    949     $params = wp_parse_args( $args, $defaults );
     949    $parsed_args = wp_parse_args( $args, $defaults );
    950950
    951951    /**
     
    954954     * @since 3.0.0
    955955     *
    956      * @param array $params An array of arguments for page links for paginated posts.
     956     * @param array $parsed_args An array of arguments for page links for paginated posts.
    957957     */
    958     $r = apply_filters( 'wp_link_pages_args', $params );
     958    $parsed_args = apply_filters( 'wp_link_pages_args', $parsed_args );
    959959
    960960    $output = '';
    961961    if ( $multipage ) {
    962         if ( 'number' == $r['next_or_number'] ) {
    963             $output .= $r['before'];
     962        if ( 'number' == $parsed_args['next_or_number'] ) {
     963            $output .= $parsed_args['before'];
    964964            for ( $i = 1; $i <= $numpages; $i++ ) {
    965                 $link = $r['link_before'] . str_replace( '%', $i, $r['pagelink'] ) . $r['link_after'];
     965                $link = $parsed_args['link_before'] . str_replace( '%', $i, $parsed_args['pagelink'] ) . $parsed_args['link_after'];
    966966                if ( $i != $page || ! $more && 1 == $page ) {
    967967                    $link = _wp_link_page( $i ) . $link . '</a>';
    968968                } elseif ( $i === $page ) {
    969                     $link = '<span class="post-page-numbers current" aria-current="' . esc_attr( $r['aria_current'] ) . '">' . $link . '</span>';
     969                    $link = '<span class="post-page-numbers current" aria-current="' . esc_attr( $parsed_args['aria_current'] ) . '">' . $link . '</span>';
    970970                }
    971971                /**
     
    980980
    981981                // Use the custom links separator beginning with the second link.
    982                 $output .= ( 1 === $i ) ? ' ' : $r['separator'];
     982                $output .= ( 1 === $i ) ? ' ' : $parsed_args['separator'];
    983983                $output .= $link;
    984984            }
    985             $output .= $r['after'];
     985            $output .= $parsed_args['after'];
    986986        } elseif ( $more ) {
    987             $output .= $r['before'];
     987            $output .= $parsed_args['before'];
    988988            $prev    = $page - 1;
    989989            if ( $prev > 0 ) {
    990                 $link = _wp_link_page( $prev ) . $r['link_before'] . $r['previouspagelink'] . $r['link_after'] . '</a>';
     990                $link = _wp_link_page( $prev ) . $parsed_args['link_before'] . $parsed_args['previouspagelink'] . $parsed_args['link_after'] . '</a>';
    991991
    992992                /** This filter is documented in wp-includes/post-template.php */
     
    996996            if ( $next <= $numpages ) {
    997997                if ( $prev ) {
    998                     $output .= $r['separator'];
     998                    $output .= $parsed_args['separator'];
    999999                }
    1000                 $link = _wp_link_page( $next ) . $r['link_before'] . $r['nextpagelink'] . $r['link_after'] . '</a>';
     1000                $link = _wp_link_page( $next ) . $parsed_args['link_before'] . $parsed_args['nextpagelink'] . $parsed_args['link_after'] . '</a>';
    10011001
    10021002                /** This filter is documented in wp-includes/post-template.php */
    10031003                $output .= apply_filters( 'wp_link_pages_link', $link, $next );
    10041004            }
    1005             $output .= $r['after'];
     1005            $output .= $parsed_args['after'];
    10061006        }
    10071007    }
     
    10171017    $html = apply_filters( 'wp_link_pages', $output, $args );
    10181018
    1019     if ( $r['echo'] ) {
     1019    if ( $parsed_args['echo'] ) {
    10201020        echo $html;
    10211021    }
     
    11821182    );
    11831183
    1184     $r = wp_parse_args( $args, $defaults );
    1185 
    1186     $pages  = get_pages( $r );
     1184    $parsed_args = wp_parse_args( $args, $defaults );
     1185
     1186    $pages  = get_pages( $parsed_args );
    11871187    $output = '';
    11881188    // Back-compat with old system where both id and name were based on $name argument
    1189     if ( empty( $r['id'] ) ) {
    1190         $r['id'] = $r['name'];
     1189    if ( empty( $parsed_args['id'] ) ) {
     1190        $parsed_args['id'] = $parsed_args['name'];
    11911191    }
    11921192
    11931193    if ( ! empty( $pages ) ) {
    11941194        $class = '';
    1195         if ( ! empty( $r['class'] ) ) {
    1196             $class = " class='" . esc_attr( $r['class'] ) . "'";
    1197         }
    1198 
    1199         $output = "<select name='" . esc_attr( $r['name'] ) . "'" . $class . " id='" . esc_attr( $r['id'] ) . "'>\n";
    1200         if ( $r['show_option_no_change'] ) {
    1201             $output .= "\t<option value=\"-1\">" . $r['show_option_no_change'] . "</option>\n";
    1202         }
    1203         if ( $r['show_option_none'] ) {
    1204             $output .= "\t<option value=\"" . esc_attr( $r['option_none_value'] ) . '">' . $r['show_option_none'] . "</option>\n";
    1205         }
    1206         $output .= walk_page_dropdown_tree( $pages, $r['depth'], $r );
     1195        if ( ! empty( $parsed_args['class'] ) ) {
     1196            $class = " class='" . esc_attr( $parsed_args['class'] ) . "'";
     1197        }
     1198
     1199        $output = "<select name='" . esc_attr( $parsed_args['name'] ) . "'" . $class . " id='" . esc_attr( $parsed_args['id'] ) . "'>\n";
     1200        if ( $parsed_args['show_option_no_change'] ) {
     1201            $output .= "\t<option value=\"-1\">" . $parsed_args['show_option_no_change'] . "</option>\n";
     1202        }
     1203        if ( $parsed_args['show_option_none'] ) {
     1204            $output .= "\t<option value=\"" . esc_attr( $parsed_args['option_none_value'] ) . '">' . $parsed_args['show_option_none'] . "</option>\n";
     1205        }
     1206        $output .= walk_page_dropdown_tree( $pages, $parsed_args['depth'], $parsed_args );
    12071207        $output .= "</select>\n";
    12081208    }
     
    12121212     *
    12131213     * @since 2.1.0
    1214      * @since 4.4.0 `$r` and `$pages` added as arguments.
    1215      *
    1216      * @param string $output HTML output for drop down list of pages.
    1217      * @param array  $r      The parsed arguments array.
    1218      * @param array  $pages  List of WP_Post objects returned by `get_pages()`
     1214     * @since 4.4.0 `$parsed_args` and `$pages` added as arguments.
     1215     *
     1216     * @param string $output      HTML output for drop down list of pages.
     1217     * @param array  $parsed_args The parsed arguments array.
     1218     * @param array  $pages       List of WP_Post objects returned by `get_pages()`
    12191219     */
    1220     $html = apply_filters( 'wp_dropdown_pages', $output, $r, $pages );
    1221 
    1222     if ( $r['echo'] ) {
     1220    $html = apply_filters( 'wp_dropdown_pages', $output, $parsed_args, $pages );
     1221
     1222    if ( $parsed_args['echo'] ) {
    12231223        echo $html;
    12241224    }
     
    12831283    );
    12841284
    1285     $r = wp_parse_args( $args, $defaults );
    1286 
    1287     if ( ! in_array( $r['item_spacing'], array( 'preserve', 'discard' ), true ) ) {
     1285    $parsed_args = wp_parse_args( $args, $defaults );
     1286
     1287    if ( ! in_array( $parsed_args['item_spacing'], array( 'preserve', 'discard' ), true ) ) {
    12881288        // invalid value, fall back to default.
    1289         $r['item_spacing'] = $defaults['item_spacing'];
     1289        $parsed_args['item_spacing'] = $defaults['item_spacing'];
    12901290    }
    12911291
     
    12941294
    12951295    // sanitize, mostly to keep spaces out
    1296     $r['exclude'] = preg_replace( '/[^0-9,]/', '', $r['exclude'] );
     1296    $parsed_args['exclude'] = preg_replace( '/[^0-9,]/', '', $parsed_args['exclude'] );
    12971297
    12981298    // Allow plugins to filter an array of excluded pages (but don't put a nullstring into the array)
    1299     $exclude_array = ( $r['exclude'] ) ? explode( ',', $r['exclude'] ) : array();
     1299    $exclude_array = ( $parsed_args['exclude'] ) ? explode( ',', $parsed_args['exclude'] ) : array();
    13001300
    13011301    /**
     
    13061306     * @param array $exclude_array An array of page IDs to exclude.
    13071307     */
    1308     $r['exclude'] = implode( ',', apply_filters( 'wp_list_pages_excludes', $exclude_array ) );
     1308    $parsed_args['exclude'] = implode( ',', apply_filters( 'wp_list_pages_excludes', $exclude_array ) );
     1309
     1310    $parsed_args['hierarchical'] = 0;
    13091311
    13101312    // Query pages.
    1311     $r['hierarchical'] = 0;
    1312     $pages             = get_pages( $r );
     1313    $pages = get_pages( $parsed_args );
    13131314
    13141315    if ( ! empty( $pages ) ) {
    1315         if ( $r['title_li'] ) {
    1316             $output .= '<li class="pagenav">' . $r['title_li'] . '<ul>';
     1316        if ( $parsed_args['title_li'] ) {
     1317            $output .= '<li class="pagenav">' . $parsed_args['title_li'] . '<ul>';
    13171318        }
    13181319        global $wp_query;
     
    13261327        }
    13271328
    1328         $output .= walk_page_tree( $pages, $r['depth'], $current_page, $r );
    1329 
    1330         if ( $r['title_li'] ) {
     1329        $output .= walk_page_tree( $pages, $parsed_args['depth'], $current_page, $parsed_args );
     1330
     1331        if ( $parsed_args['title_li'] ) {
    13311332            $output .= '</ul></li>';
    13321333        }
     
    13411342     * @see wp_list_pages()
    13421343     *
    1343      * @param string $output HTML output of the pages list.
    1344      * @param array  $r      An array of page-listing arguments.
    1345      * @param array  $pages  List of WP_Post objects returned by `get_pages()`
     1344     * @param string $output      HTML output of the pages list.
     1345     * @param array  $parsed_args An array of page-listing arguments.
     1346     * @param array  $pages       List of WP_Post objects returned by `get_pages()`
    13461347     */
    1347     $html = apply_filters( 'wp_list_pages', $output, $r, $pages );
    1348 
    1349     if ( $r['echo'] ) {
     1348    $html = apply_filters( 'wp_list_pages', $output, $parsed_args, $pages );
     1349
     1350    if ( $parsed_args['echo'] ) {
    13501351        echo $html;
    13511352    } else {
Note: See TracChangeset for help on using the changeset viewer.