Opened 13 years ago
Closed 10 years ago
#20151 closed enhancement (fixed)
Add parameter for wp_embed_defaults()
Reported by: | wpsmith | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 4.0 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Media | Keywords: | has-patch good-first-bug commit |
Focuses: | Cc: |
Description
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.
Attachments (5)
Change History (15)
#2
follow-up:
↓ 4
@
13 years ago
I'm sorry but I am unsure how to do that. The files are located in wordpress/wp-includes/
#3
@
13 years ago
20151.diff is a refresh of wpsmith's patch, fixed to be relative to root and with trailing CR's removed.
#4
in reply to:
↑ 2
@
13 years ago
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 than from within wp-includes.
#5
@
10 years ago
- Keywords needs-refresh needs-docs added
- Milestone changed from Awaiting Review to Future Release
could go in 4.0 with refresh/docs
#6
follow-up:
↓ 7
@
10 years ago
- Keywords good-first-bug added
Marking good-first-bug for WC Seattle contributor day. See comment:5
#7
in reply to:
↑ 6
@
10 years ago
Replying to DrewAPicture:
Marking good-first-bug for WC Seattle contributor day. See comment:5
Updated docs. Ready for review. Thanks!
#8
follow-up:
↓ 9
@
10 years ago
- Keywords commit added; needs-refresh needs-docs removed
- Milestone changed from Future Release to 4.0
Docs look good.
Could you please make your patch relative to the root dir?