Context Navigation

← Previous Ticket
Next Ticket →

#22578 closed defect (bug) (fixed)

'attachment_fields_to_edit' not fired from the new "Add media" modal

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

Description

in WP 3.5 RC1, if I directly upload file(s), the attachment details is displayed without fields created by 'attachment_fields_to_edit'

However, if I first go to the library, the fields are correctly displayed.

Attachments (1)

22578.diff (717 bytes ) - added by koopersmith 14 years ago.

Download all attachments as: .zip

Change History (6)

#1 @ocean90
14 years ago

Milestone Awaiting Review → 3.5
Owner set to koopersmith
Status new → reviewing

@koopersmith
14 years ago

Attachment 22578.diff added

#2 @koopersmith
14 years ago

Keywords has-patch added

Always render the attachment compat view, as it already handles the empty case.

#3 @nacin
14 years ago

Keywords commit added

Tested, including logging the fired filters before and after. Thumbs up.

#4 @ryan
14 years ago

Resolution → fixed
Status reviewing → closed

In 22841:

Always render the attachment compat view. Assures attachment_fields_to_edit is always fired.

Props koopersmith, Chouby
fixes #22578

#5 @nacin
14 years ago

#22629 was marked as a duplicate.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#22578 closed defect (bug) (fixed)

'attachment_fields_to_edit' not fired from the new "Add media" modal

Description

Attachments (1)

Change History (6)

#1 @ocean90 14 years ago

@koopersmith 14 years ago

#2 @koopersmith 14 years ago

#3 @nacin 14 years ago

#4 @ryan 14 years ago

#5 @nacin 14 years ago