Add parameter for wp_embed_defaults()

[wpsmith]

If we can add a parameter (like $url) to wp_embed_defaults(), then people can filter more intelligently as WordPress extends to allow more and more oEmbed.

[ocean90]

Could you please make your patch relative to the root dir?

[wpsmith]

I'm sorry but I am unsure how to do that. The files are located in wordpress/wp-includes/

[sabreuse]

20151.diff is a refresh of wpsmith's patch, fixed to be relative to root and with trailing CR's removed.

[DrewAPicture]

Replying to wpsmith:

I'm sorry but I am unsure how to do that. The files are located in wordpress/wp-includes/

You would generate the diff/patch from the site root (in your case wordpress/) rather from within wp-includes.

[wonderboymusic]

could go in 4.0 with refresh/docs

[DrewAPicture]

Marking good-first-bug for WC Seattle contributor day. See comment:5

[jacobdubail]

Replying to DrewAPicture:

Marking good-first-bug for WC Seattle contributor day. See comment:5

Updated docs. Ready for review. Thanks!

[DrewAPicture]

Docs look good.

[JasonM4563]

reapplied changes into existing code and added some documentation

[jacobdubail]

dh-shredder and I removed the code changes to comment-template.php because we couldn't figure out why they were relevant to this ticket. Could you explain why they were re-added?

[SergeyBiryukov]

In 28923:

Pass $url parameter to wp_embed_defaults() and 'embed_defaults' filter.

props wpsmith, sabreuse, jacobdubail.
fixes #20151.