embed_oembed_html not used on sitewide activity?

[petervandoorn]

I’m using this code ( from ​http://wpsites.org/how-to-make-oembed-videos-responsive-10520 ) to filter WordPress’s oEmbed to make YouTube videos responsive.

add_filter('embed_oembed_html', 'ct_wrap_embed_container', 10, 3);
 
function ct_wrap_embed_container($html, $url, $attr) {
        return "<div class=\"responsive-video\">".$html."</div>";
}

It works a treat in bbPress forum topic views, but it isn’t working on the BP site-wide activity feed.

This Codex page ​https://codex.buddypress.org/getting-started/embeds/ says that WP’s native oEmbed functionality is used since BP 1.5, so I would expect it to work.

Thanks

[DJPaul]

Hi. What isn't working, exactly?

Are the videos appearing in the activity stream? If so, does your wrapper HTML appear around the activity item? If so, are you reporting this is basically a CSS/responsive issue?

[henry.wright]

Hi Paul

I think the solution would be for BuddyPress to apply filters attached to the WordPress hook embed_oembed_html?

[r-a-y]

BP doesn't use the 'embed_oembed_html' filter.

I originally did use this filter when implementing embeds into BuddyPress, but it was decided that we should prefix the filter name with 'bp_'.

Try the 'bp_embed_oembed_html' filter instead.

[henry.wright]

Try the 'bp_embed_oembed_html' filter instead.

And there we have it. Thanks r-a-y :)

[DJPaul]

@r-a-y How do you feel about calling this action in addition to ours? I ended up doing this for the email code with wp_mail, and I think it'd help more here, too.

[r-a-y]

@r-a-y How do you feel about calling this action in addition to ours? I ended up doing this for the email code with wp_mail, and I think it'd help more here, too.

The problem is the 'embed_oembed_html' filter doesn't quite have the same filter signature. The 'embed_oembed_html' filter has a fourth parameter used for the WP post ID, but we do not utilize the item ID in our 'bp_embed_oembed_html' filter -- we use the raw shortcode attributes instead.

In 01.patch, I've implemented the 'embed_oembed_html' filter, but for the fourth parameter, I've just passed integer 0 in its place.

I'll leave it up to you if we want to commit this or not as I don't really have any strong opinions about this ticket.

[hnla]

Re-visit in 3.0 then.

[DJPaul]

Let's not do this per previous discussion between @r-a-y and I. Worried about breaking the function signature here, that's annoying to run into in someone else's codebase.