WordPress.org
Get WordPress

Make WordPress Core


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

    r45424 r45667  
    6363        );
    6464
    65         $r = wp_parse_args( $args, $defaults );
     65        $parsed_args = wp_parse_args( $args, $defaults );
    6666
    6767        $output = ''; // Blank string to start with.
    … …  
    7171                        $bookmark->recently_updated = false;
    7272                }
    73                 $output .= $r['before'];
    74                 if ( $r['show_updated'] && $bookmark->recently_updated ) {
     73                $output .= $parsed_args['before'];
     74                if ( $parsed_args['show_updated'] && $bookmark->recently_updated ) {
    7575                        $output .= '<em>';
    7676                }
    … …  
    8383                $title = $desc;
    8484
    85                 if ( $r['show_updated'] ) {
     85                if ( $parsed_args['show_updated'] ) {
    8686                        if ( '00' != substr( $bookmark->link_updated_f, 0, 2 ) ) {
    8787                                $title .= ' (';
    … …  
    9696                        }
    9797                }
    98                 $alt = ' alt="' . $name . ( $r['show_description'] ? ' ' . $title : '' ) . '"';
     98                $alt = ' alt="' . $name . ( $parsed_args['show_description'] ? ' ' . $title : '' ) . '"';
    9999
    100100                if ( '' != $title ) {
    … …  
    111111                $output .= '<a href="' . $the_link . '"' . $rel . $title . $target . '>';
    112112
    113                 $output .= $r['link_before'];
    114 
    115                 if ( $bookmark->link_image != null && $r['show_images'] ) {
     113                $output .= $parsed_args['link_before'];
     114
     115                if ( $bookmark->link_image != null && $parsed_args['show_images'] ) {
    116116                        if ( strpos( $bookmark->link_image, 'http' ) === 0 ) {
    117117                                $output .= "<img src=\"$bookmark->link_image\" $alt $title />";
    … …  
    119119                                $output .= '<img src="' . get_option( 'siteurl' ) . "$bookmark->link_image\" $alt $title />";
    120120                        }
    121                         if ( $r['show_name'] ) {
     121                        if ( $parsed_args['show_name'] ) {
    122122                                $output .= " $name";
    123123                        }
    … …  
    126126                }
    127127
    128                 $output .= $r['link_after'];
     128                $output .= $parsed_args['link_after'];
    129129
    130130                $output .= '</a>';
    131131
    132                 if ( $r['show_updated'] && $bookmark->recently_updated ) {
     132                if ( $parsed_args['show_updated'] && $bookmark->recently_updated ) {
    133133                        $output .= '</em>';
    134134                }
    135135
    136                 if ( $r['show_description'] && '' != $desc ) {
    137                         $output .= $r['between'] . $desc;
    138                 }
    139 
    140                 if ( $r['show_rating'] ) {
    141                         $output .= $r['between'] . sanitize_bookmark_field(
     136                if ( $parsed_args['show_description'] && '' != $desc ) {
     137                        $output .= $parsed_args['between'] . $desc;
     138                }
     139
     140                if ( $parsed_args['show_rating'] ) {
     141                        $output .= $parsed_args['between'] . sanitize_bookmark_field(
    142142                                'link_rating',
    143143                                $bookmark->link_rating,
    … …  
    146146                        );
    147147                }
    148                 $output .= $r['after'] . "\n";
     148                $output .= $parsed_args['after'] . "\n";
    149149        } // end while
    150150
    … …  
    226226        );
    227227
    228         $r = wp_parse_args( $args, $defaults );
     228        $parsed_args = wp_parse_args( $args, $defaults );
    229229
    230230        $output = '';
    231231
    232         if ( ! is_array( $r['class'] ) ) {
    233                 $r['class'] = explode( ' ', $r['class'] );
     232        if ( ! is_array( $parsed_args['class'] ) ) {
     233                $parsed_args['class'] = explode( ' ', $parsed_args['class'] );
    234234        }
    235         $r['class'] = array_map( 'sanitize_html_class', $r['class'] );
    236         $r['class'] = trim( join( ' ', $r['class'] ) );
    237 
    238         if ( $r['categorize'] ) {
     235        $parsed_args['class'] = array_map( 'sanitize_html_class', $parsed_args['class'] );
     236        $parsed_args['class'] = trim( join( ' ', $parsed_args['class'] ) );
     237
     238        if ( $parsed_args['categorize'] ) {
    239239                $cats = get_terms(
    240240                        'link_category',
    241241                        array(
    242                                 'name__like'   => $r['category_name'],
    243                                 'include'      => $r['category'],
    244                                 'exclude'      => $r['exclude_category'],
    245                                 'orderby'      => $r['category_orderby'],
    246                                 'order'        => $r['category_order'],
     242                                'name__like'   => $parsed_args['category_name'],
     243                                'include'      => $parsed_args['category'],
     244                                'exclude'      => $parsed_args['exclude_category'],
     245                                'orderby'      => $parsed_args['category_orderby'],
     246                                'order'        => $parsed_args['category_order'],
    247247                                'hierarchical' => 0,
    248248                        )
    249249                );
    250250                if ( empty( $cats ) ) {
    251                         $r['categorize'] = false;
     251                        $parsed_args['categorize'] = false;
    252252                }
    253253        }
    254254
    255         if ( $r['categorize'] ) {
     255        if ( $parsed_args['categorize'] ) {
    256256                // Split the bookmarks into ul's for each category
    257257                foreach ( (array) $cats as $cat ) {
    258                         $params    = array_merge( $r, array( 'category' => $cat->term_id ) );
     258                        $params    = array_merge( $parsed_args, array( 'category' => $cat->term_id ) );
    259259                        $bookmarks = get_bookmarks( $params );
    260260                        if ( empty( $bookmarks ) ) {
    … …  
    263263                        $output .= str_replace(
    264264                                array( '%id', '%class' ),
    265                                 array( "linkcat-$cat->term_id", $r['class'] ),
    266                                 $r['category_before']
     265                                array( "linkcat-$cat->term_id", $parsed_args['class'] ),
     266                                $parsed_args['category_before']
    267267                        );
    268268                        /**
    … …  
    275275                        $catname = apply_filters( 'link_category', $cat->name );
    276276
    277                         $output .= $r['title_before'];
     277                        $output .= $parsed_args['title_before'];
    278278                        $output .= $catname;
    279                         $output .= $r['title_after'];
     279                        $output .= $parsed_args['title_after'];
    280280                        $output .= "\n\t<ul class='xoxo blogroll'>\n";
    281                         $output .= _walk_bookmarks( $bookmarks, $r );
     281                        $output .= _walk_bookmarks( $bookmarks, $parsed_args );
    282282                        $output .= "\n\t</ul>\n";
    283                         $output .= $r['category_after'] . "\n";
     283                        $output .= $parsed_args['category_after'] . "\n";
    284284                }
    285285        } else {
    286286                //output one single list using title_li for the title
    287                 $bookmarks = get_bookmarks( $r );
     287                $bookmarks = get_bookmarks( $parsed_args );
    288288
    289289                if ( ! empty( $bookmarks ) ) {
    290                         if ( ! empty( $r['title_li'] ) ) {
     290                        if ( ! empty( $parsed_args['title_li'] ) ) {
    291291                                $output .= str_replace(
    292292                                        array( '%id', '%class' ),
    293                                         array( 'linkcat-' . $r['category'], $r['class'] ),
    294                                         $r['category_before']
     293                                        array( 'linkcat-' . $parsed_args['category'], $parsed_args['class'] ),
     294                                        $parsed_args['category_before']
    295295                                );
    296                                 $output .= $r['title_before'];
    297                                 $output .= $r['title_li'];
    298                                 $output .= $r['title_after'];
     296                                $output .= $parsed_args['title_before'];
     297                                $output .= $parsed_args['title_li'];
     298                                $output .= $parsed_args['title_after'];
    299299                                $output .= "\n\t<ul class='xoxo blogroll'>\n";
    300                                 $output .= _walk_bookmarks( $bookmarks, $r );
     300                                $output .= _walk_bookmarks( $bookmarks, $parsed_args );
    301301                                $output .= "\n\t</ul>\n";
    302                                 $output .= $r['category_after'] . "\n";
     302                                $output .= $parsed_args['category_after'] . "\n";
    303303                        } else {
    304                                 $output .= _walk_bookmarks( $bookmarks, $r );
     304                                $output .= _walk_bookmarks( $bookmarks, $parsed_args );
    305305                        }
    306306                }
    … …  
    316316        $html = apply_filters( 'wp_list_bookmarks', $output );
    317317
    318         if ( ! $r['echo'] ) {
     318        if ( ! $parsed_args['echo'] ) {
    319319                return $html;
    320320        }
Note: See TracChangeset for help on using the changeset viewer.