#31921 closed defect (bug) (fixed)
String not i18n and produces error
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.2 | Priority: | low |
Severity: | normal | Version: | 4.2 |
Component: | I18N | Keywords: | i18n-change |
Focuses: | Cc: |
Description
In [31941] one string is decoupled, but one of new strings is not i18n, and it also produces error because there is one argument for printf
but it assumes there should be two.
You can test this if you remove lines 1218-1222 from wp-includes/comment-template.php
.
Attachments (1)
Change History (7)
#1
@
10 years ago
- Keywords i18n-change added
- Milestone changed from Awaiting Review to 4.2
- Priority changed from normal to low
This ticket was mentioned in Slack in #core by drew. View the logs.
10 years ago
#4
@
10 years ago
I'd suggest fixing the bug in 4.2 and leaving capitalization correction for #31927.
Note: See
TracTickets for help on using
tickets.
Similar to another i18n ticket about to be created I think
Response
should not be capitalised:src/wp-includes/theme-compat/comments.php
'One Response to %2$s', '“' . get_the_title() . '”' );Response to %2$s', '%1$s Responses to %2$s', get_comments_number() ),