Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
06/02/2024 02:07:21 AM (4 months ago)
Author:
espellcaste
Message:

WPCS: Part VII: miscellaneous fixes for some of the files of the core component.

Follow-up to [13883], [13886], [13887], [13888], [13891], and [13892]

See #9164 and #7228

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-core/classes/class-bp-core-user.php

    r13890 r13893  
    131131     *
    132132     * @param integer $user_id         The ID for the user being queried.
    133      * @param bool    $populate_extras Whether to fetch extra information such as
     133     * @param bool    $populate_extras Optional. Whether to fetch extra information such as
    134134     *                                 group/friendship counts or not. Default: false.
    135135     */
    136136    public function __construct( $user_id, $populate_extras = false ) {
    137         if ( !empty( $user_id ) ) {
     137        if ( ! empty( $user_id ) ) {
    138138            $this->id = $user_id;
    139139            $this->populate();
    140140
    141             if ( !empty( $populate_extras ) ) {
     141            if ( ! empty( $populate_extras ) ) {
    142142                $this->populate_extras();
    143143            }
     
    150150    public function populate() {
    151151
    152         if ( bp_is_active( 'xprofile' ) )
     152        if ( bp_is_active( 'xprofile' ) ) {
    153153            $this->profile_data = $this->get_profile_data();
    154 
    155         if ( !empty( $this->profile_data ) ) {
     154        }
     155
     156        if ( ! empty( $this->profile_data ) ) {
    156157            $full_name_field_name = bp_xprofile_fullname_field_name();
    157158
    158159            $this->user_url  = bp_members_get_user_url( $this->id );
    159             $this->fullname  = esc_attr( $this->profile_data[$full_name_field_name]['field_data'] );
     160            $this->fullname  = esc_attr( $this->profile_data[ $full_name_field_name ]['field_data'] );
    160161            $this->user_link = "<a href='{$this->user_url}'>{$this->fullname}</a>";
    161162            $this->email     = esc_attr( $this->profile_data['user_email'] );
     
    175176                    __( 'Profile photo of %s', 'buddypress' ),
    176177                    $this->fullname
    177                 )
     178                ),
    178179            )
    179180        );
     
    187188                    __( 'Profile photo of %s', 'buddypress' ),
    188189                    $this->fullname
    189                 )
     190                ),
    190191            )
    191192        );
     
    201202                ),
    202203                'width'   => 30,
    203                 'height'  => 30
     204                'height'  => 30,
    204205            )
    205206        );
     
    248249     *
    249250     * @deprecated 1.7.0 Use {@link BP_User_Query}.
     251     *
     252     * @global wpdb $wpdb WordPress database object.
    250253     *
    251254     * @see BP_User_Query for a description of parameters, most of which
     
    282285        $sql = array();
    283286
    284         $sql['select_main'] = "SELECT DISTINCT u.ID as id, u.user_registered, u.user_nicename, u.user_login, u.display_name, u.user_email";
    285 
    286         if ( 'active' == $type || 'online' == $type || 'newest' == $type ) {
    287             $sql['select_active'] = ", um.meta_value as last_activity";
    288         }
    289 
    290         if ( 'popular' == $type ) {
    291             $sql['select_popular'] = ", um.meta_value as total_friend_count";
    292         }
    293 
    294         if ( 'alphabetical' == $type ) {
    295             $sql['select_alpha'] = ", pd.value as fullname";
     287        $sql['select_main'] = 'SELECT DISTINCT u.ID as id, u.user_registered, u.user_nicename, u.user_login, u.display_name, u.user_email';
     288
     289        if ( 'active' === $type || 'online' === $type || 'newest' === $type ) {
     290            $sql['select_active'] = ', um.meta_value as last_activity';
     291        }
     292
     293        if ( 'popular' === $type ) {
     294            $sql['select_popular'] = ', um.meta_value as total_friend_count';
     295        }
     296
     297        if ( 'alphabetical' === $type ) {
     298            $sql['select_alpha'] = ', pd.value as fullname';
    296299        }
    297300
    298301        if ( $meta_key ) {
    299             $sql['select_meta'] = ", umm.meta_key";
     302            $sql['select_meta'] = ', umm.meta_key';
    300303
    301304            if ( $meta_value ) {
    302                 $sql['select_meta'] .= ", umm.meta_value";
     305                $sql['select_meta'] .= ', umm.meta_value';
    303306            }
    304307        }
     
    312315
    313316        // Alphabetical sorting is done by the xprofile Full Name field.
    314         if ( 'alphabetical' == $type ) {
     317        if ( 'alphabetical' === $type ) {
    315318            $sql['join_profiledata_alpha'] = "LEFT JOIN {$bp->profile->table_name_data} pd ON u.ID = pd.user_id";
    316319        }
     
    322325        $sql['where'] = 'WHERE ' . bp_core_get_status_sql( 'u.' );
    323326
    324         if ( 'active' == $type || 'online' == $type || 'newest' == $type ) {
    325             $sql['where_active'] = $wpdb->prepare( "AND um.meta_key = %s", bp_get_user_meta_key( 'last_activity' ) );
    326         }
    327 
    328         if ( 'popular' == $type ) {
    329             $sql['where_popular'] = $wpdb->prepare( "AND um.meta_key = %s", bp_get_user_meta_key( 'total_friend_count' ) );
    330         }
    331 
    332         if ( 'online' == $type ) {
    333             $sql['where_online'] = "AND DATE_ADD( um.meta_value, INTERVAL 5 MINUTE ) >= UTC_TIMESTAMP()";
    334         }
    335 
    336         if ( 'alphabetical' == $type ) {
    337             $sql['where_alpha'] = "AND pd.field_id = 1";
    338         }
    339 
    340         if ( !empty( $exclude ) ) {
     327        if ( 'active' === $type || 'online' === $type || 'newest' === $type ) {
     328            $sql['where_active'] = $wpdb->prepare( 'AND um.meta_key = %s', bp_get_user_meta_key( 'last_activity' ) );
     329        }
     330
     331        if ( 'popular' === $type ) {
     332            $sql['where_popular'] = $wpdb->prepare( 'AND um.meta_key = %s', bp_get_user_meta_key( 'total_friend_count' ) );
     333        }
     334
     335        if ( 'online' === $type ) {
     336            $sql['where_online'] = 'AND DATE_ADD( um.meta_value, INTERVAL 5 MINUTE ) >= UTC_TIMESTAMP()';
     337        }
     338
     339        if ( 'alphabetical' === $type ) {
     340            $sql['where_alpha'] = 'AND pd.field_id = 1';
     341        }
     342
     343        if ( ! empty( $exclude ) ) {
    341344            $exclude              = implode( ',', wp_parse_id_list( $exclude ) );
    342345            $sql['where_exclude'] = "AND u.ID NOT IN ({$exclude})";
     
    346349        // returned. The default value of false will hit the 'else' clause.
    347350        if ( 0 === $include || '0' === $include ) {
    348             $sql['where_users'] = "AND 0 = 1";
    349         } else {
    350             if ( !empty( $include ) ) {
    351                 $include = implode( ',',  wp_parse_id_list( $include ) );
     351            $sql['where_users'] = 'AND 0 = 1';
     352        } elseif ( ! empty( $include ) ) {
     353                $include            = implode( ',', wp_parse_id_list( $include ) );
    352354                $sql['where_users'] = "AND u.ID IN ({$include})";
    353             } elseif ( !empty( $user_id ) && bp_is_active( 'friends' ) ) {
    354                 $friend_ids = friends_get_friend_user_ids( $user_id );
    355 
    356                 if ( !empty( $friend_ids ) ) {
    357                     $friend_ids = implode( ',', wp_parse_id_list( $friend_ids ) );
    358                     $sql['where_friends'] = "AND u.ID IN ({$friend_ids})";
     355        } elseif ( ! empty( $user_id ) && bp_is_active( 'friends' ) ) {
     356            $friend_ids = friends_get_friend_user_ids( $user_id );
     357
     358            if ( ! empty( $friend_ids ) ) {
     359                $friend_ids          = implode( ',', wp_parse_id_list( $friend_ids ) );
     360                $sql['where_friends'] = "AND u.ID IN ({$friend_ids})";
    359361
    360362                // User has no friends, return false since there will be no users to fetch.
    361                 } else {
    362                     return false;
    363                 }
    364             }
    365         }
    366 
    367         if ( !empty( $search_terms ) && bp_is_active( 'xprofile' ) ) {
     363            } else {
     364                return false;
     365            }
     366        }
     367
     368        if ( ! empty( $search_terms ) && bp_is_active( 'xprofile' ) ) {
    368369            $search_terms_like        = '%' . bp_esc_like( $search_terms ) . '%';
    369             $sql['where_searchterms'] = $wpdb->prepare( "AND spd.value LIKE %s", $search_terms_like );
    370         }
    371 
    372         if ( !empty( $meta_key ) ) {
    373             $sql['where_meta'] = $wpdb->prepare( " AND umm.meta_key = %s", $meta_key );
     370            $sql['where_searchterms'] = $wpdb->prepare( 'AND spd.value LIKE %s', $search_terms_like );
     371        }
     372
     373        if ( ! empty( $meta_key ) ) {
     374            $sql['where_meta'] = $wpdb->prepare( ' AND umm.meta_key = %s', $meta_key );
    374375
    375376            // If a meta value is provided, match it.
    376377            if ( $meta_value ) {
    377                 $sql['where_meta'] .= $wpdb->prepare( " AND umm.meta_value = %s", $meta_value );
     378                $sql['where_meta'] .= $wpdb->prepare( ' AND umm.meta_value = %s', $meta_value );
    378379            }
    379380        }
    380381
    381382        switch ( $type ) {
    382             case 'active': case 'online': default:
    383                 $sql[] = "ORDER BY um.meta_value DESC";
     383            case 'active':
     384            case 'online':
     385            default:
     386                    $sql[] = 'ORDER BY um.meta_value DESC';
    384387                break;
    385388            case 'newest':
    386                 $sql[] = "ORDER BY u.ID DESC";
     389                $sql[] = 'ORDER BY u.ID DESC';
    387390                break;
    388391            case 'alphabetical':
    389                 $sql[] = "ORDER BY pd.value ASC";
     392                $sql[] = 'ORDER BY pd.value ASC';
    390393                break;
    391394            case 'random':
    392                 $sql[] = "ORDER BY rand()";
     395                $sql[] = 'ORDER BY rand()';
    393396                break;
    394397            case 'popular':
    395                 $sql[] = "ORDER BY CONVERT(um.meta_value, SIGNED) DESC";
     398                $sql[] = 'ORDER BY CONVERT(um.meta_value, SIGNED) DESC';
    396399                break;
    397400        }
    398401
    399         if ( !empty( $limit ) && !empty( $page ) ) {
    400             $sql['pagination'] = $wpdb->prepare( "LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
     402        if ( ! empty( $limit ) && ! empty( $page ) ) {
     403            $sql['pagination'] = $wpdb->prepare( 'LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
    401404        }
    402405
     
    415418        unset( $sql['select_main'] );
    416419
    417         if ( !empty( $sql['select_active'] ) ) {
     420        if ( ! empty( $sql['select_active'] ) ) {
    418421            unset( $sql['select_active'] );
    419422        }
    420423
    421         if ( !empty( $sql['select_popular'] ) ) {
     424        if ( ! empty( $sql['select_popular'] ) ) {
    422425            unset( $sql['select_popular'] );
    423426        }
    424427
    425         if ( !empty( $sql['select_alpha'] ) ) {
     428        if ( ! empty( $sql['select_alpha'] ) ) {
    426429            unset( $sql['select_alpha'] );
    427430        }
    428431
    429         if ( !empty( $sql['pagination'] ) ) {
     432        if ( ! empty( $sql['pagination'] ) ) {
    430433            unset( $sql['pagination'] );
    431434        }
    432435
    433         array_unshift( $sql, "SELECT COUNT(u.ID)" );
     436        array_unshift( $sql, 'SELECT COUNT(u.ID)' );
    434437
    435438        /**
     
    448451         * We can't add these to the main query above since only users who have this information will be returned (since the much of the data is in usermeta and won't support any type of directional join).
    449452         */
    450         if ( !empty( $populate_extras ) ) {
     453        if ( ! empty( $populate_extras ) ) {
    451454            $user_ids = array();
    452455
     
    456459
    457460            // Add additional data to the returned results.
    458             $paged_users = BP_Core_User::get_user_extras( $paged_users, $user_ids, $type );
    459         }
    460 
    461         return array( 'users' => $paged_users, 'total' => $total_users );
    462     }
    463 
     461            $paged_users = self::get_user_extras( $paged_users, $user_ids, $type );
     462        }
     463
     464        return array(
     465            'users' => $paged_users,
     466            'total' => $total_users,
     467        );
     468    }
    464469
    465470    /**
     
    482487        $pag_sql = '';
    483488        if ( $limit && $page ) {
    484             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
     489            $pag_sql = $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
    485490        }
    486491
    487492        // Multibyte compliance.
    488493        if ( function_exists( 'mb_strlen' ) ) {
    489             if ( mb_strlen( $letter, 'UTF-8' ) > 1 || is_numeric( $letter ) || !$letter ) {
     494            if ( mb_strlen( $letter, 'UTF-8' ) > 1 || is_numeric( $letter ) || ! $letter ) {
    490495                return false;
    491496            }
    492         } else {
    493             if ( strlen( $letter ) > 1 || is_numeric( $letter ) || !$letter ) {
    494                 return false;
    495             }
     497        } elseif ( strlen( $letter ) > 1 || is_numeric( $letter ) || ! $letter ) {
     498            return false;
    496499        }
    497500
     
    501504        $status_sql  = bp_core_get_status_sql( 'u.' );
    502505
    503         if ( !empty( $exclude ) ) {
     506        if ( ! empty( $exclude ) ) {
    504507            $exclude     = implode( ',', wp_parse_id_list( $exclude ) );
    505508            $exclude_sql = " AND u.id NOT IN ({$exclude})";
     
    524527         * @param string $value SQL prepared statement for the user query.
    525528         */
    526         $paged_users_sql = apply_filters( 'bp_core_users_by_letter_sql',       $wpdb->prepare( "SELECT DISTINCT u.ID as id, u.user_registered, u.user_nicename, u.user_login, u.user_email FROM {$wpdb->users} u LEFT JOIN {$bp->profile->table_name_data} pd ON u.ID = pd.user_id LEFT JOIN {$bp->profile->table_name_fields} pf ON pd.field_id = pf.id WHERE {$status_sql} AND pf.name = %s {$exclude_sql} AND pd.value LIKE %s ORDER BY pd.value ASC{$pag_sql}", bp_xprofile_fullname_field_name(), $letter_like ) );
     529        $paged_users_sql = apply_filters( 'bp_core_users_by_letter_sql', $wpdb->prepare( "SELECT DISTINCT u.ID as id, u.user_registered, u.user_nicename, u.user_login, u.user_email FROM {$wpdb->users} u LEFT JOIN {$bp->profile->table_name_data} pd ON u.ID = pd.user_id LEFT JOIN {$bp->profile->table_name_fields} pf ON pd.field_id = pf.id WHERE {$status_sql} AND pf.name = %s {$exclude_sql} AND pd.value LIKE %s ORDER BY pd.value ASC{$pag_sql}", bp_xprofile_fullname_field_name(), $letter_like ) );
    527530
    528531        $total_users = $wpdb->get_var( $total_users_sql );
     
    537540         */
    538541        $user_ids = array();
    539         foreach ( (array) $paged_users as $user )
     542        foreach ( (array) $paged_users as $user ) {
    540543            $user_ids[] = (int) $user->id;
     544        }
    541545
    542546        // Add additional data to the returned results.
    543547        if ( $populate_extras ) {
    544             $paged_users = BP_Core_User::get_user_extras( $paged_users, $user_ids );
    545         }
    546 
    547         return array( 'users' => $paged_users, 'total' => $total_users );
     548            $paged_users = self::get_user_extras( $paged_users, $user_ids );
     549        }
     550
     551        return array(
     552            'users' => $paged_users,
     553            'total' => $total_users,
     554        );
    548555    }
    549556
     
    566573
    567574        $pag_sql = '';
    568         if ( $limit && $page )
    569             $pag_sql = $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * $limit), intval( $limit ) );
     575        if ( $limit && $page ) {
     576            $pag_sql = $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * $limit ), intval( $limit ) );
     577        }
    570578
    571579        $user_ids   = implode( ',', wp_parse_id_list( $user_ids ) );
     
    638646
    639647        // Add additional data to the returned results.
    640         if ( !empty( $populate_extras ) ) {
    641             $paged_users = BP_Core_User::get_user_extras( $paged_users, $user_ids );
    642         }
    643 
    644         return array( 'users' => $paged_users, 'total' => $total_users );
     648        if ( ! empty( $populate_extras ) ) {
     649            $paged_users = self::get_user_extras( $paged_users, $user_ids );
     650        }
     651
     652        return array(
     653            'users' => $paged_users,
     654            'total' => $total_users,
     655        );
    645656    }
    646657
     
    663674
    664675        $user_ids = array();
    665         $pag_sql  = $limit && $page ? $wpdb->prepare( " LIMIT %d, %d", intval( ( $page - 1 ) * intval( $limit ) ), intval( $limit ) ) : '';
     676        $pag_sql  = $limit && $page ? $wpdb->prepare( ' LIMIT %d, %d', intval( ( $page - 1 ) * intval( $limit ) ), intval( $limit ) ) : '';
    666677
    667678        $search_terms_like = '%' . bp_esc_like( $search_terms ) . '%';
     
    684695         * @param string $value SQL statement for the searched users query.
    685696         */
    686         $paged_users_sql = apply_filters( 'bp_core_search_users_sql',       $wpdb->prepare( "SELECT DISTINCT u.ID as id, u.user_registered, u.user_nicename, u.user_login, u.user_email FROM {$wpdb->users} u LEFT JOIN {$bp->profile->table_name_data} pd ON u.ID = pd.user_id WHERE {$status_sql} AND pd.value LIKE %s ORDER BY pd.value ASC{$pag_sql}", $search_terms_like ), $search_terms, $pag_sql );
     697        $paged_users_sql = apply_filters( 'bp_core_search_users_sql', $wpdb->prepare( "SELECT DISTINCT u.ID as id, u.user_registered, u.user_nicename, u.user_login, u.user_email FROM {$wpdb->users} u LEFT JOIN {$bp->profile->table_name_data} pd ON u.ID = pd.user_id WHERE {$status_sql} AND pd.value LIKE %s ORDER BY pd.value ASC{$pag_sql}", $search_terms_like ), $search_terms, $pag_sql );
    687698
    688699        $total_users = $wpdb->get_var( $total_users_sql );
     
    693704         * We can't add these to the main query above since only users who have this information will be returned (since the much of the data is in usermeta and won't support any type of directional join)
    694705         */
    695         foreach ( (array) $paged_users as $user )
     706        foreach ( (array) $paged_users as $user ) {
    696707            $user_ids[] = $user->id;
     708        }
    697709
    698710        // Add additional data to the returned results.
    699         if ( $populate_extras )
    700             $paged_users = BP_Core_User::get_user_extras( $paged_users, $user_ids );
    701 
    702         return array( 'users' => $paged_users, 'total' => $total_users );
     711        if ( $populate_extras ) {
     712            $paged_users = self::get_user_extras( $paged_users, $user_ids );
     713        }
     714
     715        return array(
     716            'users' => $paged_users,
     717            'total' => $total_users,
     718        );
    703719    }
    704720
     
    720736        $bp = buddypress();
    721737
    722         if ( empty( $user_ids ) )
     738        if ( empty( $user_ids ) ) {
    723739            return $paged_users;
     740        }
    724741
    725742        // Sanitize user IDs.
     
    727744
    728745        // Fetch the user's full name.
    729         if ( bp_is_active( 'xprofile' ) && 'alphabetical' != $type ) {
     746        if ( bp_is_active( 'xprofile' ) && 'alphabetical' !== $type ) {
    730747            $names = $wpdb->get_results( $wpdb->prepare( "SELECT pd.user_id as id, pd.value as fullname FROM {$bp->profile->table_name_fields} pf, {$bp->profile->table_name_data} pd WHERE pf.id = pd.field_id AND pf.name = %s AND pd.user_id IN ( {$user_ids} )", bp_xprofile_fullname_field_name() ) );
    731748            for ( $i = 0, $count = count( $paged_users ); $i < $count; ++$i ) {
    732749                foreach ( (array) $names as $name ) {
    733                     if ( $name->id == $paged_users[$i]->id )
    734                         $paged_users[$i]->fullname = $name->fullname;
     750                    if ( $name->id === $paged_users[ $i ]->id ) {
     751                        $paged_users[ $i ]->fullname = $name->fullname;
     752                    }
    735753                }
    736754            }
     
    738756
    739757        // Fetch the user's total friend count.
    740         if ( 'popular' != $type ) {
     758        if ( 'popular' !== $type ) {
    741759            $friend_count = $wpdb->get_results( $wpdb->prepare( "SELECT user_id as id, meta_value as total_friend_count FROM {$wpdb->usermeta} WHERE meta_key = %s AND user_id IN ( {$user_ids} )", bp_get_user_meta_key( 'total_friend_count' ) ) );
    742760            for ( $i = 0, $count = count( $paged_users ); $i < $count; ++$i ) {
    743761                foreach ( (array) $friend_count as $fcount ) {
    744                     if ( $fcount->id == $paged_users[$i]->id )
    745                         $paged_users[$i]->total_friend_count = (int) $fcount->total_friend_count;
     762                    if ( $fcount->id === $paged_users[ $i ]->id ) {
     763                        $paged_users[ $i ]->total_friend_count = (int) $fcount->total_friend_count;
     764                    }
    746765                }
    747766            }
     
    753772            for ( $i = 0, $count = count( $paged_users ); $i < $count; ++$i ) {
    754773                foreach ( (array) $friend_status as $status ) {
    755                     if ( $status->initiator_user_id == $paged_users[$i]->id || $status->friend_user_id == $paged_users[$i]->id )
    756                         $paged_users[$i]->is_friend = $status->is_confirmed;
     774                    if ( $status->initiator_user_id === $paged_users[ $i ]->id || $status->friend_user_id === $paged_users[ $i ]->id ) {
     775                        $paged_users[ $i ]->is_friend = $status->is_confirmed;
     776                    }
    757777                }
    758778            }
     
    760780
    761781        // Fetch the user's last_activity.
    762         if ( 'active' != $type ) {
     782        if ( 'active' !== $type ) {
    763783            $user_activity = self::get_last_activity( $user_ids );
    764784            for ( $i = 0, $count = count( $paged_users ); $i < $count; ++$i ) {
    765785                foreach ( (array) $user_activity as $activity ) {
    766                     if ( ! empty( $activity['user_id'] ) && (int) $activity['user_id'] === (int) $paged_users[$i]->id ) {
    767                         $paged_users[$i]->last_activity = $activity['date_recorded'];
     786                    if ( ! empty( $activity['user_id'] ) && (int) $activity['user_id'] === (int) $paged_users[ $i ]->id ) {
     787                        $paged_users[ $i ]->last_activity = $activity['date_recorded'];
    768788                    }
    769789                }
     
    775795        for ( $i = 0, $count = count( $paged_users ); $i < $count; ++$i ) {
    776796            foreach ( (array) $user_update as $update ) {
    777                 if ( $update->id == $paged_users[$i]->id )
    778                     $paged_users[$i]->latest_update = $update->latest_update;
     797                if ( $update->id === $paged_users[ $i ]->id ) {
     798                    $paged_users[ $i ]->latest_update = $update->latest_update;
     799                }
    779800            }
    780801        }
     
    789810     *
    790811     * @param int $user_id User ID.
    791      * @return false|object WP_User if successful, false on failure.
     812     * @return false|object User object if successful, false on failure.
    792813     */
    793814    public static function get_core_userdata( $user_id ) {
     
    797818    /**
    798819     * Get last activity data for a user or set of users.
     820     *
     821     * @global wpdb $wpdb WordPress database object.
    799822     *
    800823     * @param int|array $user_id User IDs or multiple user IDs.
     
    821844
    822845            foreach ( $last_activities as $last_activity ) {
    823                 wp_cache_set( $last_activity->user_id, array(
    824                     'user_id'       => $last_activity->user_id,
    825                     'date_recorded' => $last_activity->date_recorded,
    826                     'activity_id'   => $last_activity->id,
    827                 ), 'bp_last_activity' );
     846                wp_cache_set(
     847                    $last_activity->user_id,
     848                    array(
     849                        'user_id'       => $last_activity->user_id,
     850                        'date_recorded' => $last_activity->date_recorded,
     851                        'activity_id'   => $last_activity->id,
     852                    ),
     853                    'bp_last_activity'
     854                );
    828855            }
    829856        }
     
    835862
    836863            if ( isset( $retval['user_id'] ) ) {
    837                 $retval[ $user_id ]['user_id']     = (int) $retval[ $user_id ]['user_id'];
     864                $retval[ $user_id ]['user_id'] = (int) $retval[ $user_id ]['user_id'];
    838865            }
    839866            if ( isset( $retval['activity_id'] ) ) {
     
    852879     *
    853880     * @since 2.0.0
     881     *
     882     * @global wpdb $wpdb WordPress database object.
    854883     *
    855884     * @param int    $user_id ID of the user whose last_activity you are updating.
     
    867896            $updated = $wpdb->update(
    868897                $table_name,
    869 
    870898                // Data to update.
    871899                array(
    872900                    'date_recorded' => $time,
    873901                ),
    874 
    875902                // WHERE.
    876903                array(
    877904                    'id' => $activity[ $user_id ]['activity_id'],
    878905                ),
    879 
    880906                // Data sanitization format.
    881907                array(
    882908                    '%s',
    883909                ),
    884 
    885910                // WHERE sanitization format.
    886911                array(
     
    895920            $updated = $wpdb->insert(
    896921                $table_name,
    897 
    898922                // Data.
    899923                array(
     
    907931                    'date_recorded' => $time,
    908932                ),
    909 
    910933                // Data sanitization format.
    911934                array(
     
    923946            // Set up activity array for caching.
    924947            // View the foreach loop in the get_last_activity() method for format.
    925             $activity = array();
     948            $activity             = array();
    926949            $activity[ $user_id ] = array(
    927950                'user_id'       => $user_id,
     
    952975     * @since 2.0.0
    953976     *
     977     * @global wpdb $wpdb WordPress database object.
     978     *
    954979     * @param int $user_id ID of the user whose activity should be deleted.
    955980     * @return bool
     
    966991        $deleted = $wpdb->delete(
    967992            buddypress()->members->table_name_last_activity,
    968 
    969993            // WHERE.
    970994            array(
    971995                'id' => $existing[ $user_id ]['activity_id'],
    972996            ),
    973 
    974997            // WHERE sanitization format.
    975998            array(
Note: See TracChangeset for help on using the changeset viewer.