[wp-meta] [Making WordPress.org] #6256: Disable translation warning discard for unexpected additional printf placeholders.

Making WordPress.org noreply at wordpress.org
Fri Apr 29 08:34:52 UTC 2022


#6256: Disable translation warning discard for unexpected additional printf
placeholders.
--------------------------------------+---------------------
 Reporter:  dd32                      |       Owner:  (none)
     Type:  defect (bug)              |      Status:  new
 Priority:  high                      |   Milestone:
Component:  Translate Site & Plugins  |  Resolution:
 Keywords:  needs-patch               |
--------------------------------------+---------------------

Comment (by akirk):

 I have proposed a patch for this. The screenshot is a little outdated
 (since this scenario is now allowed) but it demonstrates the UI response
 the user will get when discarding the warning.

 The proposed code will prevent discarding the
 [https://github.com/GlotPress/GlotPress/blob/741b4dd91138530d550e86d7d7c71179016b058e
 /gp-includes/warnings.php#L610 unexpected_sprintf_token warning] and the
 [https://github.com/GlotPress/GlotPress/blob/741b4dd91138530d550e86d7d7c71179016b058e
 /gp-includes/warnings.php#L671 placeholders warning] when there is an
 extra placeholder that is not a `%%`.

 A warning complaining about a missing placeholder in the translation can
 still be discarded.

-- 
Ticket URL: <https://meta.trac.wordpress.org/ticket/6256#comment:1>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org


More information about the wp-meta mailing list