Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#35679 closed defect (bug) (fixed)

Avoid using HTML tags in translation strings (wp-admin/plugins.php)

Reported by: ramiy's profile ramiy Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.5 Priority: normal
Severity: normal Version:
Component: I18N Keywords: has-patch
Focuses: Cc:

Description

See the attached patch.

Attachments (2)

35679.patch (1.4 KB) - added by ramiy 8 years ago.
35679.2.patch (1.4 KB) - added by ramiy 8 years ago.

Download all attachments as: .zip

Change History (11)

@ramiy
8 years ago

@ramiy
8 years ago

#1 @ramiy
8 years ago

  • Keywords has-patch added

#2 @ramiy
8 years ago

Old strings:

  • <strong>%1$s</strong> by <em>%2$s</em> (will also <strong>delete its data</strong>)
  • <strong>%1$s</strong> by <em>%2$s</em>

New strings:

  • %1$s by %2$s
  • (will also <strong>delete its data</strong>)

This ticket was mentioned in Slack in #core-i18n by ramiy. View the logs.


8 years ago

This ticket was mentioned in Slack in #core-i18n by akirk. View the logs.


8 years ago

#5 @SergeyBiryukov
8 years ago

(will also <strong>delete its data</strong>) is confusing by itself (what would delete its data)?

I would suggest just moving the tags into placeholders:

  • %1$s by %2$s (will also <strong>delete its data</strong>)
  • %1$s by %2$s

The latter string also probably needs a context, to avoid clashing with the one in media_handle_upload() if we remove the trailing dot there at some point.

#6 @ramiy
8 years ago

  • Keywords close added

Ok, close this ticket.

#7 @SergeyBiryukov
8 years ago

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

In 36662:

I18N: Remove HTML tags from translatable strings in wp-admin/plugins.php.

Props ramiy.
Fixes #35679.

#8 @SergeyBiryukov
8 years ago

  • Milestone changed from Awaiting Review to 4.5

#9 @ocean90
8 years ago

  • Keywords close removed
Note: See TracTickets for help on using tickets.