Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#26783 closed defect (bug) (fixed)

Wrong behavior of the deletion link in the Media Library window when MEDIA_TRASH is set to 'true'

Reported by: toszcze's profile toszcze Owned by: wonderboymusic's profile wonderboymusic
Milestone: 3.9 Priority: normal
Severity: normal Version: 3.9
Component: Media Keywords: has-patch
Focuses: javascript Cc:


When the trash for attachments is enabled (MEDIA_TRASH constant is set to 'true'), the link for deleting the attachment in the Media Library window still says 'Delete Permanently' instead of 'Trash'. Also clicking this link displays an incorrect confirmation message.

Attachments (2)

media_trash_delete.png (43.6 KB) - added by toszcze 10 years ago.
26783.diff (3.1 KB) - added by toszcze 10 years ago.

Download all attachments as: .zip

Change History (6)

10 years ago

#1 @toszcze
10 years ago

  • Keywords has-patch added

#2 @wonderboymusic
10 years ago

  • Milestone changed from Awaiting Review to 3.9

Sounds like a good idea.

#3 @wonderboymusic
10 years ago

  • Focuses javascript added

#4 @wonderboymusic
10 years ago

  • Owner set to wonderboymusic
  • Resolution set to fixed
  • Status changed from new to closed

In 27454:

When MEDIA_TRASH is true, show the proper context in the attachment-details sidebar. Without this fix, it always showed "Delete Permanently."

Props toszcze.
Fixes #26783.

Note: See TracTickets for help on using tickets.