Making WordPress.org

Opened 6 years ago

Closed 6 years ago

#4196 closed enhancement (fixed)

Using gp_warning_placeholders_re update the GlotPress regex to include ###[A-Z_]+### placeholders for WP emails

Reported by: garrett-eclipse's profile garrett-eclipse Owned by: dd32's profile dd32
Milestone: Priority: normal
Component: Translate Site & Plugins Keywords:
Cc:

Description

In Core#46305 it was mentioned to introduce a warning to GlotPress to flag manipulated placeholders from emails that follow the format of ###[A-Z_]+###. This would flag to translators a warning if they've manipulated any of these placeholders accidentally.

Attachments (1)

Screen Shot 2019-02-21 at 11.00.11 PM.png (28.3 KB) - added by garrett-eclipse 6 years ago.
Works like a charm, thanks @dd32

Download all attachments as: .zip

Change History (2)

#1 @dd32
6 years ago

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

In 8317:

Translate: Add a custom warning to catch translated placeholders.

See https://core.trac.wordpress.org/ticket/46305
Fixes #4196.

@garrett-eclipse
6 years ago

Works like a charm, thanks @dd32

Note: See TracTickets for help on using tickets.