Skip to:
Content

BuddyPress.org

Changeset 8710


Ignore:
Timestamp:
07/29/2014 01:24:17 PM (12 years ago)
Author:
johnjamesjacoby
Message:

Clean up code formatting in bp-activity-functions.php.

  • Brackets
  • Indentation
  • Fix incorrect phpdoc
  • Add inline doc
  • Move some variable assignments outside of comparisons
  • Remove some trailing whitespace
  • Use bp_parse_args()
  • Remove some one-time use variables
  • Replace $bp global usages with buddypress() calls
File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/bp-activity/bp-activity-functions.php

    r8530 r8710  
    6464 */
    6565function bp_activity_find_mentions( $content ) {
     66
    6667        $pattern = '/[@]+([A-Za-z0-9-_\.@]+)\b/';
    6768        preg_match_all( $pattern, $content, $usernames );
    6869
    6970        // Make sure there's only one instance of each username
    70         if ( !$usernames = array_unique( $usernames[1] ) )
    71                 return false;
     71        $usernames = array_unique( $usernames[1] );
     72
     73        // Bail if no usernames
     74        if ( empty( $usernames ) ) {
     75                return false;
     76        }
    7277
    7378        $mentioned_users = array();
     
    8388        }
    8489
    85         if ( empty( $mentioned_users ) )
    86                 return false;
     90        if ( empty( $mentioned_users ) ) {
     91                return false;
     92        }
    8793
    8894        return $mentioned_users;
     
    100106function bp_activity_clear_new_mentions( $user_id ) {
    101107        bp_delete_user_meta( $user_id, 'bp_new_mention_count' );
    102         bp_delete_user_meta( $user_id, 'bp_new_mentions' );
     108        bp_delete_user_meta( $user_id, 'bp_new_mentions'      );
    103109}
    104110
     
    120126 */
    121127function bp_activity_adjust_mention_count( $activity_id = 0, $action = 'add' ) {
    122         if ( empty( $activity_id ) )
    123                 return false;
     128
     129        // Bail if no activity ID passed
     130        if ( empty( $activity_id ) ) {
     131                return false;
     132        }
    124133
    125134        // Get activity object
    126         $activity = new BP_Activity_Activity( (int) $activity_id );
     135        $activity  = new BP_Activity_Activity( (int) $activity_id );
    127136
    128137        // Try to find mentions
     
    130139
    131140        // Still empty? Stop now
    132         if ( empty( $usernames ) )
    133                 return false;
     141        if ( empty( $usernames ) ) {
     142                return false;
     143        }
    134144
    135145        // Increment mention count foreach mentioned user
     
    156166 */
    157167function bp_activity_update_mention_count_for_user( $user_id, $activity_id, $action = 'add' ) {
    158         if ( empty( $user_id ) || empty( $activity_id ) )
    159                 return false;
     168
     169        if ( empty( $user_id ) || empty( $activity_id ) ) {
     170                return false;
     171        }
    160172
    161173        // Adjust the mention list and count for the member
    162174        $new_mention_count = (int) bp_get_user_meta( $user_id, 'bp_new_mention_count', true );
    163         if ( !$new_mentions = bp_get_user_meta( $user_id, 'bp_new_mentions', true ) )
     175        $new_mentions      =       bp_get_user_meta( $user_id, 'bp_new_mentions',      true );
     176
     177        // Make sure new mentions is an array
     178        if ( empty( $new_mentions ) ) {
    164179                $new_mentions = array();
     180        }
    165181
    166182        switch ( $action ) {
     
    322338 * @since BuddyPress (1.1.0)
    323339 *
    324  * @global object $bp BuddyPress global settings.
    325340 * @uses apply_filters() To call the 'bp_activity_get_action' hook.
    326341 *
     
    330345 */
    331346function bp_activity_get_action( $component_id, $key ) {
    332         global $bp;
    333347
    334348        // Return false if any of the above values are not set
    335         if ( empty( $component_id ) || empty( $key ) )
    336                 return false;
    337 
    338         return apply_filters( 'bp_activity_get_action', $bp->activity->actions->{$component_id}->{$key}, $component_id, $key );
     349        if ( empty( $component_id ) || empty( $key ) ) {
     350                return false;
     351        }
     352
     353        $bp     = buddypress();
     354        $retval = isset( $bp->activity->actions->{$component_id}->{$key} )
     355                ? $bp->activity->actions->{$component_id}->{$key}
     356                : false;
     357
     358        return apply_filters( 'bp_activity_get_action', $retval, $component_id, $key );
    339359}
    340360
     
    353373                $action = array_values( (array) $action );
    354374
    355                 for ( $i = 0, $i_count = count( $action ); $i < $i_count; $i++ )
     375                for ( $i = 0, $i_count = count( $action ); $i < $i_count; $i++ ) {
    356376                        $actions[ $action[$i]['key'] ] = $action[$i]['value'];
     377                }
    357378        }
    358379
     
    379400
    380401        // Fallback to logged in user if no user_id is passed
    381         if ( empty( $user_id ) )
     402        if ( empty( $user_id ) ) {
    382403                $user_id = bp_displayed_user_id();
     404        }
    383405
    384406        // Get favorites for user
     
    408430
    409431        // Favorite activity stream items are for logged in users only
    410         if ( !is_user_logged_in() )
    411                 return false;
     432        if ( ! is_user_logged_in() ) {
     433                return false;
     434        }
    412435
    413436        // Fallback to logged in user if no user_id is passed
    414         if ( empty( $user_id ) )
     437        if ( empty( $user_id ) ) {
    415438                $user_id = bp_loggedin_user_id();
     439        }
    416440
    417441        $my_favs = bp_get_user_meta( $user_id, 'bp_favorite_activities', true );
     
    472496
    473497        // Favorite activity stream items are for logged in users only
    474         if ( !is_user_logged_in() )
    475                 return false;
     498        if ( ! is_user_logged_in() ) {
     499                return false;
     500        }
    476501
    477502        // Fallback to logged in user if no user_id is passed
    478         if ( empty( $user_id ) )
     503        if ( empty( $user_id ) ) {
    479504                $user_id = bp_loggedin_user_id();
     505        }
    480506
    481507        $my_favs = bp_get_user_meta( $user_id, 'bp_favorite_activities', true );
     
    492518
    493519        // Update the total number of users who have favorited this activity
    494         if ( $fav_count = bp_activity_get_meta( $activity_id, 'favorite_count' ) ) {
     520        $fav_count = bp_activity_get_meta( $activity_id, 'favorite_count' );
     521        if ( ! empty( $fav_count ) ) {
    495522
    496523                // Deduct from total favorites
     
    564591
    565592        // Fallback on displayed user, and then logged in user
    566         if ( empty( $user_id ) )
     593        if ( empty( $user_id ) ) {
    567594                $user_id = ( bp_displayed_user_id() ) ? bp_displayed_user_id() : bp_loggedin_user_id();
     595        }
    568596
    569597        return BP_Activity_Activity::total_favorite_count( $user_id );
     
    593621 */
    594622function bp_activity_delete_meta( $activity_id, $meta_key = '', $meta_value = '', $delete_all = false ) {
    595         global $wpdb, $bp;
    596623
    597624        // Legacy - if no meta_key is passed, delete all for the item
     
    674701 * @param string $meta_key Metadata key.
    675702 * @param mixed $meta_value Metadata value.
    676  * @param bool $unique. Optional. Whether to enforce a single metadata value
     703 * @param bool $unique Optional. Whether to enforce a single metadata value
    677704 *        for the given key. If true, and the object already has a value for
    678705 *        the key, no change will be made. Default: false.
     
    705732
    706733        // Do not delete user data unless a logged in user says so
    707         if ( empty( $user_id ) || !is_user_logged_in() )
    708                 return false;
     734        if ( empty( $user_id ) || ! is_user_logged_in() ) {
     735                return false;
     736        }
    709737
    710738        // Clear the user's activity from the sitewide stream and clear their activity tables
     
    712740
    713741        // Remove any usermeta
    714         bp_delete_user_meta( $user_id, 'bp_latest_update' );
     742        bp_delete_user_meta( $user_id, 'bp_latest_update'       );
    715743        bp_delete_user_meta( $user_id, 'bp_favorite_activities' );
    716744
     
    735763 */
    736764function bp_activity_spam_all_user_data( $user_id = 0 ) {
    737         global $bp, $wpdb;
     765        global $wpdb;
    738766
    739767        // Do not delete user data unless a logged in user says so
    740         if ( empty( $user_id ) || ! is_user_logged_in() )
    741                 return false;
     768        if ( empty( $user_id ) || ! is_user_logged_in() ) {
     769                return false;
     770        }
    742771
    743772        // Get all the user's activities.
    744         $activities = bp_activity_get( array( 'display_comments' => 'stream', 'filter' => array( 'user_id' => $user_id ), 'show_hidden' => true, ) );
     773        $activities = bp_activity_get( array(
     774                'display_comments' => 'stream',
     775                'filter'           => array( 'user_id' => $user_id ),
     776                'show_hidden'      => true
     777        ) );
     778
     779        $bp = buddypress();
    745780
    746781        // Mark each as spam
     
    749784                // Create an activity object
    750785                $activity_obj = new BP_Activity_Activity;
    751                 foreach ( $activity as $k => $v )
     786                foreach ( $activity as $k => $v ) {
    752787                        $activity_obj->$k = $v;
     788                }
    753789
    754790                // Mark as spam
     
    762798                 * we need to update manually.
    763799                 */
    764                 if ( ! empty( $bp->activity->akismet ) )
     800                if ( ! empty( $bp->activity->akismet ) ) {
    765801                        $bp->activity->akismet->update_activity_spam_meta( $activity_obj );
     802                }
    766803
    767804                // Tidy up
     
    788825 */
    789826function bp_activity_ham_all_user_data( $user_id = 0 ) {
    790         global $bp, $wpdb;
     827        global $wpdb;
    791828
    792829        // Do not delete user data unless a logged in user says so
    793         if ( empty( $user_id ) || ! is_user_logged_in() )
    794                 return false;
     830        if ( empty( $user_id ) || ! is_user_logged_in() ) {
     831                return false;
     832        }
    795833
    796834        // Get all the user's activities.
    797         $activities = bp_activity_get( array( 'display_comments' => 'stream', 'filter' => array( 'user_id' => $user_id ), 'show_hidden' => true, 'spam' => 'all', ) );
     835        $activities = bp_activity_get( array(
     836                'display_comments' => 'stream',
     837                'filter'           => array( 'user_id' => $user_id ),
     838                'show_hidden'      => true,
     839                'spam'             => 'all'
     840        ) );
     841
     842        $bp = buddypress();
    798843
    799844        // Mark each as not spam
     
    802847                // Create an activity object
    803848                $activity_obj = new BP_Activity_Activity;
    804                 foreach ( $activity as $k => $v )
     849                foreach ( $activity as $k => $v ) {
    805850                        $activity_obj->$k = $v;
     851                }
    806852
    807853                // Mark as not spam
     
    815861                 * we need to update manually.
    816862                 */
    817                 if ( ! empty( $bp->activity->akismet ) )
     863                if ( ! empty( $bp->activity->akismet ) ) {
    818864                        $bp->activity->akismet->update_activity_ham_meta( $activity_obj );
     865                }
    819866
    820867                // Tidy up
     
    834881 *
    835882 * @since BuddyPress (1.6.0)
    836  *
    837  * @global object $bp BuddyPress global settings.
    838883 */
    839884function bp_activity_register_activity_actions() {
    840         global $bp;
     885        $bp = buddypress();
    841886
    842887        bp_activity_set_action(
     
    872917 */
    873918function bp_activity_generate_action_string( $activity ) {
     919
    874920        // Check for valid input
    875921        if ( empty( $activity->component ) || empty( $activity->type ) ) {
     
    952998 */
    953999function bp_activity_get( $args = '' ) {
    954         $defaults = array(
     1000
     1001        $r = bp_parse_args( $args, array(
    9551002                'max'               => false,        // Maximum number of results to return
    9561003                'page'              => 1,            // page 1 without a per_page will result in no pagination.
     
    9791026                 */
    9801027                'filter' => array()
    981         );
    982         $r = wp_parse_args( $args, $defaults );
     1028        ) );
    9831029
    9841030        // Attempt to return a cached copy of the first page of sitewide activity.
    985         if ( 1 == (int) $r['page'] && empty( $r['max'] ) && empty( $r['search_terms'] ) && empty( $r['meta_query'] ) && empty( $r['filter'] ) && empty( $r['exclude'] ) && empty( $r['in'] ) && 'DESC' == $r['sort'] && empty( $r['exclude'] ) && 'ham_only' == $r['spam'] ) {
    986                 if ( !$activity = wp_cache_get( 'bp_activity_sitewide_front', 'bp' ) ) {
    987                         $args = array(
     1031        if ( ( 1 === (int) $r['page'] ) && empty( $r['max'] ) && empty( $r['search_terms'] ) && empty( $r['meta_query'] ) && empty( $r['filter'] ) && empty( $r['exclude'] ) && empty( $r['in'] ) && ( 'DESC' === $r['sort'] ) && empty( $r['exclude'] ) && ( 'ham_only' === $r['spam'] ) ) {
     1032
     1033                $activity = wp_cache_get( 'bp_activity_sitewide_front', 'bp' );
     1034                if ( false === $activity ) {
     1035
     1036                        $activity = BP_Activity_Activity::get( array(
    9881037                                'page'              => $r['page'],
    9891038                                'per_page'          => $r['per_page'],
     
    9981047                                'update_meta_cache' => $r['update_meta_cache'],
    9991048                                'count_total'       => $r['count_total'],
    1000                         );
    1001                         $activity = BP_Activity_Activity::get( $args );
     1049                        ) );
     1050
    10021051                        wp_cache_set( 'bp_activity_sitewide_front', $activity, 'bp' );
    10031052                }
    10041053
    10051054        } else {
    1006                 $args = array(
     1055                $activity = BP_Activity_Activity::get( array(
    10071056                        'page'             => $r['page'],
    10081057                        'per_page'         => $r['per_page'],
     
    10181067                        'spam'             => $r['spam'],
    10191068                        'count_total'      => $r['count_total'],
    1020                 );
    1021                 $activity = BP_Activity_Activity::get( $args );
     1069                ) );
    10221070        }
    10231071
     
    10441092 */
    10451093function bp_activity_get_specific( $args = '' ) {
    1046         $defaults = array(
    1047                 'activity_ids'      => false,       // A single activity_id or array of IDs.
    1048                 'display_comments'  => false,       // true or false to display threaded comments for these specific activity items
    1049                 'max'               => false,       // Maximum number of results to return
    1050                 'page'              => 1,           // page 1 without a per_page will result in no pagination.
    1051                 'per_page'          => false,       // results per page
    1052                 'show_hidden'       => true,        // When fetching specific items, show all
    1053                 'sort'              => 'DESC',      // sort ASC or DESC
    1054                 'spam'              => 'ham_only',  // Retrieve items marked as spam
     1094
     1095        $r = bp_parse_args( $args, array(
     1096                'activity_ids'      => false,      // A single activity_id or array of IDs.
     1097                'display_comments'  => false,      // true or false to display threaded comments for these specific activity items
     1098                'max'               => false,      // Maximum number of results to return
     1099                'page'              => 1,          // page 1 without a per_page will result in no pagination.
     1100                'per_page'          => false,      // results per page
     1101                'show_hidden'       => true,       // When fetching specific items, show all
     1102                'sort'              => 'DESC',     // sort ASC or DESC
     1103                'spam'              => 'ham_only', // Retrieve items marked as spam
    10551104                'update_meta_cache' => true,
    1056         );
    1057         $r = wp_parse_args( $args, $defaults );
     1105        ) );
    10581106
    10591107        $get_args = array(
     1108                'display_comments'  => $r['display_comments'],
     1109                'in'                => $r['activity_ids'],
     1110                'max'               => $r['max'],
    10601111                'page'              => $r['page'],
    10611112                'per_page'          => $r['per_page'],
    1062                 'max'               => $r['max'],
     1113                'show_hidden'       => $r['show_hidden'],
    10631114                'sort'              => $r['sort'],
    1064                 'display_comments'  => $r['display_comments'],
    1065                 'show_hidden'       => $r['show_hidden'],
    1066                 'in'                => $r['activity_ids'],
    10671115                'spam'              => $r['spam'],
    10681116                'update_meta_cache' => $r['update_meta_cache'],
    10691117        );
     1118
    10701119        return apply_filters( 'bp_activity_get_specific', BP_Activity_Activity::get( $get_args ), $args, $get_args );
    10711120}
     
    11171166function bp_activity_add( $args = '' ) {
    11181167
    1119         $defaults = array(
    1120                 'id'                => false, // Pass an existing activity ID to update an existing entry.
    1121 
    1122                 'action'            => '',    // The activity action - e.g. "Jon Doe posted an update"
    1123                 'content'           => '',    // Optional: The content of the activity item e.g. "BuddyPress is awesome guys!"
    1124 
    1125                 'component'         => false, // The name/ID of the component e.g. groups, profile, mycomponent
    1126                 'type'              => false, // The activity type e.g. activity_update, profile_updated
    1127                 'primary_link'      => '',    // Optional: The primary URL for this item in RSS feeds (defaults to activity permalink)
    1128 
    1129                 'user_id'           => bp_loggedin_user_id(), // Optional: The user to record the activity for, can be false if this activity is not for a user.
    1130                 'item_id'           => false, // Optional: The ID of the specific item being recorded, e.g. a blog_id
    1131                 'secondary_item_id' => false, // Optional: A second ID used to further filter e.g. a comment_id
     1168        $r = bp_parse_args( $args, array(
     1169                'id'                => false,                  // Pass an existing activity ID to update an existing entry.
     1170                'action'            => '',                     // The activity action - e.g. "Jon Doe posted an update"
     1171                'content'           => '',                     // Optional: The content of the activity item e.g. "BuddyPress is awesome guys!"
     1172                'component'         => false,                  // The name/ID of the component e.g. groups, profile, mycomponent
     1173                'type'              => false,                  // The activity type e.g. activity_update, profile_updated
     1174                'primary_link'      => '',                     // Optional: The primary URL for this item in RSS feeds (defaults to activity permalink)
     1175                'user_id'           => bp_loggedin_user_id(),  // Optional: The user to record the activity for, can be false if this activity is not for a user.
     1176                'item_id'           => false,                  // Optional: The ID of the specific item being recorded, e.g. a blog_id
     1177                'secondary_item_id' => false,                  // Optional: A second ID used to further filter e.g. a comment_id
    11321178                'recorded_time'     => bp_core_current_time(), // The GMT time that this activity was recorded
    1133                 'hide_sitewide'     => false, // Should this be hidden on the sitewide activity stream?
    1134                 'is_spam'           => false, // Is this activity item to be marked as spam?
    1135         );
    1136         $r = wp_parse_args( $args, $defaults );
     1179                'hide_sitewide'     => false,                  // Should this be hidden on the sitewide activity stream?
     1180                'is_spam'           => false,                  // Is this activity item to be marked as spam?
     1181        ) );
    11371182
    11381183        // Make sure we are backwards compatible
    1139         if ( empty( $r['component'] ) && !empty( $r['component_name'] ) )
     1184        if ( empty( $r['component'] ) && !empty( $r['component_name'] ) ) {
    11401185                $r['component'] = $r['component_name'];
    1141 
    1142         if ( empty( $r['type'] ) && !empty( $r['component_action'] ) )
     1186        }
     1187
     1188        if ( empty( $r['type'] ) && !empty( $r['component_action'] ) ) {
    11431189                $r['type'] = $r['component_action'];
     1190        }
    11441191
    11451192        // Setup activity to be added
     
    11611208
    11621209        // If this is an activity comment, rebuild the tree
    1163         if ( 'activity_comment' == $activity->type )
     1210        if ( 'activity_comment' == $activity->type ) {
    11641211                BP_Activity_Activity::rebuild_activity_comment_tree( $activity->item_id );
     1212        }
    11651213
    11661214        wp_cache_delete( 'bp_activity_sitewide_front', 'bp' );
     
    11751223 * @since BuddyPress (1.2.0)
    11761224 *
    1177  * @global object $bp BuddyPress global settings.
    11781225 * @uses wp_parse_args()
    11791226 * @uses bp_is_user_inactive()
     
    11941241 */
    11951242function bp_activity_post_update( $args = '' ) {
    1196         global $bp;
    1197 
    1198         $defaults = array(
     1243
     1244        $r = wp_parse_args( $args, array(
    11991245                'content' => false,
    12001246                'user_id' => bp_loggedin_user_id()
    1201         );
    1202         $r = wp_parse_args( $args, $defaults );
    1203 
    1204         if ( empty( $r['content'] ) || !strlen( trim( $r['content'] ) ) )
    1205                 return false;
    1206 
    1207         if ( bp_is_user_inactive( $r['user_id'] ) )
    1208                 return false;
     1247        ) );
     1248
     1249        if ( empty( $r['content'] ) || !strlen( trim( $r['content'] ) ) ) {
     1250                return false;
     1251        }
     1252
     1253        if ( bp_is_user_inactive( $r['user_id'] ) ) {
     1254                return false;
     1255        }
    12091256
    12101257        // Record this on the user's profile
    1211         $from_user_link   = bp_core_get_userlink( $r['user_id'] );
    12121258        $activity_content = $r['content'];
    12131259        $primary_link     = bp_core_get_userlink( $r['user_id'], false, true );
     
    12181264                'content'      => apply_filters( 'bp_activity_new_update_content', $activity_content ),
    12191265                'primary_link' => apply_filters( 'bp_activity_new_update_primary_link', $primary_link ),
    1220                 'component'    => $bp->activity->id,
     1266                'component'    => buddypress()->activity->id,
    12211267                'type'         => 'activity_update',
    12221268        ) );
     
    12251271
    12261272        // Add this update to the "latest update" usermeta so it can be fetched anywhere.
    1227         bp_update_user_meta( bp_loggedin_user_id(), 'bp_latest_update', array( 'id' => $activity_id, 'content' => $activity_content ) );
     1273        bp_update_user_meta( bp_loggedin_user_id(), 'bp_latest_update', array(
     1274                'id'      => $activity_id,
     1275                'content' => $activity_content
     1276        ) );
    12281277
    12291278        do_action( 'bp_activity_posted_update', $r['content'], $r['user_id'], $activity_id );
     
    13281377 */
    13291378function bp_activity_get_activity_id( $args = '' ) {
    1330         $defaults = array(
     1379
     1380        $r = bp_parse_args( $args, array(
    13311381                'user_id'           => false,
    13321382                'component'         => false,
     
    13371387                'content'           => false,
    13381388                'date_recorded'     => false,
    1339         );
    1340 
    1341         $r = wp_parse_args( $args, $defaults );
    1342 
    1343         return apply_filters( 'bp_activity_get_activity_id', BP_Activity_Activity::get_id( $r['user_id'], $r['component'], $r['type'], $r['item_id'], $r['secondary_item_id'], $r['action'], $r['content'], $r['date_recorded'] ) );
     1389        ) );
     1390
     1391        return apply_filters( 'bp_activity_get_activity_id', BP_Activity_Activity::get_id(
     1392                $r['user_id'],
     1393                $r['component'],
     1394                $r['type'],
     1395                $r['item_id'],
     1396                $r['secondary_item_id'],
     1397                $r['action'],
     1398                $r['content'],
     1399                $r['date_recorded']
     1400        ) );
    13441401}
    13451402
     
    13791436
    13801437        // Pass one or more the of following variables to delete by those variables
    1381         $defaults = array(
     1438        $args = bp_parse_args( $args, array(
    13821439                'id'                => false,
    13831440                'action'            => false,
     
    13911448                'date_recorded'     => false,
    13921449                'hide_sitewide'     => false
    1393         );
    1394 
    1395         $args = wp_parse_args( $args, $defaults );
     1450        ) );
    13961451
    13971452        do_action( 'bp_before_activity_delete', $args );
     
    14001455        bp_activity_adjust_mention_count( $args['id'], 'delete' );
    14011456
    1402         if ( !$activity_ids_deleted = BP_Activity_Activity::delete( $args ) )
    1403                 return false;
     1457        $activity_ids_deleted = BP_Activity_Activity::delete( $args );
     1458        if ( empty( $activity_ids_deleted ) ) {
     1459                return false;
     1460        }
    14041461
    14051462        // Check if the user's latest update has been deleted
    1406         if ( empty( $args['user_id'] ) )
    1407                 $user_id = bp_loggedin_user_id();
    1408         else
    1409                 $user_id = $args['user_id'];
     1463        $user_id = empty( $args['user_id'] )
     1464                ? bp_loggedin_user_id()
     1465                : $args['user_id'];
    14101466
    14111467        $latest_update = bp_get_user_meta( $user_id, 'bp_latest_update', true );
     
    14421498        function bp_activity_delete_by_item_id( $args = '' ) {
    14431499
    1444                 $defaults = array(
     1500                $r = bp_parse_args( $args, array(
    14451501                        'item_id'           => false,
    14461502                        'component'         => false,
     
    14481504                        'user_id'           => false,
    14491505                        'secondary_item_id' => false
    1450                 );
    1451                 $r = wp_parse_args( $args, $defaults );
    1452 
    1453                 return bp_activity_delete( array( 'item_id' => $r['item_id'], 'component' => $r['component'], 'type' => $r['type'], 'user_id' => $r['user_id'], 'secondary_item_id' => $r['secondary_item_id'] ) );
     1506                ) );
     1507
     1508                return bp_activity_delete( $r );
    14541509        }
    14551510
     
    14851540         */
    14861541        function bp_activity_delete_by_content( $user_id, $content, $component, $type ) {
    1487                 return bp_activity_delete( array( 'user_id' => $user_id, 'content' => $content, 'component' => $component, 'type' => $type ) );
     1542                return bp_activity_delete( array(
     1543                        'user_id'   => $user_id,
     1544                        'content'   => $content,
     1545                        'component' => $component,
     1546                        'type'      => $type
     1547                ) );
    14881548        }
    14891549
     
    15031563         */
    15041564        function bp_activity_delete_for_user_by_component( $user_id, $component ) {
    1505                 return bp_activity_delete( array( 'user_id' => $user_id, 'component' => $component ) );
     1565                return bp_activity_delete( array(
     1566                        'user_id'   => $user_id,
     1567                        'component' => $component
     1568                ) );
    15061569        }
    15071570
     
    15311594         * handle the deletion of child comments differently. Make sure you return false.
    15321595         */
    1533         if ( !apply_filters( 'bp_activity_delete_comment_pre', true, $activity_id, $comment_id ) )
    1534                 return false;
     1596        if ( ! apply_filters( 'bp_activity_delete_comment_pre', true, $activity_id, $comment_id ) ) {
     1597                return false;
     1598        }
    15351599
    15361600        // Delete any children of this comment.
     
    15381602
    15391603        // Delete the actual comment
    1540         if ( !bp_activity_delete( array( 'id' => $comment_id, 'type' => 'activity_comment' ) ) )
    1541                 return false;
     1604        if ( ! bp_activity_delete( array( 'id' => $comment_id, 'type' => 'activity_comment' ) ) ) {
     1605                return false;
     1606        }
    15421607
    15431608        // Recalculate the comment tree
     
    15921657function bp_activity_get_permalink( $activity_id, $activity_obj = false ) {
    15931658
    1594         if ( empty( $activity_obj ) )
     1659        if ( empty( $activity_obj ) ) {
    15951660                $activity_obj = new BP_Activity_Activity( $activity_id );
     1661        }
    15961662
    15971663        if ( isset( $activity_obj->current_comment ) ) {
     
    19071973 * @param string $cache An empty string passed by BP_Embed::parse_oembed() for
    19081974 *        functions like this one to filter.
     1975 * @param string $cachekey The cache key generated in BP_Embed::parse_oembed().
    19091976 * @param int $id The ID of the activity item.
    1910  * @param string $cachekey The cache key generated in BP_Embed::parse_oembed().
    19111977 * @return bool True on success, false on failure.
    19121978 */
Note: See TracChangeset for help on using the changeset viewer.