Ticket #3429: 3429.01.patch

bp-activity/bp-activity-functions.php

@@ -923 +923 @@
         if ( isset( $activity_obj->current_comment ) ) {
                 $activity_obj = $activity_obj->current_comment;
         }
-        
+
         if ( 'new_blog_post' == $activity_obj->type || 'new_blog_comment' == $activity_obj->type || 'new_forum_topic' == $activity_obj->type || 'new_forum_post' == $activity_obj->type )
                 $link = $activity_obj->primary_link;
         else {
@@ -994 +994 @@
 /** Embeds *******************************************************************/
 
 /**
- * Grabs the activity ID and attempts to retrieve the oEmbed cache (if it exists)
+ * Grabs the activity update ID and attempts to retrieve the oEmbed cache (if it exists)
  * during the activity loop.  If no cache and link is embeddable, cache it.
  *
+ * This does not cover recursive activity comments, as they do not use a real loop.
+ * For that, see {@link bp_activity_comment_embed()}.
+ *
  * @see BP_Embed
  * @see bp_embed_activity_cache()
  * @see bp_embed_activity_save_cache()
@@ -1011 +1014 @@
 add_action( 'activity_loop_start', 'bp_activity_embed' );
 
 /**
+ * Grabs the activity comment ID and attempts to retrieve the oEmbed cache (if it exists)
+ * when BP is recursing through activity comments {@link bp_activity_recurse_comments()}.
+ * If no cache and link is embeddable, cache it.
+ *
+ * @see BP_Embed
+ * @see bp_embed_activity_cache()
+ * @see bp_embed_activity_save_cache()
+ * @package BuddyPress Activity
+ * @since 1.5
+ */
+function bp_activity_comment_embed() {
+        add_filter( 'embed_post_id',            'bp_get_activity_comment_id' );
+        add_filter( 'bp_embed_get_cache',       'bp_embed_activity_cache',      10, 3 );
+        add_action( 'bp_embed_update_cache',    'bp_embed_activity_save_cache', 10, 3 );
+}
+add_action( 'bp_before_activity_comment', 'bp_activity_comment_embed' );
+
+/**
+ * Removes the 'embed_post_id' filter after {@link bp_activity_recurse_comments()}
+ * is rendered to avoid conflict with the 'embed_post_id' filter in
+ * {@link bp_activity_embed()} or any other component embeds.
+ *
+ * @see bp_activity_comment_embed()
+ * @package BuddyPress Activity
+ * @since 1.5
+ */
+function bp_activity_comment_after_recurse() {
+        remove_filter( 'embed_post_id',         'bp_get_activity_comment_id' );
+}
+add_action( 'bp_after_activity_comment', 'bp_activity_comment_after_recurse' );
+
+/**
  * Wrapper function for {@link bp_activity_get_meta()}.
  * Used during {@link BP_Embed::parse_oembed()} via {@link bp_activity_embed()}.
  *

bp-core/bp-core-classes.php

@@ -1177 +1177 @@
                 }
 
                 // Get object ID
+                $id = '';
                 $id = apply_filters( 'embed_post_id', $id );
 
                 // Is oEmbed discovery on?
@@ -1229 +1230 @@
 
                         // Grab cache and return it if available
                         if ( !empty( $cache ) ) {
-                                return apply_filters( 'embed_oembed_html', $cache, $url, $attr, $rawattr );
+                                return apply_filters( 'bp_embed_oembed_html', $cache, $url, $attr, $rawattr );
 
                         // If no cache, ping the oEmbed provider and cache the result
                         } else {

bp-themes/bp-default/_inc/ajax.php

@@ -339 +339 @@
         if ( !$activity )
                 exit(); // todo: error?
 
+        do_action( 'bp_activity_action_get_single_activity_content', $_POST['activity_id'] );
+
         // Activity content retrieved through AJAX should run through normal filters, but not be
         // truncated
         remove_filter( 'bp_get_activity_content_body', 'bp_activity_truncate_entry', 5 );