[wp-trac] [WordPress Trac] #21719: Remove autoembed_urls, embed_size_w, embed_size_h
WordPress Trac
wp-trac at lists.automattic.com
Tue Sep 25 06:45:13 UTC 2012
#21719: Remove autoembed_urls, embed_size_w, embed_size_h
-------------------------+------------------
Reporter: nacin | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.5
Component: Embeds | Version: 2.9
Severity: normal | Resolution:
Keywords: has-patch |
-------------------------+------------------
Comment (by nacin):
[attachment:remove-auto-embeds.diff] looks good.
I am debating as to whether we should still listen to existing options,
even while hiding the UI (and thus preventing those options from being
changed); or not hiding the UI if the existing options are set to a non-
default value; or ignoring all options and clearing them out, as
[attachment:remove-auto-embeds.diff] does.
I think for 3.5 beta 1, we should implement this minus the schema change
to actually clear them out of the DB, in case we find the desire to roll
them back. We could also consider leaving them in the DB in case anyone is
using them directly for some esoteric reason, though the default filters
are a nice touch. (We could also return wp_embed_defaults() through those
option calls, too.)
--
Ticket URL: <http://core.trac.wordpress.org/ticket/21719#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list