Skip to:
Content

BuddyPress.org


Ignore:
Timestamp:
02/23/2015 06:41:51 PM (10 years ago)
Author:
johnjamesjacoby
Message:

Activity: Clean up bp-activity-template.php:

  • Brackets
  • Whitespace
  • Remove unused variables
File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-activity/bp-activity-template.php

    r9536 r9537  
    240240     * }
    241241     */
    242     function __construct( $args ) {
     242    public function __construct( $args ) {
    243243        $bp = buddypress();
    244244
    … …  
    367367        // Fetch parent content for activity comments so we do not have to query in the loop
    368368        foreach ( (array) $this->activities as $activity ) {
    369             if ( 'activity_comment' != $activity->type )
     369            if ( 'activity_comment' != $activity->type ) {
    370370                continue;
     371            }
    371372
    372373            $parent_ids[] = $activity->item_id;
    373374        }
    374375
    375         if ( !empty( $parent_ids ) )
     376        if ( !empty( $parent_ids ) ) {
    376377            $activity_parents = bp_activity_get_specific( array( 'activity_ids' => $parent_ids ) );
     378        }
    377379
    378380        if ( !empty( $activity_parents['activities'] ) ) {
    379             foreach( $activity_parents['activities'] as $parent )
     381            foreach( $activity_parents['activities'] as $parent ) {
    380382                $this->activity_parents[$parent->id] = $parent;
     383            }
    381384
    382385            unset( $activity_parents );
    … …  
    405408     */
    406409    function has_activities() {
    407         if ( $this->activity_count )
     410        if ( $this->activity_count ) {
    408411            return true;
     412        }
    409413
    410414        return false;
    … …  
    416420     * @return object The next activity item to iterate over.
    417421     */
    418     function next_activity() {
     422    public function next_activity() {
    419423        $this->current_activity++;
    420         $this->activity = $this->activities[$this->current_activity];
     424        $this->activity = $this->activities[ $this->current_activity ];
    421425
    422426        return $this->activity;
    … …  
    426430     * Rewind the posts and reset post index.
    427431     */
    428     function rewind_activities() {
     432    public function rewind_activities() {
    429433        $this->current_activity = -1;
    430434        if ( $this->activity_count > 0 ) {
    … …  
    445449     *              otherwise false.
    446450     */
    447     function user_activities() {
    448         if ( $this->current_activity + 1 < $this->activity_count ) {
     451    public function user_activities() {
     452        if ( ( $this->current_activity + 1 ) < $this->activity_count ) {
    449453            return true;
    450         } elseif ( $this->current_activity + 1 == $this->activity_count ) {
     454        } elseif ( ( $this->current_activity + 1 ) == $this->activity_count ) {
    451455
    452456            /**
    … …  
    456460             */
    457461            do_action( 'activity_loop_end' );
     462
    458463            // Do some cleaning up after the loop
    459464            $this->rewind_activities();
    … …  
    461466
    462467        $this->in_the_loop = false;
     468
    463469        return false;
    464470    }
    … …  
    473479     * @see bp_the_activity()
    474480     */
    475     function the_activity() {
     481    public function the_activity() {
    476482
    477483        $this->in_the_loop = true;
    478484        $this->activity    = $this->next_activity();
    479485
    480         if ( is_array( $this->activity ) )
     486        if ( is_array( $this->activity ) ) {
    481487            $this->activity = (object) $this->activity;
    482 
    483         if ( $this->current_activity == 0 ) { // loop has just started
     488        }
     489
     490        // loop has just started
     491        if ( $this->current_activity == 0 ) {
    484492
    485493            /**
    … …  
    15051513
    15061514        // If this is a user object pass the users' email address for Gravatar so we don't have to prefetch it.
    1507         if ( 'user' == $object && empty( $user_id ) && empty( $email ) && isset( $current_activity_item->user_email ) )
     1515        if ( 'user' == $object && empty( $user_id ) && empty( $email ) && isset( $current_activity_item->user_email ) ) {
    15081516            $email = $current_activity_item->user_email;
     1517        }
    15091518
    15101519        /**
    … …  
    17451754        global $activities_template;
    17461755
    1747         $defaults = array(
     1756        $args = wp_parse_args( $args, array(
    17481757            'no_timestamp' => false,
    1749         );
    1750 
    1751         $args = wp_parse_args( $args, $defaults );
    1752         extract( $args, EXTR_SKIP );
     1758        ) );
    17531759
    17541760        $action = $activities_template->activity->action;
    … …  
    17631769        $action = apply_filters_ref_array( 'bp_get_activity_action_pre_meta', array( $action, &$activities_template->activity, $args ) );
    17641770
    1765         if ( ! empty( $action ) && ! $no_timestamp )
     1771        if ( ! empty( $action ) && empty( $r['no_timestamp'] ) ) {
    17661772            $action = bp_insert_activity_meta( $action );
     1773        }
    17671774
    17681775        /**
    … …  
    18021809
    18031810        // Backwards compatibility if action is not being used
    1804         if ( empty( $activities_template->activity->action ) && !empty( $activities_template->activity->content ) )
     1811        if ( empty( $activities_template->activity->action ) && ! empty( $activities_template->activity->content ) ) {
    18051812            $activities_template->activity->content = bp_insert_activity_meta( $activities_template->activity->content );
     1813        }
    18061814
    18071815        /**
    … …  
    18271835    global $activities_template;
    18281836
    1829     if ( !empty( $activities_template->activity->content ) )
     1837    if ( ! empty( $activities_template->activity->content ) ) {
    18301838        return true;
     1839    }
    18311840
    18321841    return false;
    … …  
    20122021     *
    20132022     * @global object $activities_template {@link BP_Activity_Template}
    2014      * @uses wp_parse_args()
    20152023     * @uses apply_filters() To call the 'bp_get_activity_parent_content' hook.
    20162024     *
    2017      * @param array $args {
    2018      *     Array of optional arguments.
    2019      *     @deprecated bool $hide_user No longer used.
    2020      * }
     2025     * @param string $args Unused. Left over from an earlier implementation.
     2026     *
    20212027     * @return mixed False on failure, otherwise the activity parent content
    20222028     */
    … …  
    20242030        global $activities_template;
    20252031
    2026         $defaults = array(
    2027             'hide_user' => false
    2028         );
    2029 
    2030         $r = wp_parse_args( $args, $defaults );
    2031         extract( $r, EXTR_SKIP );
     2032        // Bail if no activity on no item ID
     2033        if ( empty( $activities_template->activity ) || empty( $activities_template->activity->item_id ) ) {
     2034            return false;
     2035        }
    20322036
    20332037        // Get the ID of the parent activity content
    2034         if ( !$parent_id = $activities_template->activity->item_id )
     2038        $parent_id = (int) $activities_template->activity->item_id;
     2039
     2040        // Bail if no parent content
     2041        if ( empty( $activities_template->activity_parents[ $parent_id ] ) ) {
    20352042            return false;
    2036 
    2037         // Bail if no parent content
    2038         if ( empty( $activities_template->activity_parents[$parent_id] ) )
     2043        }
     2044
     2045        // Bail if no action
     2046        if ( empty( $activities_template->activity_parents[ $parent_id ]->action ) ) {
    20392047            return false;
    2040 
    2041         // Bail if no action
    2042         if ( empty( $activities_template->activity_parents[$parent_id]->action ) )
    2043             return false;
     2048        }
    20442049
    20452050        // Content always includes action
    2046         $content = $activities_template->activity_parents[$parent_id]->action;
     2051        $content = $activities_template->activity_parents[ $parent_id ]->action;
    20472052
    20482053        // Maybe append activity content, if it exists
    2049         if ( ! empty( $activities_template->activity_parents[$parent_id]->content ) )
    2050             $content .= ' ' . $activities_template->activity_parents[$parent_id]->content;
     2054        if ( ! empty( $activities_template->activity_parents[ $parent_id ]->content ) ) {
     2055            $content .= ' ' . $activities_template->activity_parents[ $parent_id ]->content;
     2056        }
    20512057
    20522058        // Remove the time since content for backwards compatibility
    … …  
    20892095
    20902096        // Bail if no activity on no item ID
    2091         if ( empty( $activities_template->activity ) || empty( $activities_template->activity->item_id ) )
     2097        if ( empty( $activities_template->activity ) || empty( $activities_template->activity->item_id ) ) {
    20922098            return false;
     2099        }
    20932100
    20942101        // Get the ID of the parent activity content
    2095         if ( !$parent_id = $activities_template->activity->item_id )
     2102        $parent_id = (int) $activities_template->activity->item_id;
     2103
     2104        // Bail if no parent item
     2105        if ( empty( $activities_template->activity_parents[ $parent_id ] ) ) {
    20962106            return false;
    2097 
    2098         // Bail if no parent item
    2099         if ( empty( $activities_template->activity_parents[$parent_id] ) )
     2107        }
     2108
     2109        // Bail if no parent user ID
     2110        if ( empty( $activities_template->activity_parents[ $parent_id ]->user_id ) ) {
    21002111            return false;
    2101 
    2102         // Bail if no parent user ID
    2103         if ( empty( $activities_template->activity_parents[$parent_id]->user_id ) )
    2104             return false;
    2105 
    2106         $retval = $activities_template->activity_parents[$parent_id]->user_id;
     2112        }
     2113
     2114        $retval = $activities_template->activity_parents[ $parent_id ]->user_id;
    21072115
    21082116        /**
    … …  
    21572165 * @todo deprecate $args param
    21582166 *
    2159  * @param string $args Unused. Appears to be left over from an earlier implementation.
     2167 * @param array $args See {@link bp_activity_get_comments} for description.
    21602168 */
    21612169function bp_activity_comments( $args = '' ) {
    … …  
    21732181     *    this function can probably be streamlined or removed.
    21742182     *
    2175      * @param string $args Unused. Appears to be left over from an earlier implementation.
     2183     * @param string $args Unused. Left over from an earlier implementation.
    21762184     *
    21772185     * @global object $activities_template {@link BP_Activity_Template}
    … …  
    21812189        global $activities_template;
    21822190
    2183         if ( !isset( $activities_template->activity->children ) || !$activities_template->activity->children )
     2191        if ( empty( $activities_template->activity->children ) ) {
    21842192            return false;
     2193        }
    21852194
    21862195        bp_activity_recurse_comments( $activities_template->activity );
    … …  
    22032212            global $activities_template;
    22042213
    2205             if ( empty( $comment ) )
     2214            if ( empty( $comment ) ) {
    22062215                return false;
    2207 
    2208             if ( empty( $comment->children ) )
     2216            }
     2217
     2218            if ( empty( $comment->children ) ) {
    22092219                return false;
     2220            }
    22102221
    22112222            /**
    … …  
    22162227             * @param string $value Opening tag for the HTML markup to use.
    22172228             */
    2218             echo apply_filters( 'bp_activity_recurse_comments_start_ul', '<ul>');
     2229            echo apply_filters( 'bp_activity_recurse_comments_start_ul', '<ul>' );
    22192230            foreach ( (array) $comment->children as $comment_child ) {
     2231
    22202232                // Put the comment into the global so it's available to filters
    22212233                $activities_template->activity->current_comment = $comment_child;
    … …  
    22432255             * @param string $value Closing tag for the HTML markup to use.
    22442256             */
    2245             echo apply_filters( 'bp_activity_recurse_comments_end_ul', '</ul>');
     2257            echo apply_filters( 'bp_activity_recurse_comments_end_ul', '</ul>' );
    22462258        }
    22472259
    … …  
    24552467        global $activities_template;
    24562468
    2457         if ( empty( $activities_template->activity->current_comment->date_recorded ) )
     2469        if ( empty( $activities_template->activity->current_comment->date_recorded ) ) {
    24582470            return false;
     2471        }
    24592472
    24602473        $date_recorded = bp_core_time_since( $activities_template->activity->current_comment->date_recorded );
    … …  
    27242737    function bp_get_activity_comment_form_nojs_display() {
    27252738        global $activities_template;
    2726         if ( isset( $_GET['ac'] ) && $_GET['ac'] == $activities_template->activity->id . '/' )
     2739
     2740        if ( isset( $_GET['ac'] ) && ( $_GET['ac'] === ( $activities_template->activity->id . '/' ) ) ) {
    27272741            return 'style="display: block"';
     2742        }
    27282743
    27292744        return false;
    … …  
    29983013        $class = ' activity-item';
    29993014
    3000         if ( in_array( $activities_template->activity->type, (array) $mini_activity_actions ) || empty( $activities_template->activity->content ) )
     3015        if ( in_array( $activities_template->activity->type, (array) $mini_activity_actions ) || empty( $activities_template->activity->content ) ) {
    30013016            $class .= ' mini';
    3002 
    3003         if ( bp_activity_get_comment_count() && bp_activity_can_comment() )
     3017        }
     3018
     3019        if ( bp_activity_get_comment_count() && bp_activity_can_comment() ) {
    30043020            $class .= ' has-comments';
     3021        }
    30053022
    30063023        /**
    … …  
    31493166    function bp_get_activity_latest_update( $user_id = 0 ) {
    31503167
    3151         if ( empty( $user_id ) )
     3168        if ( empty( $user_id ) ) {
    31523169            $user_id = bp_displayed_user_id();
    3153 
    3154         if ( bp_is_user_inactive( $user_id ) )
     3170        }
     3171
     3172        if ( bp_is_user_inactive( $user_id ) ) {
    31553173            return false;
    3156 
    3157         if ( !$update = bp_get_user_meta( $user_id, 'bp_latest_update', true ) )
     3174        }
     3175
     3176        if ( !$update = bp_get_user_meta( $user_id, 'bp_latest_update', true ) ) {
    31583177            return false;
     3178        }
    31593179
    31603180        /**
    … …  
    32203240    function bp_get_activity_filter_links( $args = false ) {
    32213241
    3222         $defaults = array(
     3242        $r = wp_parse_args( $args, array(
    32233243            'style' => 'list'
    3224         );
    3225 
    3226         $r = wp_parse_args( $args, $defaults );
    3227         extract( $r, EXTR_SKIP );
     3244        ) );
    32283245
    32293246        // Define local variable
    … …  
    32333250        $components = BP_Activity_Activity::get_recorded_components();
    32343251
    3235         if ( empty( $components ) )
     3252        if ( empty( $components ) ) {
    32363253            return false;
     3254        }
    32373255
    32383256        foreach ( (array) $components as $component ) {
    32393257
    32403258            // Skip the activity comment filter
    3241             if ( 'activity' == $component )
     3259            if ( 'activity' == $component ) {
    32423260                continue;
    3243 
    3244             if ( isset( $_GET['afilter'] ) && $component == $_GET['afilter'] )
     3261            }
     3262
     3263            if ( isset( $_GET['afilter'] ) && $component == $_GET['afilter'] ) {
    32453264                $selected = ' class="selected"';
    3246             else
    3247                 unset($selected);
     3265            } else {
     3266                $selected = '';
     3267            }
    32483268
    32493269            $component = esc_attr( $component );
    32503270
    3251             switch ( $style ) {
     3271            switch ( $r['style'] ) {
    32523272                case 'list':
    32533273                    $tag = 'li';
    … …  
    32853305        $link = remove_query_arg( 'afilter' , $link );
    32863306
    3287         if ( isset( $_GET['afilter'] ) )
     3307        if ( isset( $_GET['afilter'] ) ) {
    32883308            $component_links[] = '<' . $tag . ' id="afilter-clear"><a href="' . esc_attr( $link ) . '">' . __( 'Clear Filter', 'buddypress' ) . '</a></' . $tag . '>';
     3309        }
    32893310
    32903311        /**
Note: See TracChangeset for help on using the changeset viewer.