BP oEmbed should switch auto discovery to `true`

[r-a-y]

As imath noted in ticket:6772#comment:9, WordPress switched their oEmbed discover switch to true in WordPress 4.4.0.

This allows WP to embed posts automatically into the WP TinyMCE editor without having to register themselves as an oEmbed provider.

We should follow suit.

Maybe we should change our custom prefixed filter to just use WP's 'embed_oembed_discover' instead.

[DJPaul]

Happy with your patch but if you want a crazy idea: we could pass their filter's value to our filter.

[imath]

Thanks r-a-y

Just tested the patch, and it's now possible to embed WordPress posts into the activity stream.

I'm not sure about changing the filter. I guess there can be edge cases where Admins want to disable embeds in BuddyPress and not in WordPress and vice and versa :)

[imath]

I think we should only change this part if the WordPress installed version is 4.4+. See 6778.02.patch.

[r-a-y]

imath - I like your idea way better than mine! I would say commit it.

[imath]

In 10548:

Allow oEmbed discovery when WordPress version is 4.4+

WordPress 4.4 introduced Post embeds, allowing oEmbed discovery by default. We are using the same behavior for configs using at least WordPress 4.4 so that Post urls can be converted to Post embeds into the activity stream.

Props r-a-y

Fixes #6778