WordPress.org

Make WordPress Core


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