Skip to:
Content

BuddyPress.org

Ticket #4333: 4333.diff

File 4333.diff, 2.2 KB (added by cklosows, 12 years ago)

Removes hard coded arguments for bp_core_fetch_avatar and uses parsed arguments given.

  • bp-members/bp-members-template.php

     
    770770        function bp_get_loggedin_user_avatar( $args = '' ) {
    771771
    772772                $defaults = array(
    773                         'type'   => 'thumb',
    774                         'width'  => false,
    775                         'height' => false,
    776                         'html'   => true,
    777                         'alt'    => sprintf( __( 'Profile picture of %s', 'buddypress' ), bp_get_loggedin_user_fullname() )
     773                        'item_id' => bp_loggedin_user_id(),
     774                        'type'    => 'thumb',
     775                        'width'   => false,
     776                        'height'  => false,
     777                        'html'    => true,
     778                        'alt'     => sprintf( __( 'Profile picture of %s', 'buddypress' ), bp_get_loggedin_user_fullname() )
    778779                );
    779780
    780                 $r = wp_parse_args( $args, $defaults );
    781                 extract( $r, EXTR_SKIP );
     781                $avatar_args = wp_parse_args( $args, $defaults );
    782782
    783                 return apply_filters( 'bp_get_loggedin_user_avatar', bp_core_fetch_avatar( array( 'item_id' => bp_loggedin_user_id(), 'type' => $type, 'width' => $width, 'height' => $height, 'html' => $html, 'alt' => $alt ) ) );
     783                return apply_filters( 'bp_get_loggedin_user_avatar', bp_core_fetch_avatar( $avatar_args ) );
    784784        }
    785785
    786786function bp_displayed_user_avatar( $args = '' ) {
     
    789789        function bp_get_displayed_user_avatar( $args = '' ) {
    790790
    791791                $defaults = array(
    792                         'type'   => 'thumb',
    793                         'width'  => false,
    794                         'height' => false,
    795                         'html'   => true,
    796                         'alt'    => sprintf( __( 'Profile picture of %s', 'buddypress' ), bp_get_displayed_user_fullname() )
     792                        'item_id' => bp_displayed_user_id(),
     793                        'type'    => 'thumb',
     794                        'width'   => false,
     795                        'height'  => false,
     796                        'html'    => true,
     797                        'alt'     => sprintf( __( 'Profile picture of %s', 'buddypress' ), bp_get_displayed_user_fullname() )
    797798                );
    798799
    799                 $r = wp_parse_args( $args, $defaults );
    800                 extract( $r, EXTR_SKIP );
     800                $avatar_args = wp_parse_args( $args, $defaults );
    801801
    802                 return apply_filters( 'bp_get_displayed_user_avatar', bp_core_fetch_avatar( array( 'item_id' => bp_displayed_user_id(), 'type' => $type, 'width' => $width, 'height' => $height, 'html' => $html, 'alt' => $alt ) ) );
     802                return apply_filters( 'bp_get_displayed_user_avatar', bp_core_fetch_avatar( $avatar_args ) );
    803803        }
    804804
    805805function bp_displayed_user_email() {