WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#27889 closed defect (bug) (fixed)

"Installed" string is missing a context

Reported by: SergeyBiryukov Owned by:
Priority: normal Milestone: 4.0
Component: I18N Version: 3.9
Severity: normal Keywords: has-patch commit fixed-major i18n-change
Cc: Focuses:

Description

We use _x( 'Installed', 'plugin' ) for plugins and _x( 'Installed', 'theme' ) for themes:
tags/3.9/src/wp-admin/includes/class-wp-plugin-install-list-table.php#L252​
tags/3.9/src/wp-admin/includes/class-wp-theme-install-list-table.php#L256​

However, one instance (added in [28025]) is missing a context:
tags/3.9/src/wp-admin/theme-install.php#L213​

We also need a context for "Already Installed" string, since it's also used in install_blog():
tags/3.9/src/wp-includes/ms-functions.php#L1372​

Attachments (1)

27889.patch​ (989 bytes ) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (7)

@SergeyBiryukov
12 years ago

#1 @nacin
12 years ago

  • Milestone Awaiting Review → 3.9.1

This ticket was mentioned in IRC in #wordpress-dev by netweb. ​View the logs.
12 years ago

#3 @nacin
12 years ago

In 28160:

Use correct echoing context function. see #27889.

#4 @nacin
12 years ago

  • Keywords commit fixed-major i18n-change added

#5 @nacin
12 years ago

  • Milestone 3.9.1 → 3.9.2

#6 @helen
12 years ago

  • Milestone 3.9.2 → 4.0
  • Resolution → fixed
  • Status new → closed
Note: See TracTickets for help on using tickets.