Make WordPress Core

Opened 13 years ago

Closed 10 years ago

#20151 closed enhancement (fixed)

Add parameter for wp_embed_defaults()

Reported by: wpsmith's profile wpsmith Owned by: sergeybiryukov's profile 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)

wp_embed_defaults.patch (2.6 KB) - added by wpsmith 13 years ago.
20151.diff (2.6 KB) - added by sabreuse 13 years ago.
20151.1.diff (2.1 KB) - added by jacobdubail 10 years ago.
20151.2.diff (2.1 KB) - added by jacobdubail 10 years ago.
20151.3.diff (2.6 KB) - added by JasonM4563 10 years ago.
reapplied changes into existing code and added some documentation

Download all attachments as: .zip

Change History (15)

#1 @ocean90
13 years ago

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

#2 follow-up: @wpsmith
13 years ago

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

@sabreuse
13 years ago

#3 @sabreuse
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 @DrewAPicture
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.

Last edited 13 years ago by DrewAPicture (previous) (diff)

#5 @wonderboymusic
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: @DrewAPicture
10 years ago

  • Keywords good-first-bug added

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

@jacobdubail
10 years ago

@jacobdubail
10 years ago

#7 in reply to: ↑ 6 @jacobdubail
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: @DrewAPicture
10 years ago

  • Keywords commit added; needs-refresh needs-docs removed
  • Milestone changed from Future Release to 4.0

Docs look good.

@JasonM4563
10 years ago

reapplied changes into existing code and added some documentation

#9 in reply to: ↑ 8 @jacobdubail
10 years ago

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?

Last edited 10 years ago by jacobdubail (previous) (diff)

#10 @SergeyBiryukov
10 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 28923:

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

props wpsmith, sabreuse, jacobdubail.
fixes #20151.

Note: See TracTickets for help on using tickets.