[wp-trac] [WordPress Trac] #10337: Easier embeds for 2.9 (oEmbed perhaps?)

WordPress Trac wp-trac at lists.automattic.com
Tue Oct 13 20:28:04 UTC 2009


#10337: Easier embeds for 2.9  (oEmbed perhaps?)
----------------------------+-----------------------------------------------
 Reporter:  ryan            |       Owner:  Viper007Bond 
     Type:  task (blessed)  |      Status:  assigned     
 Priority:  normal          |   Milestone:  2.9          
Component:  Shortcodes      |     Version:               
 Severity:  normal          |    Keywords:  needs-testing
----------------------------+-----------------------------------------------

Comment(by Viper007Bond):

 Replying to [comment:36 ryan]:
 > Can we have Vimeo in the whitelist and still have it do discovery?

 No. Discovery is only used if the URL does not match any of the predefined
 oEmbed providers (that array that's also used as a whitelist of sorts).

 > Also, should this be commented out?
 >
 > {{{
 >                 //case 'embed_size_w':
 > }}}

 Yes, and infact it can probably be removed. It's commented out as you can
 leave the field blank at Settings -> Media and it will default to your
 theme's `$content_width` or if that isn't set, then some pre-defined
 default. If that's uncommented, then it'll end up being `0` rather than
 staying blank.

 A custom sanitizer should probably be used that allows blank OR int and
 nothing else (the value is currently int'ed before use, but no sense in
 having an invalid data type be able to show up in that field).

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10337#comment:38>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list