[wp-trac] [WordPress Trac] #17578: Merge media buttons into one

WordPress Trac wp-trac at lists.automattic.com
Fri Sep 30 01:08:54 UTC 2011


#17578: Merge media buttons into one
------------------------------------+------------------
 Reporter:  mitchoyoshitaka         |       Owner:
     Type:  defect (bug)            |      Status:  new
 Priority:  normal                  |   Milestone:  3.3
Component:  Media                   |     Version:
 Severity:  trivial                 |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+------------------

Comment (by nacin):

 Only things left really are to study the restructuring of functions.

  * A number of functions have been removed. These can become deprecated
 wrappers.

  * The remaining functions, like type_url_form_image() and
 media_upload_image(), should probably be renamed, and turned into
 deprecated wrappers.

  * We should grep through the plugins directory and see how plugins are
 using the various functions and filters at play here.

 Also, Ryan suggested we refactor mu_media_buttons into a single checkbox
 for enabling/disabling uploads, as that's what people use it for. (Even
 thought it doesn't do that currently, and never blocked anything.) Seems
 fine to me, though we'll need to probably hook in a bit deeper in spots
 (like actually restricting things from being uploaded).

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


More information about the wp-trac mailing list