[wp-trac] [WordPress Trac] #21181: simplify and add filter to is_upload_space_available

WordPress Trac wp-trac at lists.automattic.com
Mon Jul 9 15:43:53 UTC 2012


#21181: simplify and add filter to is_upload_space_available
-------------------------+------------------------------
 Reporter:  jkudish      |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  General      |     Version:  3.4.1
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------------------

Comment (by westi):

 Replying to [comment:5 dllh]:
 > Replying to [comment:4 westi]:
 > > I think a better solution here is to add a {{{pre_}}} filter into
 {{{get_upload_space_available}}} so that a replacement method of tracking
 used upload space can easily be plugged into core.
 >
 > 21181.patch adds this functionality.

 Looks good.

 I think there are some other places where we calculate space used directly
 instead of calling a function which need reworking to have filtering
 available too to allow a custom method of measurement.

 e.g. display_space_usage and check_upload_size

 We need to review anywhere where the option to disable is used and come up
 with a suitable filter for accounting for space in a different way.

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


More information about the wp-trac mailing list