Skip to:
Content

BuddyPress.org

Changeset 7308


Ignore:
Timestamp:
07/25/2013 02:28:28 AM (11 years ago)
Author:
johnjamesjacoby
Message:

Fix some PHP 5.4 static method E_STRICT warnings across several components. More to do here. See #5108.

Location:
trunk
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/bp-activity/bp-activity-classes.php

    r7228 r7308  
    2626    var $is_spam;
    2727
    28     function __construct( $id = false ) {
     28    public function __construct( $id = false ) {
    2929        if ( !empty( $id ) ) {
    3030            $this->id = $id;
    … …  
    3333    }
    3434
    35     function populate() {
     35    public function populate() {
    3636        global $wpdb, $bp;
    3737
    … …  
    5656    }
    5757
    58     function save() {
    59         global $wpdb, $bp, $current_user;
     58    public function save() {
     59        global $wpdb, $bp;
    6060
    6161        $this->id                = apply_filters_ref_array( 'bp_activity_id_before_save',                array( $this->id,                &$this ) );
    … …  
    113113     * @return array
    114114     */
    115     function get( $args = array() ) {
     115    public static function get( $args = array() ) {
    116116        global $wpdb, $bp;
    117117
    … …  
    350350     * @since BuddyPress (1.2)
    351351     */
    352     function get_specific( $activity_ids, $max = false, $page = 1, $per_page = 25, $sort = 'DESC', $display_comments = false ) {
     352    public static function get_specific( $activity_ids, $max = false, $page = 1, $per_page = 25, $sort = 'DESC', $display_comments = false ) {
    353353        _deprecated_function( __FUNCTION__, '1.5', 'Use BP_Activity_Activity::get() with the "in" parameter instead.' );
    354354        return BP_Activity_Activity::get( $max, $page, $per_page, $sort, false, false, $display_comments, false, false, $activity_ids );
    355355    }
    356356
    357     function get_id( $user_id, $component, $type, $item_id, $secondary_item_id, $action, $content, $date_recorded ) {
     357    public static function get_id( $user_id, $component, $type, $item_id, $secondary_item_id, $action, $content, $date_recorded ) {
    358358        global $bp, $wpdb;
    359359
    … …  
    392392    }
    393393
    394     function delete( $args ) {
     394    public static function delete( $args = array() ) {
    395395        global $wpdb, $bp;
    396396
    … …  
    467467    }
    468468
    469     function delete_activity_item_comments( $activity_ids = array() ) {
     469    public static function delete_activity_item_comments( $activity_ids = array() ) {
    470470        global $bp, $wpdb;
    471471
    … …  
    475475    }
    476476
    477     function delete_activity_meta_entries( $activity_ids = array() ) {
     477    public static function delete_activity_meta_entries( $activity_ids = array() ) {
    478478        global $bp, $wpdb;
    479479
    … …  
    496496     * @since BuddyPress (1.2)
    497497     */
    498     function append_comments( $activities, $spam = 'ham_only' ) {
    499         global $wpdb;
    500 
     498    public static function append_comments( $activities, $spam = 'ham_only' ) {
    501499        $activity_comments = array();
    502500
    … …  
    528526     * @since BuddyPress (1.2)
    529527     */
    530     function get_activity_comments( $activity_id, $left, $right, $spam = 'ham_only', $top_level_parent_id = 0 ) {
     528    public static function get_activity_comments( $activity_id, $left, $right, $spam = 'ham_only', $top_level_parent_id = 0 ) {
    531529        global $wpdb, $bp;
    532530
    … …  
    585583    }
    586584
    587     function rebuild_activity_comment_tree( $parent_id, $left = 1 ) {
     585    public static function rebuild_activity_comment_tree( $parent_id, $left = 1 ) {
    588586        global $wpdb, $bp;
    589587
    … …  
    609607    }
    610608
    611     function get_child_comments( $parent_id ) {
     609    public static function get_child_comments( $parent_id ) {
    612610        global $bp, $wpdb;
    613611
    … …  
    620618     * @return array
    621619     */
    622     function get_recorded_components() {
     620    public static function get_recorded_components() {
    623621        global $wpdb, $bp;
    624622        return $wpdb->get_col( "SELECT DISTINCT component FROM {$bp->activity->table_name} ORDER BY component ASC" );
    625623    }
    626624
    627     function get_sitewide_items_for_feed( $limit = 35 ) {
    628         global $bp;
    629 
     625    public static function get_sitewide_items_for_feed( $limit = 35 ) {
    630626        $activities    = bp_activity_get_sitewide( array( 'max' => $limit ) );
    631627        $activity_feed = array();
    … …  
    642638    }
    643639
    644     function get_in_operator_sql( $field, $items ) {
     640    public static function get_in_operator_sql( $field, $items ) {
    645641        global $wpdb;
    646642
    … …  
    666662    }
    667663
    668     function get_filter_sql( $filter_array ) {
     664    public static function get_filter_sql( $filter_array ) {
    669665
    670666        $filter_sql = array();
    … …  
    706702    }
    707703
    708     function get_last_updated() {
     704    public static function get_last_updated() {
    709705        global $bp, $wpdb;
    710706
    … …  
    712708    }
    713709
    714     function total_favorite_count( $user_id ) {
     710    public static function total_favorite_count( $user_id ) {
    715711        if ( !$favorite_activity_entries = bp_get_user_meta( $user_id, 'bp_favorite_activities', true ) )
    716712            return 0;
    … …  
    719715    }
    720716
    721     function check_exists_by_content( $content ) {
     717    public static function check_exists_by_content( $content ) {
    722718        global $wpdb, $bp;
    723719
    … …  
    725721    }
    726722
    727     function hide_all_for_user( $user_id ) {
     723    public static function hide_all_for_user( $user_id ) {
    728724        global $wpdb, $bp;
    729725

  • trunk/bp-core/bp-core-classes.php

    r7298 r7308  
    626626     * @var integer
    627627     */
    628     var $id;
     628    public $id;
    629629
    630630    /**
    … …  
    633633     * @var string
    634634     */
    635     var $avatar;
     635    public $avatar;
    636636
    637637    /**
    … …  
    640640     * @var string
    641641     */
    642     var $avatar_thumb;
     642    public $avatar_thumb;
    643643
    644644    /**
    … …  
    647647     * @var string
    648648     */
    649     var $avatar_mini;
     649    public $avatar_mini;
    650650
    651651    /**
    … …  
    654654     * @var string
    655655     */
    656     var $fullname;
     656    public $fullname;
    657657
    658658    /**
    … …  
    661661     * @var string
    662662     */
    663     var $email;
     663    public $email;
    664664
    665665    /**
    … …  
    668668     * @var string
    669669     */
    670     var $user_url;
     670    public $user_url;
    671671
    672672    /**
    … …  
    675675     * @var string
    676676     */
    677     var $user_link;
     677    public $user_link;
    678678
    679679    /**
    … …  
    684684     * @var string
    685685     */
    686     var $last_active;
     686    public $last_active;
    687687
    688688    /* Extras */
    … …  
    693693     * @var integer
    694694     */
    695     var $total_friends;
     695    public $total_friends;
    696696
    697697    /**
    … …  
    701701     * @deprecated No longer used
    702702     */
    703     var $total_blogs;
     703    public $total_blogs;
    704704
    705705    /**
    … …  
    710710     * @var string
    711711     */
    712     var $total_groups;
     712    public $total_groups;
    713713
    714714    /**
    … …  
    728728     * @param boolean $populate_extras Whether to fetch extra information such as group/friendship counts or not.
    729729     */
    730     function __construct( $user_id, $populate_extras = false ) {
     730    public function __construct( $user_id, $populate_extras = false ) {
    731731        if ( !empty( $user_id ) ) {
    732732            $this->id = $user_id;
    … …  
    751751     * @uses bp_profile_last_updated_date() Returns the last updated date for a user.
    752752     */
    753     function populate() {
     753    private function populate() {
    754754
    755755        if ( bp_is_active( 'xprofile' ) )
    … …  
    784784     * Populates extra fields such as group and friendship counts.
    785785     */
    786     function populate_extras() {
     786    private function populate_extras() {
    787787
    788788        if ( bp_is_active( 'friends' ) ) {
    … …  
    796796    }
    797797
    798     function get_profile_data() {
     798    private function get_profile_data() {
    799799        return BP_XProfile_ProfileData::get_all_for_user( $this->id );
    800800    }
    … …  
    802802    /** Static Methods ********************************************************/
    803803
    804     function get_users( $type, $limit = 0, $page = 1, $user_id = 0, $include = false, $search_terms = false, $populate_extras = true, $exclude = false, $meta_key = false, $meta_value = false ) {
     804    public static function get_users( $type, $limit = 0, $page = 1, $user_id = 0, $include = false, $search_terms = false, $populate_extras = true, $exclude = false, $meta_key = false, $meta_value = false ) {
    805805        global $wpdb, $bp;
    806806
    … …  
    989989     * @static
    990990     */
    991     function get_users_by_letter( $letter, $limit = null, $page = 1, $populate_extras = true, $exclude = '' ) {
     991    public static function get_users_by_letter( $letter, $limit = null, $page = 1, $populate_extras = true, $exclude = '' ) {
    992992        global $bp, $wpdb;
    993993
    … …  
    10541054     * @static
    10551055     */
    1056     function get_specific_users( $user_ids, $limit = null, $page = 1, $populate_extras = true ) {
     1056    public static function get_specific_users( $user_ids, $limit = null, $page = 1, $populate_extras = true ) {
    10571057        global $wpdb;
    10581058
    … …  
    10981098     * @static
    10991099     */
    1100     function search_users( $search_terms, $limit = null, $page = 1, $populate_extras = true ) {
     1100    public static function search_users( $search_terms, $limit = null, $page = 1, $populate_extras = true ) {
    11011101        global $bp, $wpdb;
    11021102
    … …  
    11401140     * @static
    11411141     */
    1142     function get_user_extras( &$paged_users, &$user_ids, $type = false ) {
     1142    public static function get_user_extras( &$paged_users, &$user_ids, $type = false ) {
    11431143        global $bp, $wpdb;
    11441144
    … …  
    12231223     * @static
    12241224     */
    1225     function get_core_userdata( $user_id ) {
     1225    public static function get_core_userdata( $user_id ) {
    12261226        global $wpdb;
    12271227
    … …  
    12481248     * @var integer
    12491249     */
    1250     var $id;
     1250    public $id;
    12511251
    12521252    /**
    … …  
    12551255     * @var integer
    12561256     */
    1257     var $item_id;
     1257    public $item_id;
    12581258
    12591259    /**
    … …  
    12621262     * @var integer
    12631263     */
    1264     var $secondary_item_id = null;
     1264    public $secondary_item_id = null;
    12651265
    12661266    /**
    … …  
    12691269     * @var integer
    12701270     */
    1271     var $user_id;
     1271    public $user_id;
    12721272
    12731273    /**
    … …  
    12761276     * @var string
    12771277     */
    1278     var $component_name;
     1278    public $component_name;
    12791279
    12801280    /**
    … …  
    12831283     * @var string
    12841284     */
    1285     var $component_action;
     1285    public $component_action;
    12861286
    12871287    /**
    … …  
    12901290     * @var string
    12911291     */
    1292     var $date_notified;
     1292    public $date_notified;
    12931293
    12941294    /**
    … …  
    12971297     * @var boolean
    12981298     */
    1299     var $is_new;
     1299    public $is_new;
    13001300
    13011301    /** Public Methods ********************************************************/
    … …  
    13061306     * @param integer $id
    13071307     */
    1308     function __construct( $id = 0 ) {
     1308    public function __construct( $id = 0 ) {
    13091309        if ( !empty( $id ) ) {
    13101310            $this->id = $id;
    … …  
    13201320     * @return bool Success or failure
    13211321     */
    1322     function save() {
     1322    public function save() {
    13231323        global $bp, $wpdb;
    13241324
    … …  
    13481348     * @global wpdb $wpdb WordPress database object
    13491349     */
    1350     function populate() {
     1350    private function populate() {
    13511351        global $bp, $wpdb;
    13521352
    … …  
    13641364    /** Static Methods ********************************************************/
    13651365
    1366     function check_access( $user_id, $notification_id ) {
     1366    public static function check_access( $user_id, $notification_id ) {
    13671367        global $wpdb, $bp;
    13681368
    … …  
    13801380     * @static
    13811381     */
    1382     function get_all_for_user( $user_id, $status = 'is_new' ) {
     1382    public static function get_all_for_user( $user_id, $status = 'is_new' ) {
    13831383        global $bp, $wpdb;
    13841384
    … …  
    13981398     * @static
    13991399     */
    1400     function delete_for_user_by_type( $user_id, $component_name, $component_action ) {
     1400    public static function delete_for_user_by_type( $user_id, $component_name, $component_action ) {
    14011401        global $bp, $wpdb;
    14021402
    … …  
    14161416     * @static
    14171417     */
    1418     function delete_for_user_by_item_id( $user_id, $item_id, $component_name, $component_action, $secondary_item_id = false ) {
     1418    public static function delete_for_user_by_item_id( $user_id, $item_id, $component_name, $component_action, $secondary_item_id = false ) {
    14191419        global $bp, $wpdb;
    14201420
    … …  
    14341434     * @static
    14351435     */
    1436     function delete_from_user_by_type( $user_id, $component_name, $component_action ) {
     1436    public static function delete_from_user_by_type( $user_id, $component_name, $component_action ) {
    14371437        global $bp, $wpdb;
    14381438
    … …  
    14511451     * @static
    14521452     */
    1453     function delete_all_by_type( $item_id, $component_name, $component_action, $secondary_item_id ) {
     1453    public static function delete_all_by_type( $item_id, $component_name, $component_action, $secondary_item_id ) {
    14541454        global $bp, $wpdb;
    14551455
    … …  
    17131713     * @global unknown $wp_embed
    17141714     */
    1715     function __construct() {
     1715    public function __construct() {
    17161716        global $wp_embed;
    17171717
    … …  
    17621762     * @return string The embed HTML on success, otherwise the original URL.
    17631763     */
    1764     function shortcode( $attr, $url = '' ) {
     1764    public function shortcode( $attr, $url = '' ) {
    17651765        if ( empty( $url ) )
    17661766            return '';
    … …  
    18271827     * @return string The embed HTML on success, otherwise the original URL.
    18281828     */
    1829     function parse_oembed( $id, $url, $attr, $rawattr ) {
     1829    public function parse_oembed( $id, $url, $attr, $rawattr ) {
    18301830        $id = intval( $id );
    18311831
    … …  
    18951895     * @since BuddyPress (1.7)
    18961896     */
    1897     function walk( $elements, $max_depth ) {
     1897    public function walk( $elements, $max_depth ) {
    18981898        $args   = array_slice( func_get_args(), 2 );
    18991899        $output = '';
    … …  
    19821982     * @since BuddyPress (1.7)
    19831983     */
    1984     function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
     1984    public function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {
    19851985        // If we're someway down the tree, indent the HTML with the appropriate number of tabs
    19861986        $indent = $depth ? str_repeat( "\t", $depth ) : '';

  • trunk/bp-friends/bp-friends-classes.php

    r7298 r7308  
    1111
    1212class BP_Friends_Friendship {
    13     var $id;
    14     var $initiator_user_id;
    15     var $friend_user_id;
    16     var $is_confirmed;
    17     var $is_limited;
    18     var $date_created;
    19 
    20     var $is_request;
    21     var $populate_friend_details;
    22 
    23     var $friend;
    24 
    25     function __construct( $id = null, $is_request = false, $populate_friend_details = true ) {
     13    public $id;
     14    public $initiator_user_id;
     15    public $friend_user_id;
     16    public $is_confirmed;
     17    public $is_limited;
     18    public $date_created;
     19
     20    public $is_request;
     21    public $populate_friend_details;
     22
     23    public $friend;
     24
     25    public function __construct( $id = null, $is_request = false, $populate_friend_details = true ) {
    2626        $this->is_request = $is_request;
    2727
    … …  
    3333    }
    3434
    35     function populate() {
     35    public function populate() {
    3636        global $wpdb, $bp;
    3737
    … …  
    5353    }
    5454
    55     function save() {
     55    public function save() {
    5656        global $wpdb, $bp;
    5757
    … …  
    7979    }
    8080
    81     function delete() {
    82         global $wpdb, $bp;
    83 
     81    public function delete() {
     82        global $wpdb, $bp;
    8483        return $wpdb->query( $wpdb->prepare( "DELETE FROM {$bp->friends->table_name} WHERE id = %d", $this->id ) );
    8584    }
    … …  
    8786    /** Static Methods ********************************************************/
    8887
    89     function get_friend_user_ids( $user_id, $friend_requests_only = false, $assoc_arr = false ) {
     88    public static function get_friend_user_ids( $user_id, $friend_requests_only = false, $assoc_arr = false ) {
    9089        global $wpdb, $bp;
    9190
    … …  
    112111    }
    113112
    114     function get_friendship_id( $user_id, $friend_id ) {
     113    public static function get_friendship_id( $user_id, $friend_id ) {
    115114        global $wpdb, $bp;
    116115
    … …  
    118117    }
    119118
    120     function get_friendship_request_user_ids( $user_id ) {
     119    public static function get_friendship_request_user_ids( $user_id ) {
    121120        global $wpdb, $bp;
    122121
    … …  
    124123    }
    125124
    126     function total_friend_count( $user_id = 0 ) {
     125    public static function total_friend_count( $user_id = 0 ) {
    127126        global $wpdb, $bp;
    128127
    … …  
    140139
    141140        bp_update_user_meta( $user_id, 'total_friend_count', (int) $count );
    142         return (int) $count;
    143     }
    144 
    145     function search_friends( $filter, $user_id, $limit = null, $page = null ) {
     141
     142        return absint( $count );
     143    }
     144
     145    public static function search_friends( $filter, $user_id, $limit = null, $page = null ) {
    146146        global $wpdb, $bp;
    147147
    … …  
    183183    }
    184184
    185     function check_is_friend( $loggedin_userid, $possible_friend_userid ) {
     185    public static function check_is_friend( $loggedin_userid, $possible_friend_userid ) {
    186186        global $wpdb, $bp;
    187187
    … …  
    202202    }
    203203
    204     function get_bulk_last_active( $user_ids ) {
     204    public static function get_bulk_last_active( $user_ids ) {
    205205        global $wpdb;
    206206
    … …  
    210210    }
    211211
    212     function accept($friendship_id) {
    213         global $wpdb, $bp;
    214 
     212    public static function accept($friendship_id) {
     213        global $wpdb, $bp;
    215214        return $wpdb->query( $wpdb->prepare( "UPDATE {$bp->friends->table_name} SET is_confirmed = 1, date_created = %s WHERE id = %d AND friend_user_id = %d", bp_core_current_time(), $friendship_id, bp_loggedin_user_id() ) );
    216215    }
    217216
    218     function withdraw($friendship_id) {
    219         global $wpdb, $bp;
    220 
     217    public static function withdraw($friendship_id) {
     218        global $wpdb, $bp;
    221219        return $wpdb->query( $wpdb->prepare( "DELETE FROM {$bp->friends->table_name} WHERE id = %d AND initiator_user_id = %d", $friendship_id, bp_loggedin_user_id() ) );
    222220    }
    223221
    224     function reject($friendship_id) {
    225         global $wpdb, $bp;
    226 
     222    public static function reject($friendship_id) {
     223        global $wpdb, $bp;
    227224        return $wpdb->query( $wpdb->prepare( "DELETE FROM {$bp->friends->table_name} WHERE id = %d AND friend_user_id = %d", $friendship_id, bp_loggedin_user_id() ) );
    228225    }
    229226
    230     function search_users( $filter, $user_id, $limit = null, $page = null ) {
     227    public static function search_users( $filter, $user_id, $limit = null, $page = null ) {
    231228        global $wpdb, $bp;
    232229
    … …  
    254251    }
    255252
    256     function search_users_count( $filter ) {
     253    public static function search_users_count( $filter ) {
    257254        global $wpdb, $bp;
    258255
    … …  
    277274    }
    278275
    279     function sort_by_name( $user_ids ) {
     276    public static function sort_by_name( $user_ids ) {
    280277        global $wpdb, $bp;
    281278
    … …  
    288285    }
    289286
    290     function get_random_friends( $user_id, $total_friends = 5 ) {
     287    public static function get_random_friends( $user_id, $total_friends = 5 ) {
    291288        global $wpdb, $bp;
    292289
    … …  
    306303    }
    307304
    308     function get_invitable_friend_count( $user_id, $group_id ) {
     305    public static function get_invitable_friend_count( $user_id, $group_id ) {
    309306
    310307        // Setup some data we'll use below
    … …  
    333330    }
    334331
    335     function get_user_ids_for_friendship( $friendship_id ) {
    336         global $wpdb, $bp;
    337 
     332    public static function get_user_ids_for_friendship( $friendship_id ) {
     333        global $wpdb, $bp;
    338334        return $wpdb->get_row( $wpdb->prepare( "SELECT friend_user_id, initiator_user_id FROM {$bp->friends->table_name} WHERE id = %d", $friendship_id ) );
    339335    }
    340336
    341     function delete_all_for_user( $user_id ) {
     337    public static function delete_all_for_user( $user_id ) {
    342338        global $wpdb, $bp;
    343339

  • trunk/bp-groups/bp-groups-classes.php

    r7302 r7308  
    197197    /** Static Methods ********************************************************/
    198198
    199     function group_exists( $slug, $table_name = false ) {
     199    public static function group_exists( $slug, $table_name = false ) {
    200200        global $wpdb, $bp;
    201201
    … …  
    209209    }
    210210
    211     function get_id_from_slug( $slug ) {
     211    public static function get_id_from_slug( $slug ) {
    212212        return BP_Groups_Group::group_exists( $slug );
    213213    }
    … …  
    14871487    }
    14881488
    1489     function total_group_count( $user_id = 0 ) {
     1489    public static function total_group_count( $user_id = 0 ) {
    14901490        global $bp, $wpdb;
    14911491
    … …  
    15001500    }
    15011501
    1502     function get_invites( $user_id, $limit = false, $page = false, $exclude = false ) {
     1502    public static function get_invites( $user_id, $limit = false, $page = false, $exclude = false ) {
    15031503        global $wpdb, $bp;
    15041504
    … …  
    15511551    }
    15521552
    1553     function check_is_admin( $user_id, $group_id ) {
     1553    public static function check_is_admin( $user_id, $group_id ) {
    15541554        global $wpdb, $bp;
    15551555
    … …  
    15601560    }
    15611561
    1562     function check_is_mod( $user_id, $group_id ) {
     1562    public static function check_is_mod( $user_id, $group_id ) {
    15631563        global $wpdb, $bp;
    15641564
    … …  
    15691569    }
    15701570
    1571     function check_is_member( $user_id, $group_id ) {
     1571    public static function check_is_member( $user_id, $group_id ) {
    15721572        global $wpdb, $bp;
    15731573

  • trunk/bp-messages/bp-messages-classes.php

    r7307 r7308  
    1212
    1313class BP_Messages_Thread {
    14     var $thread_id;
    15     var $messages;
    16     var $recipients;
    17     var $sender_ids;
    18 
    19     var $unread_count;
     14    public $thread_id;
     15    public $messages;
     16    public $recipients;
     17    public $sender_ids;
     18
     19    public $unread_count;
    2020
    2121    /**
    … …  
    6767    public $messages_order;
    6868
    69     function __construct( $thread_id = false, $order = 'ASC' ) {
     69    public function __construct( $thread_id = false, $order = 'ASC' ) {
    7070        if ( $thread_id )
    7171            $this->populate( $thread_id, $order );
    7272    }
    7373
    74     function populate( $thread_id, $order ) {
     74    private function populate( $thread_id, $order ) {
    7575        global $wpdb, $bp;
    7676
    … …  
    9595    }
    9696
    97     function mark_read() {
     97    public function mark_read() {
    9898        BP_Messages_Thread::mark_as_read( $this->thread_id );
    9999    }
    100100
    101     function mark_unread() {
     101    public function mark_unread() {
    102102        BP_Messages_Thread::mark_as_unread( $this->thread_id );
    103103    }
    104104
    105     function get_recipients() {
     105    public function get_recipients() {
    106106        global $wpdb, $bp;
    107107
    … …  
    115115    }
    116116
    117     /** Static Functions **/
    118 
    119     function delete( $thread_id ) {
    120         global $wpdb, $bp;
    121 
    122         $delete_for_user = $wpdb->query( $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET is_deleted = 1 WHERE thread_id = %d AND user_id = %d", $thread_id, bp_loggedin_user_id() ) );
     117    /** Static Functions ******************************************************/
     118
     119    public static function delete( $thread_id ) {
     120        global $wpdb, $bp;
     121
     122        // Mark messages as deleted
     123        $wpdb->query( $wpdb->prepare( "UPDATE {$bp->messages->table_name_recipients} SET is_deleted = 1 WHERE thread_id = %d AND user_id = %d", $thread_id, bp_loggedin_user_id() ) );
    123124
    124125        // Check to see if any more recipients remain for this message
    … …  
    137138    }
    138139
    139     function get_current_threads_for_user( $user_id, $box = 'inbox', $type = 'all', $limit = null, $page = null, $search_terms = '' ) {
     140    public static function get_current_threads_for_user( $user_id, $box = 'inbox', $type = 'all', $limit = null, $page = null, $search_terms = '' ) {
    140141        global $wpdb, $bp;
    141142
    … …  
    178179    }
    179180
    180     function mark_as_read( $thread_id ) {
     181    public static function mark_as_read( $thread_id ) {
    181182        global $wpdb, $bp;
    182183
    … …  
    185186    }
    186187
    187     function mark_as_unread( $thread_id ) {
     188    public static function mark_as_unread( $thread_id ) {
    188189        global $wpdb, $bp;
    189190
    … …  
    192193    }
    193194
    194     function get_total_threads_for_user( $user_id, $box = 'inbox', $type = 'all' ) {
     195    public static function get_total_threads_for_user( $user_id, $box = 'inbox', $type = 'all' ) {
    195196        global $wpdb, $bp;
    196197
    … …  
    207208    }
    208209
    209     function user_is_sender( $thread_id ) {
     210    public static function user_is_sender( $thread_id ) {
    210211        global $wpdb, $bp;
    211212
    … …  
    218219    }
    219220
    220     function get_last_sender( $thread_id ) {
     221    public static function get_last_sender( $thread_id ) {
    221222        global $wpdb, $bp;
    222223
    … …  
    242243    }
    243244
    244     function check_access( $thread_id, $user_id = 0 ) {
     245    public static function check_access( $thread_id, $user_id = 0 ) {
    245246        global $wpdb, $bp;
    246247
    … …  
    251252    }
    252253
    253     function is_valid( $thread_id ) {
     254    public static function is_valid( $thread_id ) {
    254255        global $wpdb, $bp;
    255256
    … …  
    257258    }
    258259
    259     function get_recipient_links( $recipients ) {
     260    public static function get_recipient_links( $recipients ) {
    260261        if ( count( $recipients ) >= 5 )
    261262            return sprintf( __( '%s Recipients', 'buddypress' ), number_format_i18n( count( $recipients ) ) );
    … …  
    276277    }
    277278
    278     // Update Functions
    279 
    280     function update_tables() {
     279    public static function update_tables() {
    281280        global $wpdb, $bp;
    282281
    … …  
    309308
    310309class BP_Messages_Message {
    311     var $id;
    312     var $thread_id;
    313     var $sender_id;
    314     var $subject;
    315     var $message;
    316     var $date_sent;
    317 
    318     var $recipients = false;
    319 
    320     function __construct( $id = null ) {
     310    public $id;
     311    public $thread_id;
     312    public $sender_id;
     313    public $subject;
     314    public $message;
     315    public $date_sent;
     316
     317    public $recipients = false;
     318
     319    public function __construct( $id = null ) {
    321320        $this->date_sent = bp_core_current_time();
    322321        $this->sender_id = bp_loggedin_user_id();
    323322
    324         if ( !empty( $id ) )
     323        if ( !empty( $id ) ) {
    325324            $this->populate( $id );
    326     }
    327 
    328     function populate( $id ) {
     325        }
     326    }
     327
     328    private function populate( $id ) {
    329329        global $wpdb, $bp;
    330330
    … …  
    339339    }
    340340
    341     function send() {
     341    public function send() {
    342342        global $wpdb, $bp;
    343343
    … …  
    392392    }
    393393
    394     function get_recipients() {
     394    public function get_recipients() {
    395395        global $bp, $wpdb;
    396 
    397396        return $wpdb->get_results( $wpdb->prepare( "SELECT user_id FROM {$bp->messages->table_name_recipients} WHERE thread_id = %d", $this->thread_id ) );
    398397    }
    399398
    400     // Static Functions
    401 
    402     function get_recipient_ids( $recipient_usernames ) {
     399    /** Static Functions ******************************************************/
     400
     401    public static function get_recipient_ids( $recipient_usernames ) {
    403402        if ( !$recipient_usernames )
    404403            return false;
    … …  
    415414    }
    416415
    417     function get_last_sent_for_user( $thread_id ) {
    418         global $wpdb, $bp;
    419 
     416    public static function get_last_sent_for_user( $thread_id ) {
     417        global $wpdb, $bp;
    420418        return $wpdb->get_var( $wpdb->prepare( "SELECT id FROM {$bp->messages->table_name_messages} WHERE sender_id = %d AND thread_id = %d ORDER BY date_sent DESC LIMIT 1", bp_loggedin_user_id(), $thread_id ) );
    421419    }
    422420
    423     function is_user_sender( $user_id, $message_id ) {
     421    public static function is_user_sender( $user_id, $message_id ) {
    424422        global $wpdb, $bp;
    425423        return $wpdb->get_var( $wpdb->prepare( "SELECT id FROM {$bp->messages->table_name_messages} WHERE sender_id = %d AND id = %d", $user_id, $message_id ) );
    426424    }
    427425
    428     function get_message_sender( $message_id ) {
     426    public static function get_message_sender( $message_id ) {
    429427        global $wpdb, $bp;
    430428        return $wpdb->get_var( $wpdb->prepare( "SELECT sender_id FROM {$bp->messages->table_name_messages} WHERE id = %d", $message_id ) );
    … …  
    433431
    434432class BP_Messages_Notice {
    435     var $id = null;
    436     var $subject;
    437     var $message;
    438     var $date_sent;
    439     var $is_active;
    440 
    441     function __construct( $id = null ) {
     433    public $id = null;
     434    public $subject;
     435    public $message;
     436    public $date_sent;
     437    public $is_active;
     438
     439    public function __construct( $id = null ) {
    442440        if ( $id ) {
    443441            $this->id = $id;
    … …  
    446444    }
    447445
    448     function populate() {
     446    private function populate() {
    449447        global $wpdb, $bp;
    450448
    … …  
    459457    }
    460458
    461     function save() {
     459    private function save() {
    462460        global $wpdb, $bp;
    463461
    … …  
    488486    }
    489487
    490     function activate() {
     488    public function activate() {
    491489        $this->is_active = 1;
    492         if ( !$this->save() )
    493             return false;
    494 
    495         return true;
    496     }
    497 
    498     function deactivate() {
     490        return (bool) $this->save();
     491    }
     492
     493    public function deactivate() {
    499494        $this->is_active = 0;
    500         if ( !$this->save() )
    501             return false;
    502 
    503         return true;
    504     }
    505 
    506     function delete() {
     495        return (bool) $this->save();
     496    }
     497
     498    public function delete() {
    507499        global $wpdb, $bp;
    508500

  • trunk/bp-xprofile/bp-xprofile-classes.php

    r7285 r7308  
    1212
    1313class BP_XProfile_Group {
    14     var $id = null;
    15     var $name;
    16     var $description;
    17     var $can_delete;
    18     var $group_order;
    19     var $fields;
    20 
    21     function __construct( $id = null ) {
     14    public $id = null;
     15    public $name;
     16    public $description;
     17    public $can_delete;
     18    public $group_order;
     19    public $fields;
     20
     21    public function __construct( $id = null ) {
    2222        if ( !empty( $id ) )
    2323            $this->populate( $id );
    2424    }
    2525
    26     function populate( $id ) {
     26    private function populate( $id ) {
    2727        global $wpdb, $bp;
    2828
    … …  
    3939    }
    4040
    41     function save() {
     41    private function save() {
    4242        global $wpdb, $bp;
    4343
    … …  
    6464    }
    6565
    66     function delete() {
     66    private function delete() {
    6767        global $wpdb, $bp;
    6868
    … …  
    112112     * @return array $groups
    113113     */
    114     function get( $args = '' ) {
     114    public static function get( $args = array() ) {
    115115        global $wpdb, $bp;
    116116
    … …  
    264264    }
    265265
    266     function admin_validate() {
     266    public static function admin_validate() {
    267267        global $message;
    268268
    … …  
    276276    }
    277277
    278     function update_position( $field_group_id, $position ) {
     278    public static function update_position( $field_group_id, $position ) {
    279279        global $wpdb, $bp;
    280280
    … …  
    294294     * @return array $fields The database results, with field_visibility added
    295295     */
    296     function fetch_visibility_level( $user_id = 0, $fields = array() ) {
     296    public static function fetch_visibility_level( $user_id = 0, $fields = array() ) {
    297297
    298298        // Get the user's visibility level preferences
    … …  
    330330     *   visibility level + allow_custom (whether the admin allows this field to be set by user)
    331331     */
    332     function fetch_default_visibility_levels() {
     332    public static function fetch_default_visibility_levels() {
    333333        global $wpdb, $bp;
    334334
    … …  
    348348    }
    349349
    350     function render_admin_form() {
     350    public static function render_admin_form() {
    351351        global $message;
    352352
    … …  
    427427
    428428class BP_XProfile_Field {
    429     var $id;
    430     var $group_id;
    431     var $parent_id;
    432     var $type;
    433     var $name;
    434     var $description;
    435     var $is_required;
    436     var $can_delete;
    437     var $field_order;
    438     var $option_order;
    439     var $order_by;
    440     var $is_default_option;
    441     var $default_visibility;
    442     var $allow_custom_visibility = 'allowed';
    443 
    444     var $data;
    445     var $message = null;
    446     var $message_type = 'err';
    447 
    448     function __construct( $id = null, $user_id = null, $get_data = true ) {
     429    public $id;
     430    public $group_id;
     431    public $parent_id;
     432    public $type;
     433    public $name;
     434    public $description;
     435    public $is_required;
     436    public $can_delete;
     437    public $field_order;
     438    public $option_order;
     439    public $order_by;
     440    public $is_default_option;
     441    public $default_visibility;
     442    public $allow_custom_visibility = 'allowed';
     443
     444    public $data;
     445    public $message = null;
     446    public $message_type = 'err';
     447
     448    public function __construct( $id = null, $user_id = null, $get_data = true ) {
    449449        if ( !empty( $id ) )
    450450            $this->populate( $id, $user_id, $get_data );
    451451    }
    452452
    453     function populate( $id, $user_id, $get_data ) {
     453    private function populate( $id, $user_id, $get_data ) {
    454454        global $wpdb, $userdata, $bp;
    455455
    … …  
    488488    }
    489489
    490     function delete( $delete_data = false ) {
     490    private function delete( $delete_data = false ) {
    491491        global $wpdb, $bp;
    492492
    … …  
    507507    }
    508508
    509     function save() {
     509    private function save() {
    510510        global $wpdb, $bp;
    511511
    … …  
    624624    }
    625625
    626     function get_field_data( $user_id ) {
     626    public function get_field_data( $user_id ) {
    627627        return new BP_XProfile_ProfileData( $this->id, $user_id );
    628628    }
    629629
    630     function get_children( $for_editing = false ) {
     630    public function get_children( $for_editing = false ) {
    631631        global $wpdb, $bp;
    632632
    … …  
    655655    }
    656656
    657     function delete_children() {
     657    public function delete_children() {
    658658        global $wpdb, $bp;
    659659
    … …  
    663663    }
    664664
    665     /* Static Functions */
    666 
    667     function get_type( $field_id ) {
     665    /** Static Methods ********************************************************/
     666
     667    public static function get_type( $field_id ) {
    668668        global $wpdb, $bp;
    669669
    … …  
    681681    }
    682682
    683     function delete_for_group( $group_id ) {
     683    public static function delete_for_group( $group_id ) {
    684684        global $wpdb, $bp;
    685685
    … …  
    697697    }
    698698
    699     function get_id_from_name( $field_name ) {
     699    public static function get_id_from_name( $field_name ) {
    700700        global $wpdb, $bp;
    701701
    … …  
    706706    }
    707707
    708     function update_position( $field_id, $position, $field_group_id ) {
     708    public static function update_position( $field_id, $position, $field_group_id ) {
    709709        global $wpdb, $bp;
    710710
    … …  
    729729
    730730    /* This function populates the items for radio buttons checkboxes and drop down boxes */
    731     function render_admin_form_children() {
     731    public function render_admin_form_children() {
    732732        $input_types = array( 'checkbox', 'selectbox', 'multiselectbox', 'radio' );
    733733
    … …  
    817817    }
    818818
    819     function render_admin_form( $message = '' ) {
     819    public function render_admin_form( $message = '' ) {
    820820        if ( empty( $this->id ) ) {
    821821            $title  = __( 'Add Field', 'buddypress' );
    … …  
    977977    }
    978978
    979     function admin_validate() {
     979    public function admin_validate() {
    980980        global $message;
    981981
    … …  
    10041004
    10051005class BP_XProfile_ProfileData {
    1006     var $id;
    1007     var $user_id;
    1008     var $field_id;
    1009     var $value;
    1010     var $last_updated;
    1011 
    1012     function __construct( $field_id = null, $user_id = null ) {
     1006    public $id;
     1007    public $user_id;
     1008    public $field_id;
     1009    public $value;
     1010    public $last_updated;
     1011
     1012    public function __construct( $field_id = null, $user_id = null ) {
    10131013        if ( !empty( $field_id ) ) {
    10141014            $this->populate( $field_id, $user_id );
    … …  
    10161016    }
    10171017
    1018     function populate( $field_id, $user_id )  {
     1018    private function populate( $field_id, $user_id )  {
    10191019        global $wpdb, $bp;
    10201020
    … …  
    10351035
    10361036    /**
    1037      * exists ()
    1038      *
    10391037     * Check if there is data already for the user.
    10401038     *
    … …  
    10431041     * @return bool
    10441042     */
    1045     function exists() {
     1043    private function exists() {
    10461044        global $wpdb, $bp;
    10471045
    … …  
    10521050
    10531051    /**
    1054      * is_valid_field()
    1055      *
    10561052     * Check if this data is for a valid field.
    10571053     *
    … …  
    10601056     * @return bool
    10611057     */
    1062     function is_valid_field() {
     1058    private function is_valid_field() {
    10631059        global $wpdb, $bp;
    10641060
    … …  
    10681064    }
    10691065
    1070     function save() {
     1066    private function save() {
    10711067        global $wpdb, $bp;
    10721068
    … …  
    11021098    }
    11031099
    1104     function delete() {
     1100    private function delete() {
    11051101        global $wpdb, $bp;
    11061102
    … …  
    11111107    }
    11121108
    1113     /** Static Functions **/
     1109    /** Static Methods ********************************************************/
    11141110
    11151111    /**
    … …  
    11181114     * Get all of the profile information for a specific user.
    11191115     */
    1120     function get_all_for_user( $user_id ) {
     1116    public static function get_all_for_user( $user_id ) {
    11211117        global $wpdb, $bp;
    11221118
    … …  
    11501146     * @return int $fielddata_id
    11511147     */
    1152     function get_fielddataid_byid( $field_id, $user_id ) {
     1148    public static function get_fielddataid_byid( $field_id, $user_id ) {
    11531149        global $wpdb, $bp;
    11541150
    … …  
    11621158    }
    11631159
    1164     function get_value_byid( $field_id, $user_ids = null ) {
     1160    public static function get_value_byid( $field_id, $user_ids = null ) {
    11651161        global $wpdb, $bp;
    11661162
    … …  
    11781174    }
    11791175
    1180     function get_value_byfieldname( $fields, $user_id = null ) {
     1176    public static function get_value_byfieldname( $fields, $user_id = null ) {
    11811177        global $bp, $wpdb;
    11821178
    … …  
    12271223    }
    12281224
    1229     function delete_for_field( $field_id ) {
     1225    public static function delete_for_field( $field_id ) {
    12301226        global $wpdb, $bp;
    12311227
    … …  
    12361232    }
    12371233
    1238     function get_last_updated( $user_id ) {
     1234    public static function get_last_updated( $user_id ) {
    12391235        global $wpdb, $bp;
    12401236
    … …  
    12441240    }
    12451241
    1246     function delete_data_for_user( $user_id ) {
     1242    public static function delete_data_for_user( $user_id ) {
    12471243        global $wpdb, $bp;
    12481244
    … …  
    12501246    }
    12511247
    1252     function get_random( $user_id, $exclude_fullname ) {
     1248    public static function get_random( $user_id, $exclude_fullname ) {
    12531249        global $wpdb, $bp;
    12541250
    … …  
    12591255    }
    12601256
    1261     function get_fullname( $user_id = 0 ) {
     1257    public static function get_fullname( $user_id = 0 ) {
    12621258
    12631259        if ( empty( $user_id ) )
Note: See TracChangeset for help on using the changeset viewer.