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

[WordPress Trac]

#23148: Add hook in wp_ajax_save_attachment for additional attachment fields
------------------------------------------------+-----------------------
 Reporter:  griffinjt                           |       Owner:  audrasjb
     Type:  enhancement                         |      Status:  reopened
 Priority:  normal                              |   Milestone:  6.2
Component:  Media                               |     Version:  3.5
 Severity:  normal                              |  Resolution:
 Keywords:  has-patch 2nd-opinion dev-feedback  |     Focuses:
------------------------------------------------+-----------------------
Changes (by costdev):

 * keywords:  has-patch 2nd-opinion changes-requested => has-patch 2nd-
     opinion dev-feedback


Comment:

 @azaozz I have submitted [https://github.com/WordPress/wordpress-
 develop/pull/4151 PR 4151] which renames the `wp_ajax_save_attachment`
 action to `wp_ajax_save_attachment_updated` which is more in line with
 when the action is specifically fired.

 I've also added a note in the docblock that when checking if an action is
 being done, `doing_action( 'wp_ajax_save-attachment' )` may be used if
 that is more convenient.

 How do these changes sound to you?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/23148#comment:46>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform