WordPress.org
Get WordPress

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#47159 closed defect (bug) (fixed)

Do not use placeholders for HTML tags in strings

Reported by: dimadin Owned by: SergeyBiryukov
Priority: normal Milestone: 5.2.1
Component: I18N Version: 5.2
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

[45158] introduced string where placeholders are used instead of HTML tags. That practice is not used in WordPress core, and it's bad anyway.

Attached patch uses approach used in other strings where it puts link and additional link attributes behind placeholders.

Note that this text is ​used in Gutenberg (introduced in ​#14668), however because of the way links are formed in Gutenberg, same strings can't be used.

Attachments (1)

47159.diff​ (2.1 KB ) - added by dimadin 7 years ago.

Download all attachments as: .zip

Change History (5)

@dimadin
7 years ago

#1 @SergeyBiryukov
7 years ago

  • Milestone Awaiting Review → 5.2.1
  • Owner set to SergeyBiryukov
  • Status new → reviewing

#2 @ramiy
7 years ago

Big like!

#3 @SergeyBiryukov
7 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 45334:

I18N: Avoid using placeholders for link start and end tags in alt text description strings added in [45158].

Use an established pattern of moving additional link attributes out of the translatable string.

Props dimadin.
Fixes #47159.

#4 @SergeyBiryukov
7 years ago

In 45335:

I18N: Avoid using placeholders for link start and end tags in alt text description strings added in [45158].

Use an established pattern of moving additional link attributes out of the translatable string.

Props dimadin.
Merges [45334] to the 5.2 branch.
Fixes #47159.

Note: See TracTickets for help on using tickets.