Skip to:
Content

bbPress.org

Changeset 4498


Ignore:
Timestamp:
11/24/2012 06:36:00 PM (14 years ago)
Author:
johnjamesjacoby
Message:

Extract:

  • Remove extract() usages from /users/template-tag.php.
  • See #2056.
File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/includes/users/template-tags.php

    r4365 r4498  
    494494            $args = array( 'text' => $args );
    495495
    496         $defaults = array(
     496        $r = bbp_parse_args( $args, array(
    497497            'text'   => __( 'Admin', 'bbpress' ),
    498498            'before' => '',
    499499            'after'  => ''
    500         );
    501         $args = bbp_parse_args( $args, $defaults, 'get_admin_link' );
    502         extract( $args, EXTR_SKIP );
    503 
    504         $uri    = admin_url();
    505         $retval = $before . '<a href="' . $uri . '">' . $text . '</a>' . $after;
    506 
    507         return apply_filters( 'bbp_get_admin_link', $retval, $args );
     500        ), 'get_admin_link' );
     501
     502        $retval = $r['before'] . '<a href="' . admin_url() . '">' . $r['text'] . '</a>' . $r['after'];
     503
     504        return apply_filters( 'bbp_get_admin_link', $retval, $r );
    508505    }
    509506
    … …  
    532529    function bbp_get_author_ip( $args = '' ) {
    533530
    534         // Default arguments
    535         $defaults = array(
    536             'post_id' => 0,
     531        // Used as post id
     532        $post_id = is_numeric( $args ) ? (int) $args : 0;
     533        $r       = bbp_parse_args( $args, array(
     534            'post_id' => $post_id,
    537535            'before'  => '<span class="bbp-author-ip">(',
    538536            'after'   => ')</span>'
    539         );
    540 
    541         $r = bbp_parse_args( $args, $defaults, 'get_author_ip' );
    542         extract( $r );
    543 
    544         // Used as post id
    545         if ( is_numeric( $args ) )
    546             $post_id = $args;
     537        ), 'get_author_ip' );
    547538
    548539        // Get the author IP meta value
    549         $author_ip = get_post_meta( $post_id, '_bbp_author_ip', true );
     540        $author_ip = get_post_meta( $r['post_id'], '_bbp_author_ip', true );
    550541        if ( !empty( $author_ip ) ) {
    551             $author_ip = $before . $author_ip . $after;
     542            $author_ip = $r['before'] . $author_ip . $r['after'];
    552543
    553544        // No IP address
    … …  
    556547        }
    557548
    558         return apply_filters( 'bbp_get_author_ip', $author_ip, $args );
     549        return apply_filters( 'bbp_get_author_ip', $author_ip, $r );
    559550    }
    560551
    … …  
    822813            return;
    823814
    824         $defaults = array (
     815        $r = bbp_parse_args( $args, array(
    825816            'subscribe'   => __( 'Subscribe',   'bbpress' ),
    826817            'unsubscribe' => __( 'Unsubscribe', 'bbpress' ),
    … …  
    829820            'before'      => '&nbsp;|&nbsp;',
    830821            'after'       => ''
    831         );
    832         $args = bbp_parse_args( $args, $defaults, 'get_user_subscribe_link' );
    833         extract( $args );
     822        ), 'get_user_subscribe_link' );
    834823
    835824        // Validate user and topic ID's
    836         $user_id  = bbp_get_user_id( $user_id, true, true );
    837         $topic_id = bbp_get_topic_id( $topic_id );
     825        $user_id  = bbp_get_user_id( $r['user_id'], true, true );
     826        $topic_id = bbp_get_topic_id( $r['topic_id'] );
    838827        if ( empty( $user_id ) || empty( $topic_id ) ) {
    839828            return false;
    … …  
    848837        $is_subscribed = bbp_is_user_subscribed( $user_id, $topic_id );
    849838        if ( !empty( $is_subscribed ) ) {
    850             $text       = $unsubscribe;
     839            $text       = $r['unsubscribe'];
    851840            $query_args = array( 'action' => 'bbp_unsubscribe', 'topic_id' => $topic_id );
    852841        } else {
    853             $text       = $subscribe;
     842            $text       = $r['subscribe'];
    854843            $query_args = array( 'action' => 'bbp_subscribe', 'topic_id' => $topic_id );
    855844        }
    … …  
    867856        }
    868857
    869         $url           = esc_url( wp_nonce_url( add_query_arg( $query_args, $permalink ), 'toggle-subscription_' . $topic_id ) );
    870         $is_subscribed = $is_subscribed ? 'is-subscribed' : '';
    871         $html          = '<span id="subscription-toggle">' . $before . '<span id="subscribe-' . $topic_id . '" class="' . $is_subscribed . '"><a href="' . $url . '" class="dim:subscription-toggle:subscribe-' . $topic_id . ':is-subscribed">' . $text . '</a></span>' . $after . '</span>';
     858        $url        = esc_url( wp_nonce_url( add_query_arg( $query_args, $permalink ), 'toggle-subscription_' . $topic_id ) );
     859        $subscribed = $is_subscribed ? ' class="is_subscribed"' : '';
     860        $html       = '<span id="subscription-toggle">' . $r['before'] . '<span id="subscribe-' . $topic_id . '"' . $subscribed . '><a href="' . $url . '" class="dim:subscription-toggle:subscribe-' . $topic_id . ':is-subscribed">' . $text . '</a></span>' . $r['after'] . '</span>';
    872861
    873862        // Return the link
    874         return apply_filters( 'bbp_get_user_subscribe_link', $html, $args, $user_id, $topic_id );
     863        return apply_filters( 'bbp_get_user_subscribe_link', $html, $r, $user_id, $topic_id );
    875864    }
    876865
    … …  
    13491338    function bbp_get_author_link( $args = '' ) {
    13501339
    1351         // Default arguments
    1352         $defaults = array(
    1353             'post_id'    => 0,
     1340        $post_id = is_numeric( $args ) ? (int) $args : 0;
     1341        $r       = bbp_parse_args( $args, array(
     1342            'post_id'    => $post_id,
    13541343            'link_title' => '',
    13551344            'type'       => 'both',
    13561345            'size'       => 80
    1357         );
    1358         $r = bbp_parse_args( $args, $defaults, 'get_author_link' );
    1359         extract( $r );
    1360 
    1361         // Used as reply_id
    1362         if ( is_numeric( $args ) )
    1363             $post_id = $args;
     1346        ), 'get_author_link' );
    13641347
    13651348        // Confirmed topic
    1366         if ( bbp_is_topic( $post_id ) ) {
    1367             return bbp_get_topic_author_link( $args );
     1349        if ( bbp_is_topic( $r['post_id'] ) ) {
     1350            return bbp_get_topic_author_link( $r );
    13681351
    13691352        // Confirmed reply
    1370         } elseif ( bbp_is_reply( $post_id ) ) {
    1371             return bbp_get_reply_author_link( $args );
     1353        } elseif ( bbp_is_reply( $r['post_id'] ) ) {
     1354            return bbp_get_reply_author_link( $r );
     1355        }
    13721356
    13731357        // Get the post author and proceed
    1374         } else {
    1375             $user_id = get_post_field( 'post_author', $post_id );
    1376         }
     1358        $user_id = get_post_field( 'post_author', $r['post_id'] );
    13771359
    13781360        // Neither a reply nor a topic, so could be a revision
    1379         if ( !empty( $post_id ) ) {
     1361        if ( !empty( $r['post_id'] ) ) {
    13801362
    13811363            // Generate title with the display name of the author
    1382             if ( empty( $link_title ) ) {
    1383                 $link_title = sprintf( !bbp_is_reply_anonymous( $post_id ) ? __( 'View %s\'s profile', 'bbpress' ) : __( 'Visit %s\'s website', 'bbpress' ), get_the_author_meta( 'display_name', $user_id ) );
     1364            if ( empty( $r['link_title'] ) ) {
     1365                $r['link_title'] = sprintf( !bbp_is_reply_anonymous( $r['post_id'] ) ? __( 'View %s\'s profile', 'bbpress' ) : __( 'Visit %s\'s website', 'bbpress' ), get_the_author_meta( 'display_name', $user_id ) );
    13841366            }
    13851367
    13861368            // Assemble some link bits
    1387             $link_title = !empty( $link_title ) ? ' title="' . $link_title . '"' : '';
     1369            $link_title = !empty( $r['link_title'] ) ? ' title="' . $r['link_title'] . '"' : '';
    13881370            $author_url = bbp_get_user_profile_url( $user_id );
    1389             $anonymous  = bbp_is_reply_anonymous( $post_id );
     1371            $anonymous  = bbp_is_reply_anonymous( $r['post_id'] );
    13901372
    13911373            // Get avatar
    1392             if ( 'avatar' == $type || 'both' == $type ) {
    1393                 $author_links[] = get_avatar( $user_id, $size );
     1374            if ( 'avatar' == $r['type'] || 'both' == $r['type'] ) {
     1375                $author_links[] = get_avatar( $user_id, $r['size'] );
    13941376            }
    13951377
    13961378            // Get display name
    1397             if ( 'name' == $type   || 'both' == $type ) {
     1379            if ( 'name' == $r['type'] || 'both' == $r['type'] ) {
    13981380                $author_links[] = get_the_author_meta( 'display_name', $user_id );
    13991381            }
    … …  
    14161398        }
    14171399
    1418         return apply_filters( 'bbp_get_author_link', $author_link, $args );
     1400        return apply_filters( 'bbp_get_author_link', $author_link, $r );
    14191401    }
    14201402
    … …  
    14441426
    14451427    // Default arguments
    1446     $defaults = array(
     1428    $r = bbp_parse_args( $args, array(
    14471429        'user_id'         => bbp_get_current_user_id(),
    14481430        'forum_id'        => bbp_get_forum_id(),
    14491431        'check_ancestors' => false
    1450     );
    1451     $r = bbp_parse_args( $args, $defaults, 'user_can_view_forum' );
    1452     extract( $r );
     1432    ), 'user_can_view_forum' );
    14531433
    14541434    // Validate parsed values
    1455     $user_id  = bbp_get_user_id ( $user_id, false, false );
    1456     $forum_id = bbp_get_forum_id( $forum_id );
     1435    $user_id  = bbp_get_user_id ( $r['user_id'], false, false );
     1436    $forum_id = bbp_get_forum_id( $r['forum_id'] );
    14571437    $retval   = false;
    14581438
    … …  
    14621442
    14631443    // Forum is public, and user can read forums or is not logged in
    1464     } elseif ( bbp_is_forum_public ( $forum_id, $check_ancestors ) ) {
     1444    } elseif ( bbp_is_forum_public ( $forum_id, $r['check_ancestors'] ) ) {
    14651445        $retval = true;
    14661446
    14671447    // Forum is private, and user can see it
    1468     } elseif ( bbp_is_forum_private( $forum_id, $check_ancestors ) && current_user_can( 'read_private_forums' ) ) {
     1448    } elseif ( bbp_is_forum_private( $forum_id, $r['check_ancestors'] ) && user_can( $user_id, 'read_private_forums' ) ) {
    14691449        $retval = true;
    14701450
    14711451    // Forum is hidden, and user can see it
    1472     } elseif ( bbp_is_forum_hidden ( $forum_id, $check_ancestors ) && current_user_can( 'read_hidden_forums'  ) ) {
     1452    } elseif ( bbp_is_forum_hidden ( $forum_id, $r['check_ancestors'] ) && user_can( $user_id, 'read_hidden_forums'  ) ) {
    14731453        $retval = true;
    14741454    }
Note: See TracChangeset for help on using the changeset viewer.