[wp-trac] [WordPress Trac] #12855: Add 2 filters on get_media_item(s) functions

WordPress Trac wp-trac at lists.automattic.com
Mon Apr 5 14:17:38 UTC 2010


#12855: Add 2 filters on get_media_item(s) functions
---------------------------+------------------------------------------------
 Reporter:  momo360modena  |       Owner:                         
     Type:  enhancement    |      Status:  new                    
 Priority:  low            |   Milestone:  Future Release         
Component:  Media          |     Version:                         
 Severity:  normal         |    Keywords:  has-patch needs-testing
---------------------------+------------------------------------------------

Comment(by nacin):

 They work for you, but there are larger implications. Are they consistent?
 Are the arguments passed to them in a good order, and are those
 consistent, and are there others that should be added, or ones in there
 now omitted? Would filtering another part of the function, instead of its
 raw output, be a better use of cpu cycles? Are there other functions that
 should get similar filters for consistency?

 We're also planning to rework media/upload significantly in 3.1. Don't
 think we should add a filter now when those functions may very well change
 significantly in a month. That all said, we're not going to add little
 enhancements at this stage, as we're not only far past feature freeze but
 we've now entered beta.

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


More information about the wp-trac mailing list