[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