Plural formula field only accepts 128 characters

This 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 :)

I 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?

The update function seems to be the next one for Drupal 6 and 7 (i put 256 instead of 128) :

function locale_update_N() {
  db_change_field('languages', 'formula', 'formula', array(
    'type' => 'varchar',
    'length' => 256,
    'not null' => TRUE,
    'default' => '',
    'description' => 'Plural formula in PHP code to evaluate to get plural indexes.',
  ));
}

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).

Right, does *not* apply to Drupal 8 as the data is not stored in a length restricted column.

Thanks.
Can someone commit the update function (see my last comment) and the fix in the schemas?

Let's move forward

+++ b/modules/locale/locale.install
@@ -202,6 +202,19 @@ function locale_update_7004() {
+    'length' => 255,

schema it self has to be altered as well, so new installs get the right length

13 days to next Drupal core point release.

Looking good

Committed and pushed to 7.x, thanks!

Does this also need backport to D6?

I guess so, I'll ask Gabor