Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
system.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Feb 2011 at 20:04 UTC
Updated:
29 Jul 2014 at 19:21 UTC
Jump to comment: Most recent file
Comments
Comment #1
wojtha commentedComment #2
wojtha commentedComment #3
wojtha commentedbumping to 8.x
Comment #4
wojtha commented#1: d7_plural_system_admin_1047920.patch queued for re-testing.
Comment #6
wojtha commentedHmm, fixed in #1099254: Typo in modules/system/system.admin.inc, so marking this as duplicate.
Comment #7
damien tournoud commentedNot really. format_plural() requires a @count in $plural. It is not optional: not having a @count would break all the languages that have more then 2 plural forms.
Comment #8
wojtha commented@Damien: Ok, renaming the issue to match the problem accurately
The placeholder @count is now included in $plural.
Edit: this test fails because of the following test assertion, so if we want this variant we just need to edit this string too. But the next patch (singular unchanged) is better IMHO.
Comment #9
wojtha commentedAlternative patch which changes only the $plural form and leaves the $singular form unchanged.
Comment #10
wojtha commented#8: 1047920-8_format_plural_count_system_admin.patch queued for re-testing.
Comment #11
pillarsdotnet commented#9: 1047920-9_format_plural_count_system_admin.patch queued for re-testing.
Comment #12
pillarsdotnet commented#9: 1047920-9_format_plural_count_system_admin.patch queued for re-testing.
Comment #13
wojtha commented@pillarsdotnet, passed again - RTBC please ;-)
Comment #14
pillarsdotnet commented@wojtha -- Looks good to me, but I don't think I'm qualified to RTBC yet, as I've not had a patch of my own RTBC'd and accepted into core.
Marking RTBC anyway, as I doubt that it makes any difference while there are 16 criticals and 100 majors ahead of this in the queue.
Comment #15
wojtha commented@pillarsdotnet thx, you don't need to remind me this... several of these criticals or majors are also "mine" ... but after 4 months of vacuum I feel that the core is finally moving again...
Comment #16
pillarsdotnet commented#9: 1047920-9_format_plural_count_system_admin.patch queued for re-testing.
Comment #17
webchickCommitted to 8.x and 7.x, thanks!
Tagging as string freeze, since this will break existing translations (for the better, though).
Comment #19
drupal_was_my_past commentedMarked #755376: Wrong plural string in system.admin.inc as a duplicate.