Skip to:
Content

BuddyPress.org

Changeset 9938


Ignore:
Timestamp:
06/11/2015 02:50:00 PM (9 years ago)
Author:
johnjamesjacoby
Message:

Groups: Bracketize bp-groups-template.php.

This change adds brackets to comparisons & loops, to improve code readability for the many, many functions and methods in this file.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-groups/bp-groups-template.php

    r9906 r9938  
    256256        $this->pag_num  = bp_sanitize_pagination_arg( 'num',          $r['per_page'] );
    257257
    258         if ( bp_current_user_can( 'bp_moderate' ) || ( is_user_logged_in() && $user_id == bp_loggedin_user_id() ) )
     258        if ( bp_current_user_can( 'bp_moderate' ) || ( is_user_logged_in() && $user_id == bp_loggedin_user_id() ) ) {
    259259            $show_hidden = true;
     260        }
    260261
    261262        if ( 'invites' == $type ) {
    … …  
    378379     */
    379380    function has_groups() {
    380         if ( $this->group_count )
     381        if ( $this->group_count ) {
    381382            return true;
     383        }
    382384
    383385        return false;
    … …  
    620622    global $groups_template;
    621623
    622     if ( bp_current_user_can( 'bp_moderate' ) )
     624    if ( bp_current_user_can( 'bp_moderate' ) ) {
    623625        return true;
    624 
    625     if ( empty( $group ) )
     626    }
     627
     628    if ( empty( $group ) ) {
    626629        $group =& $groups_template->group;
     630    }
    627631
    628632    if ( 'public' == $group->status ) {
    … …  
    656660        global $groups_template;
    657661
    658         if ( empty( $group ) )
     662        if ( empty( $group ) ) {
    659663            $group =& $groups_template->group;
     664        }
    660665
    661666        /**
    … …  
    764769        global $groups_template;
    765770
    766         if ( empty( $group ) )
     771        if ( empty( $group ) ) {
    767772            $group =& $groups_template->group;
     773        }
    768774
    769775        /**
    … …  
    798804    global $groups_template;
    799805
    800     if ( empty( $group ) )
     806    if ( empty( $group ) ) {
    801807        $group =& $groups_template->group;
     808    }
    802809
    803810    if ( 'public' == $group->status ) {
    … …  
    840847        global $groups_template;
    841848
    842         if ( empty( $group ) )
     849        if ( empty( $group ) ) {
    843850            $group =& $groups_template->group;
     851        }
    844852
    845853        /**
    … …  
    10061014        global $groups_template;
    10071015
    1008         if ( empty( $group ) )
     1016        if ( empty( $group ) ) {
    10091017            $group =& $groups_template->group;
     1018        }
    10101019
    10111020        $last_active = $group->last_activity;
    10121021
    1013         if ( !$last_active )
     1022        if ( !$last_active ) {
    10141023            $last_active = groups_get_groupmeta( $group->id, 'last_activity' );
     1024        }
    10151025
    10161026        if ( empty( $last_active ) ) {
    … …  
    10831093        global $groups_template;
    10841094
    1085         if ( empty( $group ) )
     1095        if ( empty( $group ) ) {
    10861096            $group =& $groups_template->group;
     1097        }
    10871098
    10881099        /**
    … …  
    11161127        global $groups_template;
    11171128
    1118         if ( empty( $group ) )
     1129        if ( empty( $group ) ) {
    11191130            $group =& $groups_template->group;
     1131        }
    11201132
    11211133        /**
    … …  
    11491161        global $groups_template;
    11501162
    1151         if ( empty( $group ) )
     1163        if ( empty( $group ) ) {
    11521164            $group =& $groups_template->group;
     1165        }
    11531166
    11541167        /**
    … …  
    11861199        global $groups_template;
    11871200
    1188         if ( empty( $group ) )
     1201        if ( empty( $group ) ) {
    11891202            $group =& $groups_template->group;
     1203        }
    11901204
    11911205        /**
    … …  
    12641278        global $groups_template;
    12651279
    1266         if ( empty( $group ) )
     1280        if ( empty( $group ) ) {
    12671281            $group =& $groups_template->group;
     1282        }
    12681283
    12691284        if ( $group->is_public ) {
    … …  
    12981313        global $groups_template;
    12991314
    1300         if ( empty( $group ) )
     1315        if ( empty( $group ) ) {
    13011316            $group =& $groups_template->group;
     1317        }
    13021318
    13031319        return apply_filters( 'bp_get_group_is_public', $group->is_public );
    … …  
    13241340        global $groups_template;
    13251341
    1326         if ( empty( $group ) )
     1342        if ( empty( $group ) ) {
    13271343            $group =& $groups_template->group;
     1344        }
    13281345
    13291346        /**
    … …  
    13571374        global $groups_template;
    13581375
    1359         if ( empty( $group ) )
     1376        if ( empty( $group ) ) {
    13601377            $group =& $groups_template->group;
     1378        }
    13611379
    13621380        /**
    … …  
    13901408        global $groups_template;
    13911409
    1392         if ( empty( $group ) )
     1410        if ( empty( $group ) ) {
    13931411            $group =& $groups_template->group;
     1412        }
    13941413
    13951414        /**
    … …  
    14231442        global $groups_template;
    14241443
    1425         if ( empty( $group ) )
     1444        if ( empty( $group ) ) {
    14261445            $group =& $groups_template->group;
     1446        }
    14271447
    14281448        /**
    … …  
    14481468    global $groups_template;
    14491469
    1450     if ( empty( $group ) )
     1470    if ( empty( $group ) ) {
    14511471        $group =& $groups_template->group;
    1452 
    1453     if ( empty( $user_id ) )
     1472    }
     1473
     1474    if ( empty( $user_id ) ) {
    14541475        $user_id = bp_loggedin_user_id();
     1476    }
    14551477
    14561478    return (bool) ( $group->creator_id == $user_id );
    … …  
    14911513        global $groups_template;
    14921514
    1493         if ( empty( $group ) )
     1515        if ( empty( $group ) ) {
    14941516            $group =& $groups_template->group;
     1517        }
    14951518
    14961519        $defaults = array(
    … …  
    16421665    global $groups_template;
    16431666
    1644     if ( empty( $group ) )
     1667    if ( empty( $group ) ) {
    16451668        $group =& $groups_template->group;
     1669    }
    16461670
    16471671    $admin_ids = array();
    … …  
    16531677    }
    16541678
    1655     if ( 'string' == $format )
     1679    if ( 'string' == $format ) {
    16561680        $admin_ids = implode( ',', $admin_ids );
     1681    }
    16571682
    16581683    /**
    … …  
    16821707    global $groups_template;
    16831708
    1684     if ( empty( $group ) )
     1709    if ( empty( $group ) ) {
    16851710        $group =& $groups_template->group;
     1711    }
    16861712
    16871713    $mod_ids = array();
    … …  
    16931719    }
    16941720
    1695     if ( 'string' == $format )
     1721    if ( 'string' == $format ) {
    16961722        $mod_ids = implode( ',', $mod_ids );
     1723    }
    16971724
    16981725    /**
    … …  
    17251752        global $groups_template;
    17261753
    1727         if ( empty( $group ) )
     1754        if ( empty( $group ) ) {
    17281755            $group =& $groups_template->group;
     1756        }
    17291757
    17301758        /**
    … …  
    17711799 */
    17721800function bp_group_show_no_groups_message() {
    1773     if ( !groups_total_groups_for_user( bp_displayed_user_id() ) )
     1801    if ( !groups_total_groups_for_user( bp_displayed_user_id() ) ) {
    17741802        return true;
     1803    }
    17751804
    17761805    return false;
    … …  
    17881817function bp_group_is_activity_permalink() {
    17891818
    1790     if ( !bp_is_single_item() || !bp_is_groups_component() || !bp_is_current_action( bp_get_activity_slug() ) )
     1819    if ( !bp_is_single_item() || !bp_is_groups_component() || !bp_is_current_action( bp_get_activity_slug() ) ) {
    17911820        return false;
     1821    }
    17921822
    17931823    return true;
    … …  
    18951925        global $groups_template;
    18961926
    1897         if ( empty( $group ) )
     1927        if ( empty( $group ) ) {
    18981928            $group =& $groups_template->group;
     1929        }
    18991930
    19001931        /**
    … …  
    19571988        global $groups_template;
    19581989
    1959         if ( empty( $group ) )
     1990        if ( empty( $group ) ) {
    19601991            $group =& $groups_template->group;
     1992        }
    19611993
    19621994        /**
    … …  
    19992031        extract( $r, EXTR_SKIP );
    20002032
    2001         if ( !$forum_id = groups_get_groupmeta( $groups_template->group->id, 'forum_id' ) )
     2033        if ( !$forum_id = groups_get_groupmeta( $groups_template->group->id, 'forum_id' ) ) {
    20022034            return false;
    2003 
    2004         if ( !bp_is_active( 'forums' ) )
     2035        }
     2036
     2037        if ( !bp_is_active( 'forums' ) ) {
    20052038            return false;
    2006 
    2007         if ( !$groups_template->group->forum_counts )
     2039        }
     2040
     2041        if ( !$groups_template->group->forum_counts ) {
    20082042            $groups_template->group->forum_counts = bp_forums_get_forum_topicpost_count( (int) $forum_id );
     2043        }
    20092044
    20102045        if ( (bool) $showtext ) {
    2011             if ( 1 == (int) $groups_template->group->forum_counts[0]->topics )
     2046            if ( 1 == (int) $groups_template->group->forum_counts[0]->topics ) {
    20122047                $total_topics = sprintf( __( '%d topic', 'buddypress' ), (int) $groups_template->group->forum_counts[0]->topics );
    2013             else
     2048            } else {
    20142049                $total_topics = sprintf( __( '%d topics', 'buddypress' ), (int) $groups_template->group->forum_counts[0]->topics );
     2050            }
    20152051        } else {
    20162052            $total_topics = (int) $groups_template->group->forum_counts[0]->topics;
    … …  
    20572093        extract( $r, EXTR_SKIP );
    20582094
    2059         if ( !$forum_id = groups_get_groupmeta( $groups_template->group->id, 'forum_id' ) )
     2095        if ( !$forum_id = groups_get_groupmeta( $groups_template->group->id, 'forum_id' ) ) {
    20602096            return false;
    2061 
    2062         if ( !bp_is_active( 'forums' ) )
     2097        }
     2098
     2099        if ( !bp_is_active( 'forums' ) ) {
    20632100            return false;
    2064 
    2065         if ( !$groups_template->group->forum_counts )
     2101        }
     2102
     2103        if ( !$groups_template->group->forum_counts ) {
    20662104            $groups_template->group->forum_counts = bp_forums_get_forum_topicpost_count( (int) $forum_id );
     2105        }
    20672106
    20682107        if ( (bool) $showtext ) {
    2069             if ( 1 == (int) $groups_template->group->forum_counts[0]->posts )
     2108            if ( 1 == (int) $groups_template->group->forum_counts[0]->posts ) {
    20702109                $total_posts = sprintf( __( '%d post', 'buddypress' ), (int) $groups_template->group->forum_counts[0]->posts );
    2071             else
     2110            } else {
    20722111                $total_posts = sprintf( __( '%d posts', 'buddypress' ), (int) $groups_template->group->forum_counts[0]->posts );
     2112            }
    20732113        } else {
    20742114            $total_posts = (int) $groups_template->group->forum_counts[0]->posts;
    … …  
    20962136    global $groups_template;
    20972137
    2098     if ( empty( $group ) )
     2138    if ( empty( $group ) ) {
    20992139        $group =& $groups_template->group;
    2100 
    2101     if ( ! empty( $group->enable_forum ) )
     2140    }
     2141
     2142    if ( ! empty( $group->enable_forum ) ) {
    21022143        return true;
     2144    }
    21032145
    21042146    return false;
    … …  
    21132155    global $groups_template;
    21142156
    2115     if ( empty( $group ) )
     2157    if ( empty( $group ) ) {
    21162158        $group =& $groups_template->group;
    2117 
    2118     if ( $group->enable_forum )
     2159    }
     2160
     2161    if ( $group->enable_forum ) {
    21192162        echo ' checked="checked"';
     2163    }
    21202164}
    21212165
    … …  
    21292173    global $groups_template;
    21302174
    2131     if ( empty( $group ) )
     2175    if ( empty( $group ) ) {
    21322176        $group =& $groups_template->group;
    2133 
    2134     if ( $setting == $group->status )
     2177    }
     2178
     2179    if ( $setting == $group->status ) {
    21352180        echo ' checked="checked"';
     2181    }
    21362182}
    21372183
    … …  
    21502196    $invite_status = bp_group_get_invite_status( $group_id );
    21512197
    2152     if ( $setting == $invite_status )
     2198    if ( $setting == $invite_status ) {
    21532199        echo ' checked="checked"';
     2200    }
    21542201}
    21552202
    … …  
    22962343    _deprecated_function( __FUNCTION__, '1.5', 'No longer used. See /bp-default/groups/single/admin.php' );
    22972344
    2298     if ( empty( $group ) )
     2345    if ( empty( $group ) ) {
    22992346        $group =& $groups_template->group;
     2347    }
    23002348
    23012349
    … …  
    23722420    global $groups_template;
    23732421
    2374     if ( empty( $group ) )
     2422    if ( empty( $group ) ) {
    23752423        $group =& $groups_template->group;
     2424    }
    23762425
    23772426    if ( $group_mods = groups_get_group_mods( $group->id ) ) { ?>
    … …  
    24412490    global $groups_template;
    24422491
    2443     if ( empty( $group ) )
     2492    if ( empty( $group ) ) {
    24442493        $group =& $groups_template->group;
     2494    }
    24452495
    24462496    /**
    … …  
    25412591    global $members_template;
    25422592
    2543     if ( !$user_id )
     2593    if ( !$user_id ) {
    25442594        $user_id = $members_template->member->user_id;
     2595    }
    25452596
    25462597    echo bp_get_group_member_demote_link( $user_id );
    … …  
    25582609        global $members_template, $groups_template;
    25592610
    2560         if ( empty( $group ) )
     2611        if ( empty( $group ) ) {
    25612612            $group =& $groups_template->group;
    2562 
    2563         if ( !$user_id )
     2613        }
     2614
     2615        if ( !$user_id ) {
    25642616            $user_id = $members_template->member->user_id;
     2617        }
    25652618
    25662619        /**
    … …  
    25832636    global $members_template;
    25842637
    2585     if ( !$user_id )
     2638    if ( !$user_id ) {
    25862639        $user_id = $members_template->member->user_id;
     2640    }
    25872641
    25882642    echo bp_get_group_member_ban_link( $user_id );
    … …  
    26002654        global $groups_template;
    26012655
    2602         if ( empty( $group ) )
     2656        if ( empty( $group ) ) {
    26032657            $group =& $groups_template->group;
     2658        }
    26042659
    26052660        /**
    … …  
    26222677    global $members_template;
    26232678
    2624     if ( !$user_id )
     2679    if ( !$user_id ) {
    26252680        $user_id = $members_template->member->user_id;
     2681    }
    26262682
    26272683    echo bp_get_group_member_unban_link( $user_id );
    … …  
    26392695        global $members_template, $groups_template;
    26402696
    2641         if ( !$user_id )
     2697        if ( !$user_id ) {
    26422698            $user_id = $members_template->member->user_id;
    2643 
    2644         if ( empty( $group ) )
     2699        }
     2700
     2701        if ( empty( $group ) ) {
    26452702            $group =& $groups_template->group;
     2703        }
    26462704
    26472705        /**
    … …  
    26642722    global $members_template;
    26652723
    2666     if ( !$user_id )
     2724    if ( !$user_id ) {
    26672725        $user_id = $members_template->member->user_id;
     2726    }
    26682727
    26692728    echo bp_get_group_member_remove_link( $user_id );
    … …  
    26812740        global $groups_template;
    26822741
    2683         if ( empty( $group ) )
     2742        if ( empty( $group ) ) {
    26842743            $group =& $groups_template->group;
     2744        }
    26852745
    26862746        /**
    … …  
    27952855        global $groups_template;
    27962856
    2797         if ( empty( $group ) )
     2857        if ( empty( $group ) ) {
    27982858            $group =& $groups_template->group;
     2859        }
    27992860
    28002861        /**
    … …  
    28282889        global $groups_template;
    28292890
    2830         if ( empty( $group ) )
     2891        if ( empty( $group ) ) {
    28312892            $group =& $groups_template->group;
    2832 
    2833         if ( empty( $page ) )
     2893        }
     2894
     2895        if ( empty( $page ) ) {
    28342896            $page = bp_action_variable( 0 );
     2897        }
    28352898
    28362899        /**
    … …  
    28552918    global $groups_template;
    28562919
    2857     if ( empty( $group ) )
     2920    if ( empty( $group ) ) {
    28582921        $group =& $groups_template->group;
    2859 
    2860     if ( groups_check_for_membership_request( bp_loggedin_user_id(), $group->id ) )
     2922    }
     2923
     2924    if ( groups_check_for_membership_request( bp_loggedin_user_id(), $group->id ) ) {
    28612925        return true;
     2926    }
    28622927
    28632928    return false;
    … …  
    28782943
    28792944    // Site admins always have access
    2880     if ( bp_current_user_can( 'bp_moderate' ) )
     2945    if ( bp_current_user_can( 'bp_moderate' ) ) {
    28812946        return true;
    2882 
    2883     if ( empty( $group ) )
     2947    }
     2948
     2949    if ( empty( $group ) ) {
    28842950        $group =& $groups_template->group;
     2951    }
    28852952
    28862953    /**
    … …  
    29993066        global $groups_template;
    30003067
    3001         if ( empty( $group ) )
     3068        if ( empty( $group ) ) {
    30023069            $group =& $groups_template->group;
     3070        }
    30033071
    30043072        $bp = buddypress();
    … …  
    30313099        global $groups_template;
    30323100
    3033         if ( empty( $group ) )
     3101        if ( empty( $group ) ) {
    30343102            $group =& $groups_template->group;
     3103        }
    30353104
    30363105        $bp = buddypress();
    … …  
    30633132        global $groups_template;
    30643133
    3065         if ( empty( $group ) )
     3134        if ( empty( $group ) ) {
    30663135            $group =& $groups_template->group;
     3136        }
    30673137
    30683138        /**
    … …  
    30933163        global $groups_template;
    30943164
    3095         if ( empty( $group ) )
     3165        if ( empty( $group ) ) {
    30963166            $group =& $groups_template->group;
     3167        }
    30973168
    30983169        /**
    … …  
    31233194        global $groups_template;
    31243195
    3125         if ( empty( $group ) )
     3196        if ( empty( $group ) ) {
    31263197            $group =& $groups_template->group;
     3198        }
    31273199
    31283200        /**
    … …  
    31473219    global $groups_template;
    31483220
    3149     if ( !bp_is_active( 'friends' ) )
     3221    if ( !bp_is_active( 'friends' ) ) {
    31503222        return false;
    3151 
    3152     if ( empty( $group ) )
     3223    }
     3224
     3225    if ( empty( $group ) ) {
    31533226        $group =& $groups_template->group;
    3154 
    3155     if ( !friends_check_user_has_friends( bp_loggedin_user_id() ) || !friends_count_invitable_friends( bp_loggedin_user_id(), $group->id ) )
     3227    }
     3228
     3229    if ( !friends_check_user_has_friends( bp_loggedin_user_id() ) || !friends_count_invitable_friends( bp_loggedin_user_id(), $group->id ) ) {
    31563230        return false;
     3231    }
    31573232
    31583233    return true;
    … …  
    31843259        global $groups_template;
    31853260
    3186         if ( empty( $group ) )
     3261        if ( empty( $group ) ) {
    31873262            $group =& $groups_template->group;
    3188 
    3189         if ( !is_user_logged_in() || bp_group_is_user_banned() || !bp_is_group_forum() || bp_is_group_forum_topic() )
     3263        }
     3264
     3265        if ( !is_user_logged_in() || bp_group_is_user_banned() || !bp_is_group_forum() || bp_is_group_forum_topic() ) {
    31903266            return false;
     3267        }
    31913268
    31923269        $button = array(
Note: See TracChangeset for help on using the changeset viewer.