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/comment-template.php

    r45602 r45667  
    20152015    );
    20162016
    2017     $r = wp_parse_args( $args, $defaults );
     2017    $parsed_args = wp_parse_args( $args, $defaults );
    20182018
    20192019    /**
     
    20242024     * @see wp_list_comments()
    20252025     *
    2026      * @param array $r An array of arguments for displaying comments.
    2027      */
    2028     $r = apply_filters( 'wp_list_comments_args', $r );
     2026     * @param array $parsed_args An array of arguments for displaying comments.
     2027     */
     2028    $parsed_args = apply_filters( 'wp_list_comments_args', $parsed_args );
    20292029
    20302030    // Figure out what comments we'll be looping through ($_comments)
     
    20342034            return;
    20352035        }
    2036         if ( 'all' != $r['type'] ) {
     2036        if ( 'all' != $parsed_args['type'] ) {
    20372037            $comments_by_type = separate_comments( $comments );
    2038             if ( empty( $comments_by_type[ $r['type'] ] ) ) {
     2038            if ( empty( $comments_by_type[ $parsed_args['type'] ] ) ) {
    20392039                return;
    20402040            }
    2041             $_comments = $comments_by_type[ $r['type'] ];
     2041            $_comments = $comments_by_type[ $parsed_args['type'] ];
    20422042        } else {
    20432043            $_comments = $comments;
     
    20482048         * perform a separate comment query and allow Walker_Comment to paginate.
    20492049         */
    2050         if ( $r['page'] || $r['per_page'] ) {
     2050        if ( $parsed_args['page'] || $parsed_args['per_page'] ) {
    20512051            $current_cpage = get_query_var( 'cpage' );
    20522052            if ( ! $current_cpage ) {
     
    20552055
    20562056            $current_per_page = get_query_var( 'comments_per_page' );
    2057             if ( $r['page'] != $current_cpage || $r['per_page'] != $current_per_page ) {
     2057            if ( $parsed_args['page'] != $current_cpage || $parsed_args['per_page'] != $current_per_page ) {
    20582058                $comment_args = array(
    20592059                    'post_id' => get_the_ID(),
     
    20752075                $comments = get_comments( $comment_args );
    20762076
    2077                 if ( 'all' != $r['type'] ) {
     2077                if ( 'all' != $parsed_args['type'] ) {
    20782078                    $comments_by_type = separate_comments( $comments );
    2079                     if ( empty( $comments_by_type[ $r['type'] ] ) ) {
     2079                    if ( empty( $comments_by_type[ $parsed_args['type'] ] ) ) {
    20802080                        return;
    20812081                    }
    20822082
    2083                     $_comments = $comments_by_type[ $r['type'] ];
     2083                    $_comments = $comments_by_type[ $parsed_args['type'] ];
    20842084                } else {
    20852085                    $_comments = $comments;
     
    20922092                return;
    20932093            }
    2094             if ( 'all' != $r['type'] ) {
     2094            if ( 'all' != $parsed_args['type'] ) {
    20952095                if ( empty( $wp_query->comments_by_type ) ) {
    20962096                    $wp_query->comments_by_type = separate_comments( $wp_query->comments );
    20972097                }
    2098                 if ( empty( $wp_query->comments_by_type[ $r['type'] ] ) ) {
     2098                if ( empty( $wp_query->comments_by_type[ $parsed_args['type'] ] ) ) {
    20992099                    return;
    21002100                }
    2101                 $_comments = $wp_query->comments_by_type[ $r['type'] ];
     2101                $_comments = $wp_query->comments_by_type[ $parsed_args['type'] ];
    21022102            } else {
    21032103                $_comments = $wp_query->comments;
     
    21082108                $cpage                 = get_query_var( 'cpage' );
    21092109                if ( 'newest' === $default_comments_page ) {
    2110                     $r['cpage'] = $cpage;
     2110                    $parsed_args['cpage'] = $cpage;
    21112111
    21122112                    /*
     
    21152115                    */
    21162116                } elseif ( $cpage == 1 ) {
    2117                     $r['cpage'] = '';
     2117                    $parsed_args['cpage'] = '';
    21182118                } else {
    2119                     $r['cpage'] = $cpage;
     2119                    $parsed_args['cpage'] = $cpage;
    21202120                }
    21212121
    2122                 $r['page']     = 0;
    2123                 $r['per_page'] = 0;
     2122                $parsed_args['page']     = 0;
     2123                $parsed_args['per_page'] = 0;
    21242124            }
    21252125        }
    21262126    }
    21272127
    2128     if ( '' === $r['per_page'] && get_option( 'page_comments' ) ) {
    2129         $r['per_page'] = get_query_var( 'comments_per_page' );
    2130     }
    2131 
    2132     if ( empty( $r['per_page'] ) ) {
    2133         $r['per_page'] = 0;
    2134         $r['page']     = 0;
    2135     }
    2136 
    2137     if ( '' === $r['max_depth'] ) {
     2128    if ( '' === $parsed_args['per_page'] && get_option( 'page_comments' ) ) {
     2129        $parsed_args['per_page'] = get_query_var( 'comments_per_page' );
     2130    }
     2131
     2132    if ( empty( $parsed_args['per_page'] ) ) {
     2133        $parsed_args['per_page'] = 0;
     2134        $parsed_args['page']     = 0;
     2135    }
     2136
     2137    if ( '' === $parsed_args['max_depth'] ) {
    21382138        if ( get_option( 'thread_comments' ) ) {
    2139             $r['max_depth'] = get_option( 'thread_comments_depth' );
     2139            $parsed_args['max_depth'] = get_option( 'thread_comments_depth' );
    21402140        } else {
    2141             $r['max_depth'] = -1;
     2141            $parsed_args['max_depth'] = -1;
    21422142        }
    21432143    }
    21442144
    2145     if ( '' === $r['page'] ) {
     2145    if ( '' === $parsed_args['page'] ) {
    21462146        if ( empty( $overridden_cpage ) ) {
    2147             $r['page'] = get_query_var( 'cpage' );
     2147            $parsed_args['page'] = get_query_var( 'cpage' );
    21482148        } else {
    2149             $threaded  = ( -1 != $r['max_depth'] );
    2150             $r['page'] = ( 'newest' == get_option( 'default_comments_page' ) ) ? get_comment_pages_count( $_comments, $r['per_page'], $threaded ) : 1;
    2151             set_query_var( 'cpage', $r['page'] );
     2149            $threaded            = ( -1 != $parsed_args['max_depth'] );
     2150            $parsed_args['page'] = ( 'newest' == get_option( 'default_comments_page' ) ) ? get_comment_pages_count( $_comments, $parsed_args['per_page'], $threaded ) : 1;
     2151            set_query_var( 'cpage', $parsed_args['page'] );
    21522152        }
    21532153    }
    21542154    // Validation check
    2155     $r['page'] = intval( $r['page'] );
    2156     if ( 0 == $r['page'] && 0 != $r['per_page'] ) {
    2157         $r['page'] = 1;
    2158     }
    2159 
    2160     if ( null === $r['reverse_top_level'] ) {
    2161         $r['reverse_top_level'] = ( 'desc' == get_option( 'comment_order' ) );
     2155    $parsed_args['page'] = intval( $parsed_args['page'] );
     2156    if ( 0 == $parsed_args['page'] && 0 != $parsed_args['per_page'] ) {
     2157        $parsed_args['page'] = 1;
     2158    }
     2159
     2160    if ( null === $parsed_args['reverse_top_level'] ) {
     2161        $parsed_args['reverse_top_level'] = ( 'desc' == get_option( 'comment_order' ) );
    21622162    }
    21632163
    21642164    wp_queue_comments_for_comment_meta_lazyload( $_comments );
    21652165
    2166     if ( empty( $r['walker'] ) ) {
     2166    if ( empty( $parsed_args['walker'] ) ) {
    21672167        $walker = new Walker_Comment;
    21682168    } else {
    2169         $walker = $r['walker'];
    2170     }
    2171 
    2172     $output = $walker->paged_walk( $_comments, $r['max_depth'], $r['page'], $r['per_page'], $r );
     2169        $walker = $parsed_args['walker'];
     2170    }
     2171
     2172    $output = $walker->paged_walk( $_comments, $parsed_args['max_depth'], $parsed_args['page'], $parsed_args['per_page'], $parsed_args );
    21732173
    21742174    $in_comment_loop = false;
    21752175
    2176     if ( $r['echo'] ) {
     2176    if ( $parsed_args['echo'] ) {
    21772177        echo $output;
    21782178    } else {
Note: See TracChangeset for help on using the changeset viewer.