Changes between Initial Version and Version 1 of Ticket #6256, comment 1
- Timestamp:
- 04/29/2022 08:35:07 AM (3 years ago)
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.1 I 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. 2 2 3 3 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 `%%`.