[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