[wp-trac] [WordPress Trac] #46305: Update emails w/ placeholders to use sprintf

WordPress Trac noreply at wordpress.org
Fri Feb 22 05:54:05 UTC 2019


#46305: Update emails w/ placeholders to use sprintf
-----------------------------+-------------------------------
 Reporter:  garrett-eclipse  |       Owner:  (none)
     Type:  enhancement      |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  I18N             |     Version:
 Severity:  normal           |  Resolution:
 Keywords:                   |     Focuses:  coding-standards
-----------------------------+-------------------------------

Comment (by dd32):

 Something that's important to note, is that these strings are often
 manipulated through filters, either to remove placeholders, replace them
 with something else, or otherwise modify the content.

 There are also a number of emails (not the ones shown here it seems) which
 are set through the network administration screens, so having these as
 placeholders is probably a bit simpler.

 Although the onus is on Translators, translation tooling could possibly be
 a better place, for example, adding a GlotPress requirement that any
 placeholdery things `###[A-Z_]+###` remain untranslated (I'm unsure if our
 GlotPress instance has a custom rule for that or not).

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/46305#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list