WordPress.org

Make WordPress Core

#22047 closed enhancement (duplicate)

Do action 'save_post' for attachments in 'wp_insert_attachment()'

Reported by: dnaber-de Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: has-patch
Focuses: Cc:

Description

I'm currently change my plugins to use meta boxes instead of the filter 'attachment_fields_to_edit' to add fields to the attachment edit screen. Adding meta boxes works as expected like for all other post types.

They usual way to evaluate meta boxes is a callback on 'save_post' for all other post-types. At the moment there are two actions for the attachment post type: 'add_attachment' and 'edit_attachment'. I think it would be consistent to do the action 'save_post' also in 'wp_insert_attachment()'. Or at least provide an action like 'save_attachment'.

Attachments (1)

22407.patch (1.5 KB) - added by c3mdigital 22 months ago.
adds save_post and edit_post actions to wp_insert_attachment

Download all attachments as: .zip

Change History (5)

comment:1 DrewAPicture22 months ago

Related: #21391 - Move attachment editing (media.php) to use standard post type UI

comment:2 ocean9022 months ago

  • Component changed from General to Media
  • Keywords needs-patch added; dev-feedback 2nd-opinion removed
  • Milestone changed from Awaiting Review to 3.5

c3mdigital22 months ago

adds save_post and edit_post actions to wp_insert_attachment

comment:3 c3mdigital22 months ago

  • Keywords has-patch added; needs-patch removed

comment:4 nacin22 months ago

  • Milestone 3.5 deleted
  • Resolution set to duplicate
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.