WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#22577 closed defect (bug) (fixed)

'attachment_fields_to_save' is not fired from the new "Add media" modal

Reported by: chouby's profile Chouby Owned by: koopersmith's profile koopersmith
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: has-patch commit
Focuses: Cc:

Description

In WP 3.5 RC1, it seems that the filter 'attachment_fields_to_save' is never fired from the new "Add media" modal.

In WP 3.4, it was called when clicking on "save all changes" (the button does not exist anymore) and when clicking "insert into post" (this does not work in WP 3.5 RC1).

Attachments (1)

22577.diff​ (521 bytes) - added by koopersmith 12 years ago.

Download all attachments as: .zip

Change History (5)

#1 @ocean90
12 years ago

  • Milestone changed from Awaiting Review to 3.5
  • Owner set to koopersmith
  • Status changed from new to reviewing

​http://wordpress.org/support/topic/cant-update-attachment-fields

@koopersmith
12 years ago

#2 @koopersmith
12 years ago

  • Keywords has-patch added

Correctly access the nonce to save attachment fields. The nonce was moved from l10n to settings when we introduced an ajaxurl setting as well.

Audited the file for any other instances of this bug; this is the only case.

#3 @nacin
12 years ago

  • Keywords commit added

Tested. Logged the ajax request and fired filters before and after. Thumbs up.

#4 @ryan
12 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 22842:

Correctly access the nonce to save attachment fields. Fixes firing attachment_fields_to_save.

Props koopersmith
fixes #22577

Note: See TracTickets for help on using tickets.