Ticket #3253: 3253.1.diff

bp-activity/bp-activity-template.php

@@ -536 +536 @@
                 global $bp, $activities_template;
 
                 $defaults = array(
-                        'type'   => 'thumb',
-                        'width'  => 20,
-                        'height' => 20,
-                        'class'  => 'avatar',
-                        'alt'    => __( 'Profile picture of %s', 'buddypress' ),
-                        'email'  => false
+                        'type'    => 'thumb',
+                        'width'   => 20,
+                        'height'  => 20,
+                        'class'   => 'avatar',
+                        'alt'     => __( 'Profile picture of %s', 'buddypress' ),
+                        'email'   => false,
+                        'user_id' => false
                 );
 
                 $r = wp_parse_args( $args, $defaults );
@@ -552 +553 @@
                 $item_id = apply_filters( 'bp_get_activity_avatar_item_id', $activities_template->activity->user_id );
 
                 // If this is a user object pass the users' email address for Gravatar so we don't have to refetch it.
-                if ( 'user' == $object && empty( $email ) && isset( $activities_template->activity->user_email ) )
+                if ( 'user' == $object && empty( $user_id ) && empty( $email ) && isset( $activities_template->activity->user_email ) )
                         $email = $activities_template->activity->user_email;
 
                 return apply_filters( 'bp_get_activity_avatar', bp_core_fetch_avatar( array( 'item_id' => $item_id, 'object' => $object, 'type' => $type, 'alt' => $alt, 'class' => $class, 'width' => $width, 'height' => $height, 'email' => $email ) ) );
