Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#47254 closed defect (bug) (fixed)

i18n: Use %s placeholders in WP_Debug_Data class

Reported by: ramiy's profile ramiy Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-screenshots has-patch
Focuses: administration Cc:

Description

Old translation string:

  • Unable to determine some settings, as the ini_get() function has been disabled.

New translation string:

  • Unable to determine some settings, as the %s function has been disabled.

Translators cam change the function name by mistake while translating the string. The best practice is to replace the ini_get() function with a %s placeholder.

The patch also adds translators comments.

Attachments (2)

47254.png (12.4 KB) - added by ramiy 6 years ago.
47254.patch (738 bytes) - added by ramiy 6 years ago.

Download all attachments as: .zip

Change History (5)

@ramiy
6 years ago

@ramiy
6 years ago

#1 @ramiy
6 years ago

  • Keywords has-screenshots has-patch added

#2 @SergeyBiryukov
6 years ago

  • Milestone changed from Awaiting Review to 5.3
  • Status changed from assigned to reviewing

#3 @SergeyBiryukov
6 years ago

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

In 45399:

I18N: Remove PHP function name from translatable string in WP_Debug_Data.

Props ramiy.
Fixes #47254.

Note: See TracTickets for help on using tickets.