WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#47406 closed defect (bug) (fixed)

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

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

Description

See the attached patch.

Attachments (2)

47406.png​ (10.1 KB) - added by ramiy 7 years ago.
47406.patch​ (1.5 KB) - added by ramiy 7 years ago.

Download all attachments as: .zip

Change History (5)

@ramiy
7 years ago

@ramiy
7 years ago

#1 @ramiy
7 years 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
7 years ago

  • Milestone changed from Awaiting Review to 5.3

#3 @SergeyBiryukov
7 years 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.