[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