Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The translation of some time spans collide with translations using format_plural:
'#options' => array(
0 => t('Immediate'),
1 => t('1 day'),
7 => t('1 week'),
14 => t('2 weeks'),
)
Due to the format_plural of both '1 day' and '1 week' the translations already exist. I suggest changing the code to:
'#options' => array(
0 => t('Immediate'),
1 => format_plural(1, '1 day', '@count days'),
7 => format_plural(1, '1 week', '@count weeks'),
14 => format_plural(2, '1 week', '@count weeks'),
)
I'll try to submit a patch in the comments...
Comment | File | Size | Author |
---|---|---|---|
#1 | simplenews-format_plural-1730008-1.patch | 956 bytes | lund.mikkel |
Comments
Comment #1
lund.mikkel CreditAttribution: lund.mikkel commentedPatch
Comment #2
BerdirThanks, commited and pushed.