[wp-trac] [WordPress Trac] #23148: Add hook in wp_ajax_save_attachment for additional attachment fields

WordPress Trac noreply at wordpress.org
Fri Feb 15 19:00:05 UTC 2013


#23148: Add hook in wp_ajax_save_attachment for additional attachment fields
-------------------------+------------------
 Reporter:  griffinjt    |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  3.6
Component:  Media        |     Version:  3.5
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+------------------

Comment (by griffinjt):

 Replying to [comment:12 bradyvercher]:
 > Any fields added using the backward compatible filter
 `attachment_fields_to_edit` ... These fields can also be registered to
 work on the new "Edit Media" screens, so I'm not sure a hook is really
 necessary here.

 Red flags should be thrown up when you are required to use a backwards
 compatible filter with a new API. I disagree and believe this hook is
 necessary to promote good and future-minded code with the new media APIs.
 Beyond this, the new API is JavaScript, and I don't see promoting an
 entirely PHP solution as a good thing moving forward. This hook simply
 opens up the "model" portion of Backbone so that it can be utilized by
 other people properly extending Backbone's other APIs.

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


More information about the wp-trac mailing list