@@ -780 +781 @@
 }
         function bp_activity_get_comments( $args = '' ) {
                 global $activities_template, $bp;
-
+                        
                 if ( !isset( $activities_template->activity->children ) || !$activities_template->activity->children )
                         return false;
 
@@ -791 +792 @@
                 // TODO: The HTML in this function is temporary and will be moved
                 // to the template in a future version
                 function bp_activity_recurse_comments( $comment ) {
-                        global $activities_template, $bp;
-
+                        global $activities_template, $bp, $counter;
+                        
+                        if ( !$comment )
+                                return false;
+                                
                         if ( !$comment->children )
                                 return false;
 
-                        $content = '<ul>';
+                        echo '<ul>';
                         foreach ( (array)$comment->children as $comment_child ) {
                                 // Put the comment into the global so it's available to filters
                                 $activities_template->activity->current_comment = $comment_child;
 
-                                if ( empty( $comment_child->user_fullname ) )
-                                        $comment_child->user_fullname = $comment_child->display_name;
-
-                                if ( empty( $comment_child->user_nicename ) )
-                                        $comment_child->user_nicename = '';
-
-                                if ( empty( $comment_child->user_login ) )
-                                        $comment_child->user_login = '';
-
-                                if ( empty( $comment_child->user_email ) )
-                                        $comment_child->user_email = '';
-
-                                $content .= '<li id="acomment-' . $comment_child->id . '">';
-                                $content .= '<div class="acomment-avatar"><a href="' .
-                                        bp_core_get_user_domain(
-                                                $comment_child->user_id,
-                                                $comment_child->user_nicename,
-                                                $comment_child->user_login ) . '">' .
-                                        bp_core_fetch_avatar( array(
-                                                'alt'    => __( 'Profile picture of %s', 'buddypress' ),
-                                                'item_id' => $comment_child->user_id,
-                                                'width'   => 30,
-                                                'height'  => 30,
-                                                'email'   => $comment_child->user_email
-                                        ) ) .
-                                '</a></div>';
-
-                                $content .= '<div class="acomment-meta"><a href="' .
-                                        bp_core_get_user_domain(
-                                                $comment_child->user_id,
-                                                $comment_child->user_nicename,
-                                                $comment_child->user_login ) . '">' .
-                                        apply_filters( 'bp_acomment_name', $comment_child->user_fullname, $comment_child ) .
-                                '</a> &middot; ' . sprintf( __( '%s ago', 'buddypress' ), bp_core_time_since( $comment_child->date_recorded ) );
-
-                                // Reply link - the span is so that threaded reply links can be
-                                // hidden when JS is off.
-                                if ( is_user_logged_in() && bp_activity_can_comment_reply( $comment ) )
-                                        $content .= apply_filters( 'bp_activity_comment_reply_link', '<span class="acomment-replylink"> &middot; <a href="#acomment-' . $comment_child->id . '" class="acomment-reply" id="acomment-reply-' . $comment->id . '">' . __( 'Reply', 'buddypress' ) . '</a></span>', $comment_child );
-
-                                // Delete link
-                                if ( $bp->loggedin_user->is_super_admin || $bp->loggedin_user->id == $comment->user_id ) {
-                                        $delete_url = wp_nonce_url( bp_get_root_domain() . '/' . $bp->activity->slug . '/delete/?cid=' . $comment_child->id, 'bp_activity_delete_link' );
-                                        $content .= apply_filters( 'bp_activity_comment_delete_link', ' &middot; <a href="' . $delete_url . '" class="delete acomment-delete confirm" rel="nofollow">' . __( 'Delete', 'buddypress' ) . '</a>', $comment_child, $delete_url );
-                                }
-
-                                $content .= '</div>';
-                                $content .= '<div class="acomment-content">' . apply_filters( 'bp_get_activity_content', $comment_child->content ) . '</div>';
-
-                                $content .= bp_activity_recurse_comments( $comment_child );
-                                $content .= '</li>';
-
-                                // Unset in the global in case of the last iteration
+                                locate_template( 'activity/comment.php', true, false );
+                                
                                 unset( $activities_template->activity->current_comment );
                         }
-                        $content .= '</ul>';
+                        echo '</ul>';
 
-                        return apply_filters( 'bp_activity_recurse_comments', $content );
+                        //return apply_filters( 'bp_activity_recurse_comments', $content );
                 }
 
+function bp_activity_current_comment() {
+        global $activities_template;
+        
+        $current_comment = !empty( $activities_template->activity->current_comment ) ? $activities_template->activity->current_comment : false;
+        
+        return apply_filters( 'bp_activity_current_comment', $current_comment );
+}
+
+function bp_activity_comment_id() {
+        echo bp_get_activity_comment_id();
+}
+        function bp_get_activity_comment_id() {
+                global $activities_template;
+                
+                $comment_id = isset( $activities_template->activity->current_comment->id ) ? $activities_template->activity->current_comment->id : false;
+        
+                return apply_filters( 'bp_activity_comment_id', $comment_id );
+        }
+
+function bp_activity_comment_user_id() {
+        echo bp_get_activity_comment_user_id();
+}
+        function bp_get_activity_comment_user_id() {
+                global $activities_template;
+                
+                $user_id = isset( $activities_template->activity->current_comment->user_id ) ? $activities_template->activity->current_comment->user_id : false;
+                
+                return apply_filters( 'bp_activity_comment_user_id', $user_id );
+        }
+
+function bp_activity_comment_user_link() {
+        echo bp_get_activity_comment_user_link();
+}
+        function bp_get_activity_comment_user_link() {
+                $user_link = bp_core_get_user_domain( bp_get_activity_comment_user_id() );
+        
+                return apply_filters( 'bp_activity_comment_user_link', $user_link );
+        }
+
+function bp_activity_comment_name() {
+        echo bp_get_activity_comment_name();
+}
+        function bp_get_activity_comment_name() {
+                global $activities_template;
+                
+                $name = apply_filters( 'bp_acomment_name', $activities_template->activity->current_comment->user_fullname, $activities_template->activity->current_comment ); // bp_acomment_name is for backward compatibility
+                
+                return apply_filters( 'bp_activity_comment_name', $name );
+        }
+
+function bp_activity_comment_date_recorded() {
+        echo bp_get_activity_comment_date_recorded();
+}
+        function bp_get_activity_comment_date_recorded() {
+                global $activities_template;
+                
+                if ( empty( $activities_template->activity->current_comment->date_recorded ) )
+                        return false;
+                
+                $date_recorded = sprintf( __( '%s ago', 'buddypress' ), bp_core_time_since( $activities_template->activity->current_comment->date_recorded ) );
+                
+                return apply_filters( 'bp_activity_comment_date_recorded', $date_recorded );
+        }
+
+function bp_activity_comment_delete_link() {
+        echo bp_get_activity_comment_delete_link();
+}
+        function bp_get_activity_comment_delete_link() {
+                global $bp;
+                
+                $link = wp_nonce_url( bp_get_root_domain() . '/' . $bp->activity->slug . '/delete/?cid=' . bp_get_activity_comment_id(), 'bp_activity_delete_link' );
+                
+                return apply_filters( 'bp_activity_comment_delete_link', $link );
+        }
+
+function bp_activity_comment_content() {
+        echo bp_get_activity_comment_content();
+}
+        function bp_get_activity_comment_content() {
+                global $activities_template;
+                
+                $content = apply_filters( 'bp_get_activity_content', $activities_template->activity->current_comment->content );
+                
+                return apply_filters( 'bp_activity_comment_content', $content );
+        }
+
 function bp_activity_comment_count() {
         echo bp_activity_get_comment_count();
 }

bp-themes/bp-default/activity/entry.php

@@ -80 +80 @@
 
         <?php if ( bp_activity_can_comment() ) : ?>
                 <div class="activity-comments">
-
+                        
                         <?php bp_activity_comments(); ?>
 
                         <?php if ( is_user_logged_in() ) : ?>