Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
06/11/2015 02:50:00 PM (11 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.