Make WordPress Core

Opened 10 months ago

Closed 9 months ago

#47406 closed defect (bug) (fixed)

i18n: Use %s placeholders in wp-admin/includes/image-edit.php

Reported by: ramiy Owned by: SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Media Keywords: has-screenshots has-patch
Focuses: administration Cc:


See the attached patch.

Attachments (2)

47406.png (10.1 KB) - added by ramiy 10 months ago.
47406.patch (1.5 KB) - added by ramiy 10 months ago.

Download all attachments as: .zip

Change History (5)

10 months ago

10 months ago

#1 @ramiy
10 months ago

  • Keywords has-screenshots has-patch added

Simple update, replace the variable name and the class name with %1$s and %2$s placeholders.

#2 @SergeyBiryukov
10 months ago

  • Milestone changed from Awaiting Review to 5.3

#3 @SergeyBiryukov
9 months ago

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

In 45615:

I18N: Move code out of translatable string in a _deprecated_argument() message in wp_stream_image(), wp_save_image_file(), and image_edit_apply_changes().

Props ramiy.
Fixes #47406.

Note: See TracTickets for help on using tickets.