Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
11/03/2024 10:01:19 PM (20 months ago)
Author:
espellcaste
Message:

WPCS - Part I: miscellaneous fixes for some of the files of the groups component.

See #9173
See #9174

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r14070 r14076  
    2121 * @since 1.5.0
    2222 *
    23  * @return bool True if set, False if empty.
     23 * @return bool
    2424 */
    2525function bp_groups_has_directory() {
    2626    $bp = buddypress();
    2727
    28     return (bool) !empty( $bp->pages->groups->id );
     28    return ! empty( $bp->pages->groups->id );
    2929}
    3030
    … …  
    4545function groups_get_group( $group_id ) {
    4646    /*
    47      * Backward compatibilty.
     47     * Backward compatibility.
    4848     * Old-style arguments take the form of an array or a query string.
    4949     */
    … …  
    9191    }
    9292
    93     $group = groups_get_group( array( 'group_id' => (int) $group_id ) );
     93    $group = groups_get_group( (int) $group_id );
    9494
    9595    if ( empty( $group->id ) ) {
    … …  
    200200    // Pass an existing group ID.
    201201    if ( ! empty( $group_id ) ) {
    202         $group = groups_get_group( $group_id );
    203         $name  = ! empty( $name ) ? $name : $group->name;
    204         $slug  = ! empty( $slug ) ? $slug : $group->slug;
    205         $creator_id  = ! empty( $creator_id ) ? $creator_id : $group->creator_id;
    206         $description = ! empty( $description ) ? $description : $group->description;
    207         $status = ! is_null( $status ) ? $status : $group->status;
    208         $parent_id = ! is_null( $parent_id ) ? $parent_id : $group->parent_id;
     202        $group        = groups_get_group( $group_id );
     203        $name         = ! empty( $name ) ? $name : $group->name;
     204        $slug         = ! empty( $slug ) ? $slug : $group->slug;
     205        $creator_id   = ! empty( $creator_id ) ? $creator_id : $group->creator_id;
     206        $description  = ! empty( $description ) ? $description : $group->description;
     207        $status       = ! is_null( $status ) ? $status : $group->status;
     208        $parent_id    = ! is_null( $parent_id ) ? $parent_id : $group->parent_id;
    209209        $enable_forum = ! is_null( $enable_forum ) ? $enable_forum : $group->enable_forum;
    210210        $date_created = ! is_null( $date_created ) ? $date_created : $group->date_created;
    … …  
    215215        }
    216216
    217     // Create a new group.
     217        // Create a new group.
    218218    } else {
    219219        // Instantiate new group object.
    220         $group = new BP_Groups_Group;
     220        $group = new BP_Groups_Group();
    221221
    222222        // Check for null values, reset to sensible defaults.
    223         $status = ! is_null( $status ) ? $status : 'public';
    224         $parent_id = ! is_null( $parent_id ) ? $parent_id : 0;
     223        $status       = ! is_null( $status ) ? $status : 'public';
     224        $parent_id    = ! is_null( $parent_id ) ? $parent_id : 0;
    225225        $enable_forum = ! is_null( $enable_forum ) ? $enable_forum : 0;
    226226        $date_created = ! is_null( $date_created ) ? $date_created : bp_core_current_time();
    … …  
    259259    // If this is a new group, set up the creator as the first member and admin.
    260260    if ( empty( $group_id ) ) {
    261         $member                = new BP_Groups_Member;
     261        $member                = new BP_Groups_Member();
    262262        $member->group_id      = $group->id;
    263263        $member->user_id       = $group->creator_id;
    … …  
    536536    $slug = sanitize_title( $slug );
    537537
    538     if ( 'wp' == substr( $slug, 0, 2 ) )
     538    if ( 'wp' == substr( $slug, 0, 2 ) ) {
    539539        $slug = substr( $slug, 2, strlen( $slug ) - 2 );
    540 
    541     if ( in_array( $slug, (array) $bp->groups->forbidden_names ) )
     540    }
     541
     542    if ( in_array( $slug, (array) $bp->groups->forbidden_names ) ) {
    542543        $slug = $slug . '-' . rand();
     544    }
    543545
    544546    if ( BP_Groups_Group::check_slug( $slug ) ) {
    545547        do {
    546548            $slug = $slug . '-' . rand();
    547         }
    548         while ( BP_Groups_Group::check_slug( $slug ) );
     549        } while ( BP_Groups_Group::check_slug( $slug ) );
    549550    }
    550551
    … …  
    682683    // Check if the user has an outstanding request. If so, delete it.
    683684    if ( groups_check_for_membership_request( $user_id, $group_id ) ) {
    684         groups_delete_membership_request( null, $user_id, $group_id );
     685        groups_delete_membership_request( 0, $user_id, $group_id );
    685686    }
    686687
    … …  
    737738 * @param int|string|BP_Groups_Group $group The Group ID, the Group Slug or the Group object.
    738739 *                                          Default: the current group's ID.
    739  * @return bool False on failure.
    740740 */
    741741function groups_update_last_activity( $group = 0 ) {
    … …  
    744744
    745745    if ( empty( $group->id ) ) {
    746         return false;
     746        return;
    747747    }
    748748
    … …  
    868868
    869869        // Perform the group member query (extends BP_User_Query).
    870         $members = new BP_Group_Member_Query( array(
    871             'group_id'       => $r['group_id'],
    872             'per_page'       => $r['per_page'],
    873             'page'           => $r['page'],
    874             'group_role'     => $r['group_role'],
    875             'exclude'        => $r['exclude'],
    876             'search_terms'   => $r['search_terms'],
    877             'type'           => $r['type'],
    878         ) );
     870        $members = new BP_Group_Member_Query(
     871            array(
     872                'group_id'     => $r['group_id'],
     873                'per_page'     => $r['per_page'],
     874                'page'         => $r['page'],
     875                'group_role'   => $r['group_role'],
     876                'exclude'      => $r['exclude'],
     877                'search_terms' => $r['search_terms'],
     878                'type'         => $r['type'],
     879            )
     880        );
    879881
    880882        // Structure the return value as expected by the template functions.
    … …  
    959961    );
    960962
    961     $groups = BP_Groups_Group::get( array(
    962         'type'               => $r['type'],
    963         'user_id'            => $r['user_id'],
    964         'include'            => $r['include'],
    965         'exclude'            => $r['exclude'],
    966         'slug'               => $r['slug'],
    967         'parent_id'          => $r['parent_id'],
    968         'search_terms'       => $r['search_terms'],
    969         'search_columns'     => $r['search_columns'],
    970         'group_type'         => $r['group_type'],
    971         'group_type__in'     => $r['group_type__in'],
    972         'group_type__not_in' => $r['group_type__not_in'],
    973         'meta_query'         => $r['meta_query'],
    974         'date_query'         => $r['date_query'],
    975         'show_hidden'        => $r['show_hidden'],
    976         'status'             => $r['status'],
    977         'per_page'           => $r['per_page'],
    978         'page'               => $r['page'],
    979         'update_meta_cache'  => $r['update_meta_cache'],
    980         'update_admin_cache' => $r['update_admin_cache'],
    981         'order'              => $r['order'],
    982         'orderby'            => $r['orderby'],
    983         'fields'             => $r['fields'],
    984     ) );
     963    $groups = BP_Groups_Group::get(
     964        array(
     965            'type'               => $r['type'],
     966            'user_id'            => $r['user_id'],
     967            'include'            => $r['include'],
     968            'exclude'            => $r['exclude'],
     969            'slug'               => $r['slug'],
     970            'parent_id'          => $r['parent_id'],
     971            'search_terms'       => $r['search_terms'],
     972            'search_columns'     => $r['search_columns'],
     973            'group_type'         => $r['group_type'],
     974            'group_type__in'     => $r['group_type__in'],
     975            'group_type__not_in' => $r['group_type__not_in'],
     976            'meta_query'         => $r['meta_query'],
     977            'date_query'         => $r['date_query'],
     978            'show_hidden'        => $r['show_hidden'],
     979            'status'             => $r['status'],
     980            'per_page'           => $r['per_page'],
     981            'page'               => $r['page'],
     982            'update_meta_cache'  => $r['update_meta_cache'],
     983            'update_admin_cache' => $r['update_admin_cache'],
     984            'order'              => $r['order'],
     985            'orderby'            => $r['orderby'],
     986            'fields'             => $r['fields'],
     987        )
     988    );
    985989
    986990    /**
    … …  
    10561060 * @since 2.6.0
    10571061 *
    1058  * @param int $user_id ID of the user.
    1059  * @param array $args {
     1062 * @param int       $user_id ID of the user.
     1063 * @param array     $args {
    10601064 *     Array of optional args.
    1061  *     @param bool|null   $is_confirmed Whether to return only confirmed memberships. Pass `null` to disable this
    1062  *                                      filter. Default: true.
    1063  *     @param bool|null   $is_banned    Whether to return only banned memberships. Pass `null` to disable this filter.
    1064  *                                      Default: false.
    1065  *     @param bool|null   $is_admin     Whether to return only admin memberships. Pass `null` to disable this filter.
    1066  *                                      Default: false.
    1067  *     @param bool|null   $is_mod       Whether to return only mod memberships. Pass `null` to disable this filter.
    1068  *                                      Default: false.
    1069  *     @param bool|null   $invite_sent  Whether to return only memberships with 'invite_sent'. Pass `null` to disable
    1070  *                                      this filter. Default: false.
    1071  *     @param string      $orderby      Field to order by. Accepts 'id' (membership ID), 'group_id', 'date_modified'.
    1072  *                                      Default: 'group_id'.
    1073  *     @param string      $order        Sort order. Accepts 'ASC' or 'DESC'. Default: 'ASC'.
     1065 *     @param bool|null $is_confirmed Whether to return only confirmed memberships. Pass `null` to disable this
     1066 *                                    filter. Default: true.
     1067 *     @param bool|null $is_banned    Whether to return only banned memberships. Pass `null` to disable this filter.
     1068 *                                    Default: false.
     1069 *     @param bool|null $is_admin     Whether to return only admin memberships. Pass `null` to disable this filter.
     1070 *                                    Default: false.
     1071 *     @param bool|null $is_mod       Whether to return only mod memberships. Pass `null` to disable this filter.
     1072 *                                    Default: false.
     1073 *     @param bool|null $invite_sent  Whether to return only memberships with 'invite_sent'. Pass `null` to disable
     1074 *                                    this filter. Default: false.
     1075 *     @param string    $orderby      Field to order by. Accepts 'id' (membership ID), 'group_id', 'date_modified'.
     1076 *                                    Default: 'group_id'.
     1077 *     @param string    $order        Sort order. Accepts 'ASC' or 'DESC'. Default: 'ASC'.
    10741078 * }
    10751079 * @return array Array of matching group memberships, keyed by group ID.
    … …  
    11121116    $invitation_ids = array();
    11131117    if ( true !== $r['is_confirmed'] || false !== $r['invite_sent'] ) {
    1114         $invitation_ids = groups_get_invites( array(
    1115             'user_id'     => $user_id,
    1116             'invite_sent' => 'all',
    1117             'type'        => 'all',
    1118             'fields'      => 'ids'
    1119         ) );
     1118        $invitation_ids = groups_get_invites(
     1119            array(
     1120                'user_id'     => $user_id,
     1121                'invite_sent' => 'all',
     1122                'type'        => 'all',
     1123                'fields'      => 'ids',
     1124            )
     1125        );
    11201126
    11211127        // Prime the invitations cache.
    11221128        $uncached_invitation_ids = bp_get_non_cached_ids( $invitation_ids, 'bp_groups_invitations_as_memberships' );
    11231129        if ( $uncached_invitation_ids ) {
    1124             $uncached_invitations = groups_get_invites( array(
    1125                 'id'          => $uncached_invitation_ids,
    1126                 'invite_sent' => 'all',
    1127                 'type'        => 'all'
    1128             ) );
     1130            $uncached_invitations = groups_get_invites(
     1131                array(
     1132                    'id'          => $uncached_invitation_ids,
     1133                    'invite_sent' => 'all',
     1134                    'type'        => 'all',
     1135                )
     1136            );
    11291137            foreach ( $uncached_invitations as $uncached_invitation ) {
    11301138                // Reshape the result as a membership db entry.
    1131                 $invitation = new StdClass;
     1139                $invitation                = new StdClass();
    11321140                $invitation->id            = $uncached_invitation->id;
    11331141                $invitation->group_id      = $uncached_invitation->item_id;
    … …  
    12441252function groups_total_groups_for_user( $user_id = 0 ) {
    12451253
    1246     if ( empty( $user_id ) )
     1254    if ( empty( $user_id ) ) {
    12471255        $user_id = ( bp_displayed_user_id() ) ? bp_displayed_user_id() : bp_loggedin_user_id();
     1256    }
    12481257
    12491258    $count = wp_cache_get( 'bp_total_groups_for_user_' . $user_id, 'bp' );
    … …  
    13301339     * @param array $value Array of parts related to the groups avatar upload directory.
    13311340     */
    1332     return apply_filters( 'groups_avatar_upload_dir', array(
    1333         'path'    => $path,
    1334         'url'     => $newurl,
    1335         'subdir'  => $newsubdir,
    1336         'basedir' => $newbdir,
    1337         'baseurl' => $newburl,
    1338         'error'   => false
    1339     ) );
     1341    return apply_filters(
     1342        'groups_avatar_upload_dir',
     1343        array(
     1344            'path'    => $path,
     1345            'url'     => $newurl,
     1346            'subdir'  => $newsubdir,
     1347            'basedir' => $newbdir,
     1348            'baseurl' => $newburl,
     1349            'error'   => false,
     1350        )
     1351    );
    13401352}
    13411353
    … …  
    13511363function bp_groups_get_group_roles() {
    13521364    return array(
    1353         'admin' => (object) array(
     1365        'admin'  => (object) array(
    13541366            'id'           => 'admin',
    13551367            'name'         => __( 'Administrator', 'buddypress' ),
    … …  
    13601372            'is_mod'       => false,
    13611373        ),
    1362         'mod' => (object) array(
     1374        'mod'    => (object) array(
    13631375            'id'           => 'mod',
    13641376            'name'         => __( 'Moderator', 'buddypress' ),
    … …  
    14011413    $is_admin = false;
    14021414
    1403     $user_groups = bp_get_user_groups( $user_id, array(
    1404         'is_admin' => true,
    1405     ) );
     1415    $user_groups = bp_get_user_groups(
     1416        $user_id,
     1417        array(
     1418            'is_admin' => true,
     1419        )
     1420    );
    14061421
    14071422    if ( isset( $user_groups[ $group_id ] ) ) {
    … …  
    14241439    $is_mod = false;
    14251440
    1426     $user_groups = bp_get_user_groups( $user_id, array(
    1427         'is_mod' => true,
    1428     ) );
     1441    $user_groups = bp_get_user_groups(
     1442        $user_id,
     1443        array(
     1444            'is_mod' => true,
     1445        )
     1446    );
    14291447
    14301448    if ( isset( $user_groups[ $group_id ] ) ) {
    … …  
    14471465    $is_member = false;
    14481466
    1449     $user_groups = bp_get_user_groups( $user_id, array(
    1450         'is_admin' => null,
    1451         'is_mod' => null,
    1452     ) );
     1467    $user_groups = bp_get_user_groups(
     1468        $user_id,
     1469        array(
     1470            'is_admin' => null,
     1471            'is_mod'   => null,
     1472        )
     1473    );
    14531474
    14541475    if ( isset( $user_groups[ $group_id ] ) ) {
    … …  
    14711492    $is_banned = false;
    14721493
    1473     $user_groups = bp_get_user_groups( $user_id, array(
    1474         'is_confirmed' => null,
    1475         'is_banned' => true,
    1476     ) );
     1494    $user_groups = bp_get_user_groups(
     1495        $user_id,
     1496        array(
     1497            'is_confirmed' => null,
     1498            'is_banned'    => true,
     1499        )
     1500    );
    14771501
    14781502    if ( isset( $user_groups[ $group_id ] ) ) {
    … …  
    15141538    }
    15151539
    1516     $args = array(
    1517         'user_id'     => $user_id,
    1518         'item_id'     => $group_id,
     1540    $args          = array(
     1541        'user_id' => $user_id,
     1542        'item_id' => $group_id,
    15191543    );
    15201544    $invites_class = new BP_Groups_Invitation_Manager();
    … …  
    16071631 * @return array Array of group IDs.
    16081632 */
    1609  function groups_get_invited_to_group_ids( $user_id = 0 ) {
     1633function groups_get_invited_to_group_ids( $user_id = 0 ) {
    16101634    if ( empty( $user_id ) ) {
    16111635        $user_id = bp_loggedin_user_id();
    16121636    }
    16131637
    1614     $group_ids = groups_get_invites( array(
    1615         'user_id'     => $user_id,
    1616         'invite_sent' => 'sent',
    1617         'fields'      => 'item_ids'
    1618     ) );
     1638    $group_ids = groups_get_invites(
     1639        array(
     1640            'user_id'     => $user_id,
     1641            'invite_sent' => 'sent',
     1642            'fields'      => 'item_ids',
     1643        )
     1644    );
    16191645
    16201646    return array_unique( $group_ids );
    … …  
    16971723
    16981724    $invites_class = new BP_Groups_Invitation_Manager();
    1699     $success       = $invites_class->delete( array(
    1700         'user_id'    => $user_id,
    1701         'item_id'    => $group_id,
    1702         'inviter_id' => $inviter_id,
    1703     ) );
     1725    $success       = $invites_class->delete(
     1726        array(
     1727            'user_id'    => $user_id,
     1728            'item_id'    => $group_id,
     1729            'inviter_id' => $inviter_id,
     1730        )
     1731    );
    17041732
    17051733    if ( $success ) {
    … …  
    17331761function groups_accept_invite( $user_id, $group_id ) {
    17341762    $invites_class = new BP_Groups_Invitation_Manager();
    1735     $args = array(
     1763    $args          = array(
    17361764        'user_id'     => $user_id,
    17371765        'item_id'     => $group_id,
    … …  
    17601788
    17611789    $invites_class = new BP_Groups_Invitation_Manager();
    1762     $success       = $invites_class->delete( array(
    1763         'user_id'    => $user_id,
    1764         'item_id'    => $group_id,
    1765         'inviter_id' => $inviter_id,
    1766     ) );
     1790    $success       = $invites_class->delete(
     1791        array(
     1792            'user_id'    => $user_id,
     1793            'item_id'    => $group_id,
     1794            'inviter_id' => $inviter_id,
     1795        )
     1796    );
    17671797
    17681798    /**
    … …  
    17991829
    18001830    $invites_class = new BP_Groups_Invitation_Manager();
    1801     $success       = $invites_class->delete( array(
    1802         'user_id'    => $user_id,
    1803         'item_id'    => $group_id,
    1804         'inviter_id' => $inviter_id,
    1805     ) );
     1831    $success       = $invites_class->delete(
     1832        array(
     1833            'user_id'    => $user_id,
     1834            'item_id'    => $group_id,
     1835            'inviter_id' => $inviter_id,
     1836        )
     1837    );
    18061838
    18071839    /**
    … …  
    19732005 * @return int|bool ID of the first found membership if found, otherwise false.
    19742006 */
    1975  function groups_check_has_invite_from_user( $user_id, $group_id, $inviter_id = false, $type = 'sent' ) {
     2007function groups_check_has_invite_from_user( $user_id, $group_id, $inviter_id = false, $type = 'sent' ) {
    19762008    if ( empty( $user_id ) || empty( $group_id ) ) {
    19772009        return false;
    … …  
    23372369
    23382370    $invites_class = new BP_Groups_Invitation_Manager();
    2339     $request_id = $invites_class->add_request( $inv_args );
     2371    $request_id    = $invites_class->add_request( $inv_args );
    23402372
    23412373    // If a new request was created, send the emails.
    … …  
    23892421
    23902422    $invites_class = new BP_Groups_Invitation_Manager();
    2391     $args = array(
     2423    $args          = array(
    23922424        'user_id' => $user_id,
    23932425        'item_id' => $group_id,
    … …  
    24132445function groups_reject_membership_request( $membership_id, $user_id = 0, $group_id = 0 ) {
    24142446
    2415     if ( ! empty( $membership_id ) ){
     2447    if ( ! empty( $membership_id ) ) {
    24162448        /* translators: 1: the name of the method. 2: the name of the file. */
    24172449        _deprecated_argument( __METHOD__, '5.0.0', sprintf( esc_html__( 'Argument `membership_id` passed to %1$s is deprecated. See the inline documentation at %2$s for more details.', 'buddypress' ), __METHOD__, __FILE__ ) );
    … …  
    24482480 *                           requested. Provide this value along with $user_id to
    24492481 *                           override $membership_id.
    2450  * @return false|BP_Groups_Member True on success, false on failure.
     2482 * @return int|false Number of records deleted. False if the user is not a member of the group.
    24512483 */
    24522484function groups_delete_membership_request( $membership_id, $user_id = 0, $group_id = 0 ) {
    2453     if ( ! empty( $membership_id ) ){
     2485    if ( ! empty( $membership_id ) ) {
    24542486        /* translators: 1: the name of the method. 2: the name of the file. */
    24552487        _deprecated_argument( __METHOD__, '5.0.0', sprintf( esc_html__( 'Argument `membership_id` passed to %1$s is deprecated. See the inline documentation at %2$s for more details.', 'buddypress' ), __METHOD__, __FILE__ ) );
    … …  
    24612493
    24622494    $invites_class = new BP_Groups_Invitation_Manager();
    2463     $success       = $invites_class->delete_requests( array(
    2464         'user_id' => $user_id,
    2465         'item_id' => $group_id
    2466     ) );
    2467 
    2468     return $success;
     2495
     2496    return $invites_class->delete_requests(
     2497        array(
     2498            'user_id' => $user_id,
     2499            'item_id' => $group_id,
     2500        )
     2501    );
    24692502}
    24702503
    … …  
    24992532    }
    25002533
    2501     $args = array(
     2534    $args          = array(
    25022535        'user_id' => $user_id,
    25032536        'item_id' => $group_id,
    … …  
    25082541}
    25092542
    2510  /**
    2511   * Get an array of group IDs to which a user has requested membership.
    2512   *
    2513   * @since 5.0.0
    2514   *
    2515   * @param int $user_id The user ID.
    2516   *
    2517   * @return array Array of group IDs.
    2518   */
    2519  function groups_get_membership_requested_group_ids( $user_id = 0 ) {
     2543/**
     2544 * Get an array of group IDs to which a user has requested membership.
     2545 *
     2546 * @since 5.0.0
     2547 *
     2548 * @param int $user_id The user ID.
     2549 *
     2550 * @return array Array of group IDs.
     2551 */
     2552function groups_get_membership_requested_group_ids( $user_id = 0 ) {
    25202553    if ( ! $user_id ) {
    25212554        $user_id = bp_loggedin_user_id();
    25222555    }
    25232556
    2524     $group_ids     = groups_get_requests( array(
    2525         'user_id' => $user_id,
    2526         'fields'  => 'item_ids'
    2527     ) );
     2557    $group_ids = groups_get_requests(
     2558        array(
     2559            'user_id' => $user_id,
     2560            'fields'  => 'item_ids',
     2561        )
     2562    );
    25282563
    25292564    return $group_ids;
    25302565}
    25312566
    2532  /**
    2533   * Get an array of group IDs to which a user has requested membership.
    2534   *
    2535   * @since 5.0.0
    2536   *
    2537   * @param int $user_id The user ID.
    2538   *
    2539   * @return array Array of group IDs.
    2540   */
    2541  function groups_get_membership_requested_user_ids( $group_id = 0 ) {
     2567/**
     2568 * Get an array of group IDs to which a user has requested membership.
     2569 *
     2570 * @since 5.0.0
     2571 *
     2572 * @param int $group_id The group ID.
     2573 *
     2574 * @return array Array of group IDs.
     2575 */
     2576function groups_get_membership_requested_user_ids( $group_id = 0 ) {
    25422577    if ( ! $group_id ) {
    25432578        $group_id = bp_get_current_group_id();
    25442579    }
    25452580
    2546     $requests = groups_get_requests( array(
    2547         'item_id' => $group_id,
    2548         'fields'  => 'user_ids'
    2549     ) );
     2581    $requests = groups_get_requests(
     2582        array(
     2583            'item_id' => $group_id,
     2584            'fields'  => 'user_ids',
     2585        )
     2586    );
    25502587
    25512588    return $requests;
    … …  
    27202757    do_action( 'groups_remove_data_for_user', $user_id );
    27212758}
    2722 add_action( 'wpmu_delete_user',  'groups_remove_data_for_user' );
     2759add_action( 'wpmu_delete_user', 'groups_remove_data_for_user' );
    27232760add_action( 'bp_make_spam_user', 'groups_remove_data_for_user' );
    27242761
    … …  
    27492786    // Get child groups and set the parent to the deleted parent's parent.
    27502787    $grandparent_group_id = $group->parent_id;
    2751     $child_args = array(
     2788    $child_args           = array(
    27522789        'parent_id'         => $group->id,
    27532790        'show_hidden'       => true,
    … …  
    27552792        'update_meta_cache' => false,
    27562793    );
    2757     $children = groups_get_groups( $child_args );
    2758     $children = $children['groups'];
     2794    $children             = groups_get_groups( $child_args );
     2795    $children             = $children['groups'];
    27592796
    27602797    foreach ( $children as $cgroup ) {
    … …  
    27832820     * @return string The unique Group taxonomy slug.
    27842821     */
    2785     function bp_get_group_type_tax_name() {
    2786         /**
    2787         * Filters the slug of the Group type taxonomy.
    2788         *
    2789         * @since 7.0.0
    2790         *
    2791         * @param string $value Group type taxonomy slug.
    2792         */
    2793         return apply_filters( 'bp_get_group_type_tax_name', 'bp_group_type' );
    2794     }
     2822function bp_get_group_type_tax_name() {
     2823    /**
     2824    * Filters the slug of the Group type taxonomy.
     2825    *
     2826    * @since 7.0.0
     2827    *
     2828    * @param string $value Group type taxonomy slug.
     2829    */
     2830    return apply_filters( 'bp_get_group_type_tax_name', 'bp_group_type' );
     2831}
    27952832
    27962833/**
    … …  
    30493086    // Make sure the relevant labels have been filled in.
    30503087    $default_name = isset( $r['labels']['name'] ) ? $r['labels']['name'] : ucfirst( $r['name'] );
    3051     $r['labels'] = array_merge( array(
    3052         'name'          => $default_name,
    3053         'singular_name' => $default_name,
    3054     ), $r['labels'] );
     3088    $r['labels']  = array_merge(
     3089        array(
     3090            'name'          => $default_name,
     3091            'singular_name' => $default_name,
     3092        ),
     3093        $r['labels']
     3094    );
    30553095
    30563096    // Directory slug.
    … …  
    30603100            $directory_slug = $r['has_directory'];
    30613101
    3062         // Otherwise fall back on group type.
     3102            // Otherwise fall back on group type.
    30633103        } else {
    30643104            $directory_slug = $group_type;
    … …  
    33483388 * @since 2.6.0
    33493389 *
    3350  * @param int            $group_id   ID of the user.
    3351  * @param string         $group_type Group type.
     3390 * @param int    $group_id   ID of the user.
     3391 * @param string $group_type Group type.
    33523392 * @return bool|WP_Error $deleted    True on success. False or WP_Error on failure.
    33533393 */
    … …  
    34363476 * @since 2.6.0
    34373477 *
    3438  * @param  int   $group_id ID of the group.
    3439  * @return array|null $value    See {@see bp_groups_set_group_type()}.
     3478 * @param int $group_id ID of the group.
    34403479 */
    34413480function bp_remove_group_type_on_group_delete( $group_id = 0 ) {
    … …  
    34693508    }
    34703509
    3471     $memberships = BP_Groups_Member::get_user_memberships( $user->ID, array(
    3472         'type'     => 'membership',
    3473         'page'     => $page,
    3474         'per_page' => $number,
    3475     ) );
     3510    $memberships = BP_Groups_Member::get_user_memberships(
     3511        $user->ID,
     3512        array(
     3513            'type'     => 'membership',
     3514            'page'     => $page,
     3515            'per_page' => $number,
     3516        )
     3517    );
    34763518
    34773519    foreach ( $memberships as $membership ) {
    … …  
    35583600    }
    35593601
    3560     $requests = groups_get_requests( array(
    3561         'user_id'  => $user->ID,
    3562         'page'     => $page,
    3563         'per_page' => $number,
    3564     ) );
     3602    $requests = groups_get_requests(
     3603        array(
     3604            'user_id'  => $user->ID,
     3605            'page'     => $page,
     3606            'per_page' => $number,
     3607        )
     3608    );
    35653609
    35663610    foreach ( $requests as $request ) {
    … …  
    36243668    }
    36253669
    3626     $invitations = groups_get_invites( array(
    3627         'inviter_id'  => $user->ID,
    3628         'page'        => $page,
    3629         'per_page'    => $number,
    3630     ) );
     3670    $invitations = groups_get_invites(
     3671        array(
     3672            'inviter_id' => $user->ID,
     3673            'page'       => $page,
     3674            'per_page'   => $number,
     3675        )
     3676    );
    36313677
    36323678    foreach ( $invitations as $invitation ) {
    … …  
    36943740    }
    36953741
    3696     $invitations = groups_get_invites( array(
    3697         'user_id'  => $user->ID,
    3698         'page'     => $page,
    3699         'per_page' => $number,
    3700     ) );
     3742    $invitations = groups_get_invites(
     3743        array(
     3744            'user_id'  => $user->ID,
     3745            'page'     => $page,
     3746            'per_page' => $number,
     3747        )
     3748    );
    37013749
    37023750    foreach ( $invitations as $invitation ) {
    … …  
    37563804
    37573805    $processed = array();
    3758     $values = array();
     3806    $values    = array();
    37593807    foreach ( $records as $record ) {
    3760         $values[] = $wpdb->prepare(
    3761             "(%d, %d, %s, %s, %d, %d, %s, %s, %s, %d, %d)",
     3808        $values[]    = $wpdb->prepare(
     3809            '(%d, %d, %s, %s, %d, %d, %s, %s, %s, %d, %d)',
    37623810            (int) $record->user_id,
    37633811            (int) $record->inviter_id,
    … …  
    37763824
    37773825    $table_name = BP_Invitation_Manager::get_table_name();
    3778     $query = "INSERT INTO {$table_name} (user_id, inviter_id, invitee_email, class, item_id, secondary_item_id, type, content, date_modified, invite_sent, accepted) VALUES ";
    3779     $query .= implode(', ', $values );
    3780     $query .= ';';
     3826    $query      = "INSERT INTO {$table_name} (user_id, inviter_id, invitee_email, class, item_id, secondary_item_id, type, content, date_modified, invite_sent, accepted) VALUES ";
     3827    $query     .= implode( ', ', $values );
     3828    $query     .= ';';
    37813829    $wpdb->query( $query );
    37823830
    … …  
    38283876
    38293877    foreach ( array_keys( $registered_group_extensions ) as $group_extension_class ) {
    3830         $extension = new $group_extension_class;
     3878        $extension = new $group_extension_class();
    38313879
    38323880        add_action( 'bp_actions', array( $extension, '_register' ), 8 );
    … …  
    38613909 *
    38623910 * @param bool $defer True to defer, false otherwise.
    3863  * @param int $group_id The group's ID.
     3911 * @param int  $group_id The group's ID.
    38643912 */
    38653913function bp_groups_defer_group_members_count( $defer = true, $group_id = 0 ) {
    … …  
    38743922    }
    38753923
    3876     if  ( $group_id ) {
     3924    if ( $group_id ) {
    38773925        bp_groups_update_group_members_count( 0, (int) $group_id );
    38783926    }
    … …  
    41604208 * @since 12.0.0
    41614209 *
    4162  * @param array $chunks   An array of BP URL default slugs.
     4210 * @param array  $chunks   An array of BP URL default slugs.
    41634211 * @param string $context Whether to get chunks for the 'read', 'create' or 'manage' contexts.
    41644212 * @return array An associative array containing group's customized path chunks.
    … …  
    41984246            if ( is_numeric( $chunk ) ) {
    41994247                $path_chunks[ $key_action_variables ][] = $chunk;
    4200             } else {
    4201                 if ( isset( $group_screens[ $chunk ]['rewrite_id'] ) ) {
     4248            } elseif ( isset( $group_screens[ $chunk ]['rewrite_id'] ) ) {
    42024249                    $item_action_variable_rewrite_id        = $group_screens[ $chunk ]['rewrite_id'];
    42034250                    $path_chunks[ $key_action_variables ][] = bp_rewrites_get_slug( 'groups', $item_action_variable_rewrite_id, $chunk );
    4204                 } else {
    4205                     $path_chunks[ $key_action_variables ][] = $chunk;
    4206                 }
     4251            } else {
     4252                $path_chunks[ $key_action_variables ][] = $chunk;
    42074253            }
    42084254        }
Note: See TracChangeset for help on using the changeset viewer.