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/classes/class-bp-groups-member.php

    r13890 r14076  
    148148
    149149        // User and group are not empty, and ID is.
    150         if ( !empty( $user_id ) && !empty( $group_id ) && empty( $id ) ) {
     150        if ( ! empty( $user_id ) && ! empty( $group_id ) && empty( $id ) ) {
    151151            $this->user_id  = $user_id;
    152152            $this->group_id = $group_id;
    153153
    154             if ( !empty( $populate ) ) {
     154            if ( ! empty( $populate ) ) {
    155155                $this->populate();
    156156            }
    … …  
    158158
    159159        // ID is not empty.
    160         if ( !empty( $id ) ) {
     160        if ( ! empty( $id ) ) {
    161161            $this->id = $id;
    162162
    163             if ( !empty( $populate ) ) {
     163            if ( ! empty( $populate ) ) {
    164164                $this->populate();
    165165            }
    … …  
    171171     *
    172172     * @since 1.6.0
     173     *
     174     * @global wpdb $wpdb WordPress database abstraction object.
    173175     */
    174176    public function populate() {
    … …  
    177179        $bp = buddypress();
    178180
    179         if ( $this->user_id && $this->group_id && !$this->id )
     181        if ( $this->user_id && $this->group_id && ! $this->id ) {
    180182            $sql = $wpdb->prepare( "SELECT * FROM {$bp->groups->table_name_members} WHERE user_id = %d AND group_id = %d", $this->user_id, $this->group_id );
    181 
    182         if ( !empty( $this->id ) )
     183        }
     184
     185        if ( ! empty( $this->id ) ) {
    183186            $sql = $wpdb->prepare( "SELECT * FROM {$bp->groups->table_name_members} WHERE id = %d", $this->id );
    184 
    185         $member = $wpdb->get_row($sql);
    186 
    187         if ( !empty( $member ) ) {
     187        }
     188
     189        $member = $wpdb->get_row( $sql );
     190
     191        if ( ! empty( $member ) ) {
    188192            $this->id            = (int) $member->id;
    189193            $this->group_id      = (int) $member->group_id;
    … …  
    210214     */
    211215    public function __get( $key ) {
    212         switch ( $key ) {
    213             case 'user' :
    214                 return $this->get_user_object( $this->user_id );
     216        if ( $key == 'user' ) {
     217            // @todo fix this.
     218            return $this->get_user_object( $this->user_id );
    215219        }
    216220    }
    … …  
    226230    public function __isset( $key ) {
    227231        switch ( $key ) {
    228             case 'user' :
     232            case 'user':
    229233                return true;
    230234
    231             default :
     235            default:
    232236                return isset( $this->{$key} );
    233237        }
    … …  
    256260     * @since 1.6.0
    257261     *
     262     * @global wpdb $wpdb WordPress database abstraction object.
     263     *
    258264     * @return bool
    259265     */
    … …  
    263269        $bp = buddypress();
    264270
    265         $this->user_id       = apply_filters( 'groups_member_user_id_before_save',       $this->user_id,      $this->id );
    266         $this->group_id      = apply_filters( 'groups_member_group_id_before_save',      $this->group_id,      $this->id );
    267         $this->inviter_id    = apply_filters( 'groups_member_inviter_id_before_save',    $this->inviter_id,    $this->id );
    268         $this->is_admin      = apply_filters( 'groups_member_is_admin_before_save',      $this->is_admin,      $this->id );
    269         $this->is_mod        = apply_filters( 'groups_member_is_mod_before_save',        $this->is_mod,        $this->id );
    270         $this->is_banned     = apply_filters( 'groups_member_is_banned_before_save',     $this->is_banned,    $this->id );
    271         $this->user_title    = apply_filters( 'groups_member_user_title_before_save',    $this->user_title,    $this->id );
     271        $this->user_id       = apply_filters( 'groups_member_user_id_before_save', $this->user_id, $this->id );
     272        $this->group_id      = apply_filters( 'groups_member_group_id_before_save', $this->group_id, $this->id );
     273        $this->inviter_id    = apply_filters( 'groups_member_inviter_id_before_save', $this->inviter_id, $this->id );
     274        $this->is_admin      = apply_filters( 'groups_member_is_admin_before_save', $this->is_admin, $this->id );
     275        $this->is_mod        = apply_filters( 'groups_member_is_mod_before_save', $this->is_mod, $this->id );
     276        $this->is_banned     = apply_filters( 'groups_member_is_banned_before_save', $this->is_banned, $this->id );
     277        $this->user_title    = apply_filters( 'groups_member_user_title_before_save', $this->user_title, $this->id );
    272278        $this->date_modified = apply_filters( 'groups_member_date_modified_before_save', $this->date_modified, $this->id );
    273         $this->is_confirmed  = apply_filters( 'groups_member_is_confirmed_before_save',  $this->is_confirmed, $this->id );
    274         $this->comments      = apply_filters( 'groups_member_comments_before_save',      $this->comments,      $this->id );
    275         $this->invite_sent   = apply_filters( 'groups_member_invite_sent_before_save',   $this->invite_sent,  $this->id );
     279        $this->is_confirmed  = apply_filters( 'groups_member_is_confirmed_before_save', $this->is_confirmed, $this->id );
     280        $this->comments      = apply_filters( 'groups_member_comments_before_save', $this->comments, $this->id );
     281        $this->invite_sent   = apply_filters( 'groups_member_invite_sent_before_save', $this->invite_sent, $this->id );
    276282
    277283        /**
    … …  
    291297        }
    292298
    293         if ( !empty( $this->id ) ) {
     299        if ( ! empty( $this->id ) ) {
    294300            $sql = $wpdb->prepare( "UPDATE {$bp->groups->table_name_members} SET inviter_id = %d, is_admin = %d, is_mod = %d, is_banned = %d, user_title = %s, date_modified = %s, is_confirmed = %d, comments = %s, invite_sent = %d WHERE id = %d", $this->inviter_id, $this->is_admin, $this->is_mod, $this->is_banned, $this->user_title, $this->date_modified, $this->is_confirmed, $this->comments, $this->invite_sent, $this->id );
    295301        } else {
    … …  
    302308        }
    303309
    304         if ( !$wpdb->query( $sql ) )
     310        if ( ! $wpdb->query( $sql ) ) {
    305311            return false;
     312        }
    306313
    307314        $this->id = $wpdb->insert_id;
    … …  
    371378     */
    372379    public function ban() {
    373         if ( !empty( $this->is_admin ) )
     380        if ( ! empty( $this->is_admin ) ) {
    374381            return false;
    375 
    376         $this->is_mod = 0;
     382        }
     383
     384        $this->is_mod    = 0;
    377385        $this->is_banned = 1;
    378386
    … …  
    388396     */
    389397    public function unban() {
    390         if ( !empty( $this->is_admin ) )
     398        if ( ! empty( $this->is_admin ) ) {
    391399            return false;
     400        }
    392401
    393402        $this->is_banned = 0;
    … …  
    413422     */
    414423    public function accept_request() {
    415         $this->is_confirmed = 1;
     424        $this->is_confirmed  = 1;
    416425        $this->date_modified = bp_core_current_time();
    417426    }
    … …  
    421430     *
    422431     * @since 1.6.0
     432     *
     433     * @global wpdb $wpdb WordPress database abstraction object.
    423434     *
    424435     * @return bool
    … …  
    439450        $sql = $wpdb->prepare( "DELETE FROM {$bp->groups->table_name_members} WHERE user_id = %d AND group_id = %d", $this->user_id, $this->group_id );
    440451
    441         if ( !$result = $wpdb->query( $sql ) )
     452        if ( ! $result = $wpdb->query( $sql ) ) {
    442453            return false;
     454        }
    443455
    444456        // Update the user's group count.
    … …  
    473485     * Refresh the `total_member_count` for a group.
    474486     *
    475      * The request skip the current cache so that we always grab the lastest total count.
     487     * The request skip the current cache so that we always grab the latest total count.
    476488     *
    477489     * @since 1.8.0
    … …  
    488500     *
    489501     * @since 1.6.0
     502     *
     503     * @global wpdb $wpdb WordPress database abstraction object.
    490504     *
    491505     * @param int $user_id  ID of the user.
    … …  
    529543     *
    530544     * @since 1.6.0
     545     *
     546     * @global wpdb $wpdb WordPress database abstraction object.
    531547     *
    532548     * @param int      $user_id ID of the user.
    … …  
    544560
    545561        $pag_sql = '';
    546         if ( !empty( $limit ) && !empty( $page ) )
    547             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
     562        if ( ! empty( $limit ) && ! empty( $page ) ) {
     563            $pag_sql = $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
     564        }
    548565
    549566        $bp = buddypress();
    … …  
    551568        // If the user is logged in and viewing their own groups, we can show hidden and private groups.
    552569        if ( $user_id != bp_loggedin_user_id() ) {
    553             $group_sql = $wpdb->prepare( "SELECT DISTINCT m.group_id FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE g.status != 'hidden' AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0{$pag_sql}", $user_id );
     570            $group_sql    = $wpdb->prepare( "SELECT DISTINCT m.group_id FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE g.status != 'hidden' AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0{$pag_sql}", $user_id );
    554571            $total_groups = $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE g.status != 'hidden' AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0", $user_id ) );
    555572        } else {
    556             $group_sql = $wpdb->prepare( "SELECT DISTINCT group_id FROM {$bp->groups->table_name_members} WHERE user_id = %d AND is_confirmed = 1 AND is_banned = 0{$pag_sql}", $user_id );
     573            $group_sql    = $wpdb->prepare( "SELECT DISTINCT group_id FROM {$bp->groups->table_name_members} WHERE user_id = %d AND is_confirmed = 1 AND is_banned = 0{$pag_sql}", $user_id );
    557574            $total_groups = $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(DISTINCT group_id) FROM {$bp->groups->table_name_members} WHERE user_id = %d AND is_confirmed = 1 AND is_banned = 0", $user_id ) );
    558575        }
    … …  
    560577        $groups = $wpdb->get_col( $group_sql );
    561578
    562         return array( 'groups' => $groups, 'total' => (int) $total_groups );
     579        return array(
     580            'groups' => $groups,
     581            'total'  => (int) $total_groups,
     582        );
    563583    }
    564584
    … …  
    567587     *
    568588     * @since 1.6.0
     589     *
     590     * @global wpdb $wpdb WordPress database abstraction object.
    569591     *
    570592     * @param int         $user_id ID of the user.
    … …  
    583605        global $wpdb;
    584606
    585         $user_id_sql = $pag_sql = $hidden_sql = $filter_sql = '';
    586 
     607        $pag_sql     = $hidden_sql = $filter_sql = '';
    587608        $user_id_sql = $wpdb->prepare( 'm.user_id = %d', $user_id );
    588609
    589         if ( !empty( $limit ) && !empty( $page ) )
    590             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
    591 
    592         if ( !empty( $filter ) ) {
     610        if ( ! empty( $limit ) && ! empty( $page ) ) {
     611            $pag_sql = $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
     612        }
     613
     614        if ( ! empty( $filter ) ) {
    593615            $search_terms_like = '%' . bp_esc_like( $filter ) . '%';
    594             $filter_sql = $wpdb->prepare( " AND ( g.name LIKE %s OR g.description LIKE %s )", $search_terms_like, $search_terms_like );
    595         }
    596 
    597         if ( $user_id != bp_loggedin_user_id() )
     616            $filter_sql        = $wpdb->prepare( ' AND ( g.name LIKE %s OR g.description LIKE %s )', $search_terms_like, $search_terms_like );
     617        }
     618
     619        if ( $user_id != bp_loggedin_user_id() ) {
    598620            $hidden_sql = " AND g.status != 'hidden'";
     621        }
    599622
    600623        $bp = buddypress();
    … …  
    603626        $total_groups = $wpdb->get_var( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id{$hidden_sql}{$filter_sql} AND {$user_id_sql} AND m.is_banned = 0 AND m.is_confirmed = 1 ORDER BY m.date_modified DESC" );
    604627
    605         return array( 'groups' => $paged_groups, 'total' => $total_groups );
     628        return array(
     629            'groups' => $paged_groups,
     630            'total'  => $total_groups,
     631        );
    606632    }
    607633
    … …  
    610636     *
    611637     * @since 1.6.0
     638     *
     639     * @global wpdb $wpdb WordPress database abstraction object.
    612640     *
    613641     * @param int         $user_id ID of the user.
    … …  
    626654        global $wpdb;
    627655
    628         $user_id_sql = $pag_sql = $hidden_sql = $filter_sql = '';
    629 
     656        $pag_sql     = $hidden_sql = $filter_sql = '';
    630657        $user_id_sql = $wpdb->prepare( 'm.user_id = %d', $user_id );
    631658
    632         if ( !empty( $limit ) && !empty( $page ) )
    633             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
    634 
    635         if ( !empty( $filter ) ) {
     659        if ( ! empty( $limit ) && ! empty( $page ) ) {
     660            $pag_sql = $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
     661        }
     662
     663        if ( ! empty( $filter ) ) {
    636664            $search_terms_like = '%' . bp_esc_like( $filter ) . '%';
    637             $filter_sql = $wpdb->prepare( " AND ( g.name LIKE %s OR g.description LIKE %s )", $search_terms_like, $search_terms_like );
    638         }
    639 
    640         if ( $user_id != bp_loggedin_user_id() )
     665            $filter_sql        = $wpdb->prepare( ' AND ( g.name LIKE %s OR g.description LIKE %s )', $search_terms_like, $search_terms_like );
     666        }
     667
     668        if ( $user_id != bp_loggedin_user_id() ) {
    641669            $hidden_sql = " AND g.status != 'hidden'";
     670        }
    642671
    643672        $bp = buddypress();
    … …  
    646675        $total_groups = $wpdb->get_var( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id{$hidden_sql}{$filter_sql} AND {$user_id_sql} AND m.is_confirmed = 1 AND m.is_banned = 0 AND m.is_admin = 1 ORDER BY date_modified ASC" );
    647676
    648         return array( 'groups' => $paged_groups, 'total' => $total_groups );
     677        return array(
     678            'groups' => $paged_groups,
     679            'total'  => $total_groups,
     680        );
    649681    }
    650682
    … …  
    653685     *
    654686     * @since 1.6.0
     687     *
     688     * @global wpdb $wpdb WordPress database abstraction object.
    655689     *
    656690     * @param int         $user_id ID of the user.
    … …  
    673707        $user_id_sql = $wpdb->prepare( 'm.user_id = %d', $user_id );
    674708
    675         if ( !empty( $limit ) && !empty( $page ) )
    676             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
    677 
    678         if ( !empty( $filter ) ) {
     709        if ( ! empty( $limit ) && ! empty( $page ) ) {
     710            $pag_sql = $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
     711        }
     712
     713        if ( ! empty( $filter ) ) {
    679714            $search_terms_like = '%' . bp_esc_like( $filter ) . '%';
    680             $filter_sql = $wpdb->prepare( " AND ( g.name LIKE %s OR g.description LIKE %s )", $search_terms_like, $search_terms_like );
    681         }
    682 
    683         if ( $user_id != bp_loggedin_user_id() )
     715            $filter_sql        = $wpdb->prepare( ' AND ( g.name LIKE %s OR g.description LIKE %s )', $search_terms_like, $search_terms_like );
     716        }
     717
     718        if ( $user_id != bp_loggedin_user_id() ) {
    684719            $hidden_sql = " AND g.status != 'hidden'";
     720        }
    685721
    686722        $bp = buddypress();
    … …  
    689725        $total_groups = $wpdb->get_var( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id{$hidden_sql}{$filter_sql} AND {$user_id_sql} AND m.is_confirmed = 1 AND m.is_banned = 0 AND m.is_mod = 1 ORDER BY date_modified ASC" );
    690726
    691         return array( 'groups' => $paged_groups, 'total' => $total_groups );
     727        return array(
     728            'groups' => $paged_groups,
     729            'total'  => $total_groups,
     730        );
    692731    }
    693732
    … …  
    696735     *
    697736     * @since 2.4.0
     737     *
     738     * @global wpdb $wpdb WordPress database abstraction object.
    698739     *
    699740     * @param int         $user_id ID of the user.
    … …  
    718759
    719760        if ( $limit && $page ) {
    720             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit ), intval( $limit ) );
     761            $pag_sql = $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
    721762        }
    722763
    723764        if ( $filter ) {
    724765            $search_terms_like = '%' . bp_esc_like( $filter ) . '%';
    725             $filter_sql        = $wpdb->prepare( " AND ( g.name LIKE %s OR g.description LIKE %s )", $search_terms_like, $search_terms_like );
     766            $filter_sql        = $wpdb->prepare( ' AND ( g.name LIKE %s OR g.description LIKE %s )', $search_terms_like, $search_terms_like );
    726767        }
    727768
    … …  
    733774        $total_groups = $wpdb->get_var( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id{$hidden_sql}{$filter_sql} AND {$user_id_sql} AND m.is_banned = 1 ORDER BY date_modified ASC" );
    734775
    735         return array( 'groups' => $paged_groups, 'total' => $total_groups );
     776        return array(
     777            'groups' => $paged_groups,
     778            'total'  => $total_groups,
     779        );
    736780    }
    737781
    … …  
    740784     *
    741785     * @since 1.6.0
     786     *
     787     * @global wpdb $wpdb WordPress database abstraction object.
    742788     *
    743789     * @param int $user_id Optional. Default: ID of the displayed user.
    … …  
    747793        global $wpdb;
    748794
    749         if ( empty( $user_id ) )
     795        if ( empty( $user_id ) ) {
    750796            $user_id = bp_displayed_user_id();
    751 
    752         $bp = buddypress();
    753 
    754         if ( $user_id != bp_loggedin_user_id() && !bp_current_user_can( 'bp_moderate' ) ) {
     797        }
     798
     799        $bp = buddypress();
     800
     801        if ( $user_id != bp_loggedin_user_id() && ! bp_current_user_can( 'bp_moderate' ) ) {
    755802            return (int) $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id AND g.status != 'hidden' AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0", $user_id ) );
    756         } else {
    757             return (int) $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0", $user_id ) );
    758         }
     803        }
     804
     805        return (int) $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(DISTINCT m.group_id) FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0", $user_id ) );
    759806    }
    760807
    … …  
    797844     * @since 4.0.0
    798845     *
    799      * @param int $user_id ID of the user.
     846     * @global wpdb $wpdb WordPress database abstraction object.
     847     *
     848     * @param int   $user_id ID of the user.
    800849     * @param array $args {
    801850     *    Array of optional arguments.
    … …  
    814863        $bp = buddypress();
    815864
    816         $r = array_merge( array(
    817             'page'     => 1,
    818             'per_page' => 20,
    819             'type'     => 'membership',
    820         ), $args );
     865        $r = array_merge(
     866            array(
     867                'page'     => 1,
     868                'per_page' => 20,
     869                'type'     => 'membership',
     870            ),
     871            $args
     872        );
    821873
    822874        $sql = array(
    … …  
    828880
    829881        switch ( $r['type'] ) {
    830             case 'pending_request' :
    831                 return groups_get_requests( array(
    832                     'user_id'  => $user_id,
    833                     'page'     => $r['page'],
    834                     'per_page' => $r['per_page'],
    835                 ) );
    836             break;
    837 
    838             case 'pending_received_invitation' :
    839                 return groups_get_invites( array(
    840                     'user_id'  => $user_id,
    841                     'page'     => $r['page'],
    842                     'per_page' => $r['per_page'],
    843                 ) );
    844             break;
    845 
    846             case 'pending_sent_invitation' :
    847                 return groups_get_invites( array(
    848                     'inviter_id'  => $user_id,
    849                     'page'        => $r['page'],
    850                     'per_page'    => $r['per_page'],
    851                 ) );
    852             break;
    853 
    854             case 'membership' :
    855             default :
    856                 $sql['where'] = $wpdb->prepare( "user_id = %d AND is_confirmed = 1", $user_id );
    857             break;
     882            case 'pending_request':
     883                return groups_get_requests(
     884                    array(
     885                        'user_id'  => $user_id,
     886                        'page'     => $r['page'],
     887                        'per_page' => $r['per_page'],
     888                    )
     889                );
     890
     891            case 'pending_received_invitation':
     892                return groups_get_invites(
     893                    array(
     894                        'user_id'  => $user_id,
     895                        'page'     => $r['page'],
     896                        'per_page' => $r['per_page'],
     897                    )
     898                );
     899
     900            case 'pending_sent_invitation':
     901                return groups_get_invites(
     902                    array(
     903                        'inviter_id' => $user_id,
     904                        'page'       => $r['page'],
     905                        'per_page'   => $r['per_page'],
     906                    )
     907                );
     908
     909            case 'membership':
     910            default:
     911                $sql['where'] = $wpdb->prepare( 'user_id = %d AND is_confirmed = 1', $user_id );
     912                break;
    858913        }
    859914
    860915        if ( $r['page'] && $r['per_page'] ) {
    861             $sql['limits'] = $wpdb->prepare( "LIMIT %d, %d", ( $r['page'] - 1 ) * $r['per_page'], $r['per_page'] );
     916            $sql['limits'] = $wpdb->prepare( 'LIMIT %d, %d', ( $r['page'] - 1 ) * $r['per_page'], $r['per_page'] );
    862917        }
    863918
    … …  
    931986     * @param int $user_id  ID of the user.
    932987     * @param int $group_id ID of the group.
    933      * @return int Number of records deleted.
     988     * @return int|false Number of records deleted. False if the user is not a member of the group.
    934989     */
    935990    public static function delete_request( $user_id, $group_id ) {
    … …  
    941996     *
    942997     * @since 1.6.0
     998     *
     999     * @global wpdb $wpdb WordPress database abstraction object.
    9431000     *
    9441001     * @param int $user_id  ID of the user.
    … …  
    9491006        global $wpdb;
    9501007
    951         if ( empty( $user_id ) )
     1008        if ( empty( $user_id ) ) {
    9521009            return false;
     1010        }
    9531011
    9541012        $bp = buddypress();
    … …  
    9611019     *
    9621020     * @since 1.6.0
     1021     *
     1022     * @global wpdb $wpdb WordPress database abstraction object.
    9631023     *
    9641024     * @param int $user_id  ID of the user.
    … …  
    9691029        global $wpdb;
    9701030
    971         if ( empty( $user_id ) )
     1031        if ( empty( $user_id ) ) {
    9721032            return false;
     1033        }
    9731034
    9741035        $bp = buddypress();
    … …  
    9811042     *
    9821043     * @since 1.6.0
     1044     *
     1045     * @global wpdb $wpdb WordPress database abstraction object.
    9831046     *
    9841047     * @param int $user_id  ID of the user.
    … …  
    9891052        global $wpdb;
    9901053
    991         if ( empty( $user_id ) )
     1054        if ( empty( $user_id ) ) {
    9921055            return false;
     1056        }
    9931057
    9941058        $bp = buddypress();
    … …  
    10011065     *
    10021066     * @since 1.6.0
     1067     *
     1068     * @global wpdb $wpdb WordPress database abstraction object.
    10031069     *
    10041070     * @param int $user_id  ID of the user.
    … …  
    10101076        global $wpdb;
    10111077
    1012         if ( empty( $user_id ) )
     1078        if ( empty( $user_id ) ) {
    10131079            return false;
     1080        }
    10141081
    10151082        $bp = buddypress();
    … …  
    10241091     *
    10251092     * @since 1.2.6
     1093     *
     1094     * @global wpdb $wpdb WordPress database abstraction object.
    10261095     *
    10271096     * @param int $user_id  ID of the user.
    … …  
    10321101        global $wpdb;
    10331102
    1034         if ( empty( $user_id ) )
     1103        if ( empty( $user_id ) ) {
    10351104            return false;
     1105        }
    10361106
    10371107        $bp = buddypress();
    … …  
    10601130     * @since 1.6.0
    10611131     *
     1132     * @global wpdb $wpdb WordPress database abstraction object.
     1133     *
    10621134     * @param int $user_id      ID of the user.
    10631135     * @param int $total_groups Max number of group IDs to return. Default: 5.
    … …  
    10721144        if ( bp_is_my_profile() ) {
    10731145            return array_map( 'intval', $wpdb->get_col( $wpdb->prepare( "SELECT DISTINCT group_id FROM {$bp->groups->table_name_members} WHERE user_id = %d AND is_confirmed = 1 AND is_banned = 0 ORDER BY rand() LIMIT %d", $user_id, $total_groups ) ) );
    1074         } else {
    1075             return array_map( 'intval', $wpdb->get_col( $wpdb->prepare( "SELECT DISTINCT m.group_id FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id AND g.status != 'hidden' AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0 ORDER BY rand() LIMIT %d", $user_id, $total_groups ) ) );
    1076         }
     1146        }
     1147
     1148        return array_map( 'intval', $wpdb->get_col( $wpdb->prepare( "SELECT DISTINCT m.group_id FROM {$bp->groups->table_name_members} m, {$bp->groups->table_name} g WHERE m.group_id = g.id AND g.status != 'hidden' AND m.user_id = %d AND m.is_confirmed = 1 AND m.is_banned = 0 ORDER BY rand() LIMIT %d", $user_id, $total_groups ) ) );
    10771149    }
    10781150
    … …  
    10811153     *
    10821154     * @since 1.6.0
     1155     *
     1156     * @global wpdb $wpdb WordPress database abstraction object.
    10831157     *
    10841158     * @param int $group_id ID of the group.
    … …  
    10981172     * @since 1.6.0
    10991173     *
    1100      * @param  int   $group_id ID of the group.
     1174     * @param  int $group_id ID of the group.
    11011175     * @return array           Info about group admins (user_id + date_modified).
    11021176     */
    … …  
    11311205     * @since 2.7.0
    11321206     *
     1207     * @global wpdb $wpdb WordPress database abstraction object.
     1208     *
    11331209     * @param array $group_ids IDs of the groups.
    1134      * @return bool
    11351210     */
    11361211    public static function prime_group_admins_mods_cache( $group_ids ) {
    … …  
    11401215
    11411216        if ( $uncached ) {
    1142             $bp = buddypress();
    1143             $uncached_sql = implode( ',', array_map( 'intval', $uncached ) );
     1217            $bp               = buddypress();
     1218            $uncached_sql     = implode( ',', array_map( 'intval', $uncached ) );
    11441219            $group_admin_mods = $wpdb->get_results( "SELECT user_id, group_id, date_modified, is_admin, is_mod FROM {$bp->groups->table_name_members} WHERE group_id IN ({$uncached_sql}) AND ( is_admin = 1 OR is_mod = 1 ) AND is_banned = 0" );
    11451220
    … …  
    11471222            if ( $group_admin_mods ) {
    11481223                foreach ( $group_admin_mods as $group_admin_mod ) {
    1149                     $obj = new stdClass();
    1150                     $obj->user_id = $group_admin_mod->user_id;
     1224                    $obj                = new stdClass();
     1225                    $obj->user_id       = $group_admin_mod->user_id;
    11511226                    $obj->date_modified = $group_admin_mod->date_modified;
    11521227
    … …  
    12091284     * @since 2.6.0
    12101285     *
     1286     * @global wpdb $wpdb WordPress database abstraction object.
     1287     *
    12111288     * @param int|string|array $membership_ids Single membership ID or comma-separated/array list of membership IDs.
    12121289     * @return array
    … …  
    12181295
    12191296        $membership_ids = implode( ',', wp_parse_id_list( $membership_ids ) );
     1297
    12201298        return $wpdb->get_results( "SELECT * FROM {$bp->groups->table_name_members} WHERE id IN ({$membership_ids})" );
    12211299    }
    … …  
    12351313    /**
    12361314     * Get members of a group.
     1315     *
     1316     * @global wpdb $wpdb WordPress database abstraction object.
    12371317     *
    12381318     * @deprecated 1.6.0
    … …  
    12521332
    12531333        $pag_sql = '';
    1254         if ( !empty( $limit ) && !empty( $page ) )
    1255             $pag_sql = $wpdb->prepare( "LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
     1334        if ( ! empty( $limit ) && ! empty( $page ) ) {
     1335            $pag_sql = $wpdb->prepare( 'LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
     1336        }
    12561337
    12571338        $exclude_admins_sql = '';
    1258         if ( !empty( $exclude_admins_mods ) )
    1259             $exclude_admins_sql = "AND is_admin = 0 AND is_mod = 0";
     1339        if ( ! empty( $exclude_admins_mods ) ) {
     1340            $exclude_admins_sql = 'AND is_admin = 0 AND is_mod = 0';
     1341        }
    12601342
    12611343        $banned_sql = '';
    1262         if ( !empty( $exclude_banned ) )
    1263             $banned_sql = " AND is_banned = 0";
     1344        if ( ! empty( $exclude_banned ) ) {
     1345            $banned_sql = ' AND is_banned = 0';
     1346        }
    12641347
    12651348        $exclude_sql = '';
    1266         if ( !empty( $exclude ) ) {
     1349        if ( ! empty( $exclude ) ) {
    12671350            $exclude     = implode( ',', wp_parse_id_list( $exclude ) );
    12681351            $exclude_sql = " AND m.user_id NOT IN ({$exclude})";
    … …  
    13061389
    13071390        // Fetch whether or not the user is a friend.
    1308         foreach ( (array) $members as $user )
     1391        foreach ( (array) $members as $user ) {
    13091392            $user_ids[] = $user->user_id;
     1393        }
    13101394
    13111395        $user_ids = implode( ',', wp_parse_id_list( $user_ids ) );
    … …  
    13151399            for ( $i = 0, $count = count( $members ); $i < $count; ++$i ) {
    13161400                foreach ( (array) $friend_status as $status ) {
    1317                     if ( $status->initiator_user_id == $members[$i]->user_id || $status->friend_user_id == $members[$i]->user_id ) {
    1318                         $members[$i]->is_friend = $status->is_confirmed;
     1401                    if ( $status->initiator_user_id == $members[ $i ]->user_id || $status->friend_user_id == $members[ $i ]->user_id ) {
     1402                        $members[ $i ]->is_friend = $status->is_confirmed;
    13191403                    }
    13201404                }
    … …  
    13221406        }
    13231407
    1324         return array( 'members' => $members, 'count' => $total_member_count );
     1408        return array(
     1409            'members' => $members,
     1410            'count'   => $total_member_count,
     1411        );
    13251412    }
    13261413
    … …  
    13291416     *
    13301417     * @since 2.6.0
     1418     *
     1419     * @global wpdb $wpdb WordPress database abstraction object.
    13311420     *
    13321421     * @param int $user_id ID of the user.
    … …  
    13471436     *
    13481437     * @since 1.6.0
     1438     *
     1439     * @global wpdb $wpdb WordPress database abstraction object.
    13491440     *
    13501441     * @param int $group_id ID of the group.
    … …  
    13731464     */
    13741465    public static function delete_all_for_user( $user_id ) {
    1375         $group_ids = BP_Groups_Member::get_group_ids( $user_id );
     1466        $group_ids = self::get_group_ids( $user_id );
    13761467
    13771468        foreach ( $group_ids['groups'] as $group_id ) {
    … …  
    13791470                // If the user is a sole group admin, install a site admin as their replacement.
    13801471                if ( count( groups_get_group_admins( $group_id ) ) < 2 ) {
    1381                     $admin = get_users( array(
    1382                         'blog_id' => bp_get_root_blog_id(),
    1383                         'fields'  => 'id',
    1384                         'number'  => 1,
    1385                         'orderby' => 'ID',
    1386                         'role'    => 'administrator',
    1387                     ) );
     1472                    $admin = get_users(
     1473                        array(
     1474                            'blog_id' => bp_get_root_blog_id(),
     1475                            'fields'  => 'id',
     1476                            'number'  => 1,
     1477                            'orderby' => 'ID',
     1478                            'role'    => 'administrator',
     1479                        )
     1480                    );
    13881481
    13891482                    if ( ! empty( $admin ) ) {
    … …  
    13961489            }
    13971490
    1398             BP_Groups_Member::delete( $user_id, $group_id );
     1491            self::delete( $user_id, $group_id );
    13991492        }
    14001493
Note: See TracChangeset for help on using the changeset viewer.