WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#11402 closed defect (bug) (fixed)

Plural for string in Media Library

Reported by: pavelevap Owned by: nbachiyski
Milestone: 3.0 Priority: normal
Severity: normal Version: 2.9
Component: I18N Keywords: has-patch
Focuses: Cc:

Description

String "Media permanently deleted." (wp-admin/upload.php:207) should be prepared for plural in localized versions. You can delete one, two or five media files at once.

Attachments (2)

11402.diff (773 bytes) - added by nacin 5 years ago.
11402.2.diff (3.5 KB) - added by nacin 5 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 @nacin5 years ago

  • Keywords has-patch added

Quick patch, but we may be getting ahead of ourselves keeping this consistent across messages for other message strings. Related, #11184. Could be 3.0.

@nacin5 years ago

comment:2 @westi5 years ago

  • Keywords needs-patch added; has-patch removed
  • Milestone changed from 2.9 to 3.0

In my tests the $deleted var is only set when you delete a single item and then it is set to 1 for bulk deletes it is not set.

I also don't want another string change now - moving to 3.0 for now.

comment:3 @nacin5 years ago

  • Keywords has-patch added; needs-patch removed

New patch. Refactors deleted, trashed, untrashed actions. I imagine more of media will change later in the 3.0 dev cycle but this is a start to bring it in line with how posts etc. are treated.

@nacin5 years ago

comment:4 @hakre5 years ago

Related: #11515

comment:5 @nacin5 years ago

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

(In [13153]) Improve updated messages in Media Library. Add plural strings, add Undo links for trash. Fixes #11402

Note: See TracTickets for help on using tickets.