I'm trying to create a translation group on localize.drupal.org for Breton (#1517408: Fix Breton plural formula on l.d.o), but when I try to add the correct plural formula on http://localize.drupal.org/admin/settings/language/edit/br I'm cut off after 128 characters.
(Breton plural forms are somewhat complicated, so this isn't enough...)
Comment | File | Size | Author |
---|---|---|---|
#8 | 1571076-plurla.patch | 961 bytes | droplet |
#6 | 1571076-plurla.patch | 649 bytes | droplet |
Comments
Comment #1
Gábor HojtsyThis is going to be a longer ride, since Drupal core itself limits the field in 128 chars. The UI merely exposes this limit so you don't get surprises when it is saved. It needs to be fixed first in the schemas. Drupal 6: http://drupalcode.org/project/drupal.git/blob/refs/heads/6.x:/modules/lo...
Now due to our backporting workflow/policy, this needs fixing in Drupal 8 then 7 then 6, then it can be deployed on localize.drupal.org. Yeah.
Should be a relatively easy fix but needs an update function too to change the field. It would be great if you could help work on this :)
Comment #2
yohannbzh CreditAttribution: yohannbzh commentedI am the one who asked for #1517408: Fix Breton plural formula on l.d.o.
I can maybe help but I only start on Drupal.
What is exactly the work to do?
Comment #3
yohannbzh CreditAttribution: yohannbzh commentedThe update function seems to be the next one for Drupal 6 and 7 (i put 256 instead of 128) :
For Drupal 8, number of plurals and plural forms were removed from the languages schema (#1323176: Decouple UI translation language information from language list schema).
Comment #4
Gábor HojtsyRight, does *not* apply to Drupal 8 as the data is not stored in a length restricted column.
Comment #5
yohannbzh CreditAttribution: yohannbzh commentedThanks.
Can someone commit the update function (see my last comment) and the fix in the schemas?
Comment #6
droplet CreditAttribution: droplet commentedLet's move forward
Comment #7
attiks CreditAttribution: attiks commentedschema it self has to be altered as well, so new installs get the right length
13 days to next Drupal core point release.
Comment #8
droplet CreditAttribution: droplet commentedComment #9
attiks CreditAttribution: attiks commentedLooking good
Comment #10
webchickCommitted and pushed to 7.x, thanks!
Does this also need backport to D6?
Comment #11
attiks CreditAttribution: attiks commentedI guess so, I'll ask Gabor