Making WordPress.org

Changes between Initial Version and Version 1 of Ticket #6256, comment 1


Ignore:
Timestamp:
04/29/2022 08:35:07 AM (3 years ago)
Author:
akirk
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #6256, comment 1

    initial v1  
    1 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.
     1I have proposed a patch for this. The screenshot is a little outdated (since this scenario is still allowed, see below) but it demonstrates the UI response the user will get when discarding the warning.
    22
    33The 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 `%%`.