Skip to:
Content

BuddyPress.org

Changeset 5072


Ignore:
Timestamp:
08/31/2011 01:12:04 AM (10 years ago)
Author:
johnjamesjacoby
Message:

Audit _is_ functions and _current_component_ functions. Props r-a-y. Fixes #3399.

Location:
trunk
Files:
7 edited

Legend:

Unmodified
Added
Removed
  • trunk/bp-activity/bp-activity-actions.php

    r4961 r5072  
    242242    global $bp, $wp_query;
    243243
    244     if ( !bp_is_current_component( 'activity' ) || !bp_is_current_action( 'feed' ) || bp_is_user() || !empty( $bp->groups->current_group ) )
     244    if ( !bp_is_activity_component() || !bp_is_current_action( 'feed' ) || bp_is_user() || !empty( $bp->groups->current_group ) )
    245245        return false;
    246246
     
    256256    global $bp, $wp_query;
    257257
    258     if ( !bp_is_current_component( 'activity' ) || !bp_is_user() || !bp_is_current_action( 'feed' ) )
     258    if ( !bp_is_user_activity() || !bp_is_current_action( 'feed' ) )
    259259        return false;
    260260
     
    270270    global $bp, $wp_query;
    271271
    272     if ( !bp_is_active( 'friends' ) || !bp_is_current_component( 'activity' ) || !bp_is_user() || !bp_is_current_action( bp_get_friends_slug() ) || !bp_is_action_variable( 'feed', 0 ) )
     272    if ( !bp_is_active( 'friends' ) || !bp_is_user_activity() || !bp_is_current_action( bp_get_friends_slug() ) || !bp_is_action_variable( 'feed', 0 ) )
    273273        return false;
    274274
     
    284284    global $bp, $wp_query;
    285285
    286     if ( !bp_is_active( 'groups' ) || !bp_is_current_component( 'activity' ) || !bp_is_user() || !bp_is_current_action( bp_get_groups_slug() ) || !bp_is_action_variable( 'feed', 0 ) )
     286    if ( !bp_is_active( 'groups' ) || !bp_is_user_activity() || !bp_is_current_action( bp_get_groups_slug() ) || !bp_is_action_variable( 'feed', 0 ) )
    287287        return false;
    288288
     
    298298    global $bp, $wp_query;
    299299
    300     if ( !bp_is_current_component( 'activity' ) || !bp_is_user() || !bp_is_current_action( 'mentions' ) || !bp_is_action_variable( 'feed', 0 ) )
     300    if ( !bp_is_user_activity() || !bp_is_current_action( 'mentions' ) || !bp_is_action_variable( 'feed', 0 ) )
    301301        return false;
    302302
     
    312312    global $bp, $wp_query;
    313313
    314     if ( !bp_is_current_component( 'activity' ) || !bp_is_user() || !bp_is_current_action( 'favorites' ) || !bp_is_action_variable( 'feed', 0 ) )
     314    if ( !bp_is_user_activity() || !bp_is_current_action( 'favorites' ) || !bp_is_action_variable( 'feed', 0 ) )
    315315        return false;
    316316
  • trunk/bp-core/bp-core-template.php

    r5071 r5072  
    587587function bp_action_variable( $position = 0 ) {
    588588    $action_variables = bp_action_variables();
    589 
    590     $action_variable = isset( $action_variables[$position] ) ? $action_variables[$position] : false;
     589    $action_variable  = isset( $action_variables[$position] ) ? $action_variables[$position] : false;
    591590
    592591    return apply_filters( 'bp_action_variable', $action_variable, $position );
     
    654653
    655654            // Component has specific root slug
    656             if ( !empty( $bp->{$component_name}->root_slug ) )
     655            if ( !empty( $bp->{$component_name}->root_slug ) ) {
    657656                $root_slug = $bp->{$component_name}->root_slug;
     657            }
    658658        }
    659659
     
    718718     */
    719719    function bp_get_search_slug() {
    720         global $bp;
    721 
    722720        return apply_filters( 'bp_get_search_slug', BP_SEARCH_SLUG );
    723721    }
     
    967965 */
    968966function bp_is_blog_page() {
    969     global $bp, $wp_query;
     967    global $wp_query;
    970968
    971969    $is_blog_page = false;
     
    981979
    982980function bp_is_page( $page ) {
    983     global $bp;
    984 
    985981    if ( !bp_is_user() && bp_is_current_component( $page )  )
    986982        return true;
     
    1003999}
    10041000
     1001function bp_is_members_component() {
     1002    if ( bp_is_current_component( 'members' ) )
     1003        return true;
     1004
     1005    return false;
     1006}
     1007
    10051008function bp_is_profile_component() {
    10061009    if ( bp_is_current_component( 'xprofile' ) )
     
    10321035
    10331036function bp_is_friends_component() {
    1034 
    10351037    if ( bp_is_current_component( 'friends' ) )
    10361038        return true;
     
    10651067    global $bp;
    10661068
    1067     if ( bp_is_current_component( 'activity' ) && is_numeric( $bp->current_action ) )
     1069    if ( bp_is_activity_component() && is_numeric( $bp->current_action ) )
    10681070        return true;
    10691071
     
    10941096
    10951097function bp_is_user_activity() {
    1096     global $bp;
    1097 
    1098     if ( bp_is_current_component( 'activity' ) )
     1098    if ( bp_is_user() && bp_is_activity_component() )
    10991099        return true;
    11001100
     
    11031103
    11041104function bp_is_user_friends_activity() {
    1105     global $bp;
    11061105
    11071106    if ( !bp_is_active( 'friends' ) )
    11081107        return false;
    11091108
    1110     if ( bp_is_activity_component() && bp_is_current_action( bp_get_friends_slug() ) )
     1109    $slug = bp_get_friends_slug();
     1110
     1111    if ( empty( $slug ) )
     1112        $slug = 'friends';
     1113
     1114    if ( bp_is_user_activity() && bp_is_current_action( $slug ) )
    11111115        return true;
    11121116
     
    11151119
    11161120function bp_is_user_groups_activity() {
    1117     global $bp;
    11181121
    11191122    if ( !bp_is_active( 'groups' ) )
    11201123        return false;
    11211124
    1122     if ( bp_is_activity_component() && bp_is_current_action( bp_get_groups_slug() ) )
     1125    $slug = bp_get_groups_slug();
     1126
     1127    if ( empty( $slug ) )
     1128        $slug = 'groups';
     1129
     1130    if ( bp_is_user_activity() && bp_is_current_action( $slug ) )
    11231131        return true;
    11241132
     
    11271135
    11281136function bp_is_user_profile() {
    1129     global $bp;
    1130 
    11311137    if ( bp_is_profile_component() || bp_is_current_component( 'profile' ) )
    11321138        return true;
     
    11361142
    11371143function bp_is_user_profile_edit() {
    1138     global $bp;
    1139 
    11401144    if ( bp_is_profile_component() && bp_is_current_action( 'edit' ) )
    11411145        return true;
     
    11451149
    11461150function bp_is_user_change_avatar() {
    1147     global $bp;
    1148 
    1149     if ( bp_is_current_component( 'xprofile' ) && bp_is_current_action( 'change-avatar' ) )
     1151    if ( bp_is_profile_component() && bp_is_current_action( 'change-avatar' ) )
    11501152        return true;
    11511153
     
    11611163 */
    11621164function bp_is_user_forums() {
    1163     if ( bp_is_current_component( 'forums' ) )
     1165    if ( bp_is_user() && bp_is_forums_component() )
    11641166        return true;
    11651167
     
    11761178 */
    11771179function bp_is_user_forums_started() {
    1178     if ( bp_is_current_component( 'forums' ) && bp_is_current_action( 'topics' ) )
     1180    if ( bp_is_user_forums && bp_is_current_action( 'topics' ) )
    11791181        return true;
    11801182
     
    11911193 */
    11921194function bp_is_user_forums_replied_to() {
    1193     if ( bp_is_current_component( 'forums' ) && bp_is_current_action( 'replies' ) )
     1195    if ( bp_is_user_forums && bp_is_current_action( 'replies' ) )
    11941196        return true;
    11951197
     
    11981200
    11991201function bp_is_user_groups() {
    1200     global $bp;
    1201 
    1202     if ( bp_is_current_component( 'groups' ) )
     1202    if ( bp_is_user() && bp_is_groups_component() )
    12031203        return true;
    12041204
     
    12071207
    12081208function bp_is_user_blogs() {
    1209     global $bp;
    1210 
    1211     if ( is_multisite() && bp_is_current_component( 'blogs' ) )
     1209    if ( bp_is_user() && bp_is_blogs_component() )
    12121210        return true;
    12131211
     
    12161214
    12171215function bp_is_user_recent_posts() {
    1218     global $bp;
    1219 
    1220     if ( is_multisite() && bp_is_current_component( 'blogs' ) && bp_is_current_action( 'recent-posts' ) )
     1216    if ( bp_is_user_blogs() && bp_is_current_action( 'recent-posts' ) )
    12211217        return true;
    12221218
     
    12251221
    12261222function bp_is_user_recent_commments() {
    1227     global $bp;
    1228 
    1229     if ( is_multisite() && bp_is_current_component( 'blogs' ) && bp_is_current_action( 'recent-comments' ) )
     1223    if ( bp_is_user_blogs() && bp_is_current_action( 'recent-comments' ) )
    12301224        return true;
    12311225
     
    12341228
    12351229function bp_is_user_friends() {
    1236 
    1237     if ( bp_is_current_component( 'friends' ) )
     1230    if ( bp_is_user() && bp_is_friends_component() )
    12381231        return true;
    12391232
     
    12421235
    12431236function bp_is_user_friend_requests() {
    1244     global $bp;
    1245 
    1246     if ( bp_is_current_component( 'friends' ) && bp_is_current_action( 'requests' ) )
     1237    if ( bp_is_user_friends() && bp_is_current_action( 'requests' ) )
    12471238        return true;
    12481239
     
    12621253
    12631254function bp_is_group_home() {
    1264     global $bp;
    1265 
    12661255    if ( bp_is_single_item() && bp_is_groups_component() && ( !bp_current_action() || bp_is_current_action( 'home' ) ) )
    12671256        return true;
     
    12711260
    12721261function bp_is_group_create() {
    1273     global $bp;
    1274 
    12751262    if ( bp_is_groups_component() && bp_is_current_action( 'create' ) )
    12761263        return true;
     
    12801267
    12811268function bp_is_group_admin_page() {
    1282     global $bp;
    1283 
    12841269    if ( bp_is_single_item() && bp_is_groups_component() && bp_is_current_action( 'admin' ) )
    12851270        return true;
     
    12891274
    12901275function bp_is_group_forum() {
    1291     global $bp;
    1292 
    12931276    if ( bp_is_single_item() && bp_is_groups_component() && bp_is_current_action( 'forum' ) )
    12941277        return true;
     
    12981281
    12991282function bp_is_group_activity() {
    1300     global $bp;
    1301 
    13021283    if ( bp_is_single_item() && bp_is_groups_component() && bp_is_current_action( 'activity' ) )
    13031284        return true;
     
    13141295
    13151296function bp_is_group_forum_topic_edit() {
    1316     global $bp;
    1317 
    13181297    if ( bp_is_single_item() && bp_is_groups_component() && bp_is_current_action( 'forum' ) && bp_is_action_variable( 'topic', 0 ) && bp_is_action_variable( 'edit', 2 ) )
    13191298        return true;
     
    13231302
    13241303function bp_is_group_members() {
    1325     global $bp;
    1326 
    13271304    if ( bp_is_single_item() && bp_is_groups_component() && bp_is_current_action( 'members' ) )
    13281305        return true;
     
    13321309
    13331310function bp_is_group_invites() {
    1334     global $bp;
    1335 
    13361311    if ( bp_is_groups_component() && bp_is_current_action( 'send-invites' ) )
    13371312        return true;
     
    13411316
    13421317function bp_is_group_membership_request() {
    1343     global $bp;
    1344 
    13451318    if ( bp_is_groups_component() && bp_is_current_action( 'request-membership' ) )
    13461319        return true;
     
    13501323
    13511324function bp_is_group_leave() {
    1352     global $bp;
    13531325
    13541326    if ( bp_is_groups_component() && bp_is_single_item() && bp_is_current_action( 'leave-group' ) )
     
    13591331
    13601332function bp_is_group_single() {
    1361     global $bp;
    1362 
    13631333    if ( bp_is_groups_component() && bp_is_single_item() )
    13641334        return true;
     
    13681338
    13691339function bp_is_create_blog() {
    1370     global $bp;
    1371 
    1372     if ( is_multisite() && bp_is_current_component( 'blogs' ) && bp_is_current_action( 'create' ) )
     1340    if ( bp_is_blogs_component() && bp_is_current_action( 'create' ) )
    13731341        return true;
    13741342
     
    13791347
    13801348function bp_is_user_messages() {
    1381 
    1382     if ( bp_is_current_component( 'messages' ) )
     1349    if ( bp_is_user() && bp_is_messages_component() )
    13831350        return true;
    13841351
     
    13871354
    13881355function bp_is_messages_inbox() {
    1389     global $bp;
    1390 
    1391     if ( bp_is_current_component( 'messages' ) && ( !bp_current_action() || bp_is_current_action( 'inbox' ) ) )
     1356    if ( bp_is_user_messages() && ( !bp_current_action() || bp_is_current_action( 'inbox' ) ) )
    13921357        return true;
    13931358
     
    13961361
    13971362function bp_is_messages_sentbox() {
    1398     global $bp;
    1399 
    1400     if ( bp_is_current_component( 'messages' ) && bp_is_current_action( 'sentbox' ) )
     1363    if ( bp_is_user_messages() && bp_is_current_action( 'sentbox' ) )
    14011364        return true;
    14021365
     
    14051368
    14061369function bp_is_messages_compose_screen() {
    1407     global $bp;
    1408 
    1409     if ( bp_is_current_component( 'messages' ) && bp_is_current_action( 'compose' ) )
     1370    if ( bp_is_user_messages() && bp_is_current_action( 'compose' ) )
    14101371        return true;
    14111372
     
    14141375
    14151376function bp_is_notices() {
    1416     global $bp;
    1417 
    1418     if ( bp_is_current_component( 'messages' ) && bp_is_current_action( 'notices' ) )
     1377    if ( bp_is_user_messages() && bp_is_current_action( 'notices' ) )
    14191378        return true;
    14201379
     
    14241383
    14251384function bp_is_single( $component, $callback ) {
    1426     global $bp;
    1427 
    14281385    if ( bp_is_current_component( $component ) && ( true === call_user_func( $callback ) ) )
    14291386        return true;
     
    14621419}
    14631420    function bp_get_the_body_class( $wp_classes, $custom_classes = false ) {
    1464         global $bp;
    14651421
    14661422        $bp_classes = array();
     
    16191575        // We don't want WordPress blog classes to appear on non-blog pages.
    16201576        if ( !bp_is_blog_page() ) {
     1577
    16211578            // Preserve any custom classes already set
    1622             if ( !empty( $custom_classes ) )
     1579            if ( !empty( $custom_classes ) ) {
    16231580                $wp_classes = (array) $custom_classes;
    1624             else
     1581            } else {
    16251582                $wp_classes = array();
     1583            }
    16261584        }
    16271585
     
    16341592        return apply_filters( 'bp_get_the_body_class', $classes, $bp_classes, $wp_classes, $custom_classes );
    16351593    }
    1636 add_filter( 'body_class', 'bp_get_the_body_class', 10, 2 )
     1594    add_filter( 'body_class', 'bp_get_the_body_class', 10, 2 );
     1595
    16371596?>
  • trunk/bp-forums/bp-forums-template.php

    r5047 r5072  
    10571057    extract( $r, EXTR_SKIP );
    10581058
    1059     if ( empty( $topic_id ) && bp_is_current_component( 'groups') && bp_is_current_action( 'forum' ) && bp_is_action_variable( 'topic', 0 ) && bp_action_variable( 1 ) )
     1059    if ( empty( $topic_id ) && bp_is_groups_component() && bp_is_current_action( 'forum' ) && bp_is_action_variable( 'topic', 0 ) && bp_action_variable( 1 ) )
    10601060        $topic_id = bp_forums_get_topic_id_from_slug( bp_action_variable( 1 ) );
    1061     elseif ( empty( $topic_id ) && bp_is_current_component( 'forums') && bp_is_current_action( 'topic' ) && bp_action_variable( 0 ) )
     1061    elseif ( empty( $topic_id ) && bp_is_forums_component() && bp_is_current_action( 'topic' ) && bp_action_variable( 0 ) )
    10621062        $topic_id = bp_forums_get_topic_id_from_slug( bp_action_variable( 0 ) );
    10631063
  • trunk/bp-members/bp-members-screens.php

    r4961 r5072  
    3535 */
    3636function bp_members_screen_index() {
    37     if ( !bp_is_user() && bp_is_current_component( 'members' ) ) {
     37    if ( !bp_is_user() && bp_is_members_component() ) {
    3838        bp_update_is_directory( true, 'members' );
    3939
  • trunk/bp-messages/bp-messages-template.php

    r4955 r5072  
    723723    extract( $r, EXTR_SKIP );
    724724
    725     if ( !$thread_id && bp_is_current_component( 'messages' ) && bp_is_current_action( 'view' ) )
     725    if ( !$thread_id && bp_is_messages_component() && bp_is_current_action( 'view' ) )
    726726        $thread_id = (int)bp_action_variable( 0 );
    727727
  • trunk/bp-themes/bp-default/sidebar.php

    r5001 r5072  
    6060
    6161    <?php /* Show forum tags on the forums directory */
    62     if ( bp_is_active( 'forums' ) && bp_is_current_component( 'forums' ) && bp_is_directory() ) : ?>
     62    if ( bp_is_active( 'forums' ) && bp_is_forums_component() && bp_is_directory() ) : ?>
    6363        <div id="forum-directory-tags" class="widget tags">
    6464            <h3 class="widgettitle"><?php _e( 'Forum Topic Tags', 'buddypress' ) ?></h3>
  • trunk/bp-xprofile/bp-xprofile-actions.php

    r4961 r5072  
    2424    global $bp;
    2525
    26     if ( !bp_is_current_component( 'profile' ) || !bp_is_current_action( 'change-avatar' ) || !bp_is_action_variable( 'delete-avatar', 0 ) )
     26    if ( !bp_is_user_change_avatar() || !bp_is_action_variable( 'delete-avatar', 0 ) )
    2727        return false;
    2828
Note: See TracChangeset for help on using the changeset viewer.