when importing a french translation, I was confronted with the error message:
Le fichier de traduction fr.po contient une erreur : la formule de valeurs plurielles n'a pu être analysée.
which is a translation for:
The translation file fr.po contains an error: the plural formula could not be parsed.
Of course, this is mostly useless, since *all* translation files for french are named... fr.po !
The following patch replaces it with:
Le fichier de traduction sites/all/modules/imagefield/translations/fr.po contient une erreur : la formule de valeurs plurielles n'a pu être analysée.
Comments
Comment #1
brianV commentedI'm all for more helpful error messages.
Comment #2
vm commentedThis should be patched against Drupal 7.x I think and backported from there. It certainly won't go directly into 6.13 and thus at the very least would have to be patched against Drual 6.x-dev. As such that is where I am moving the version to, especially since we have an included patch.
Comment #3
mikey_p commentedThe patch applied (with large offset) to HEAD. I've updated _localte_import_parse_plural_forms() to show the difference in variable name, and in phpdoc.
Comment #4
mikey_p commentedsetting this so hopefully the test bot picks it up
Comment #5
dries commentedCommitted to CVS HEAD. Thanks!
Comment #6
mikey_p commentedSince this was originally filed against 6...
Comment #7
gábor hojtsyWould be great to get some testing.
Comment #8
gábor hojtsy#545652: Language files import error message need to show full path was sort of a duplicate, kept open for solving other related issues in other locale error messages.
Comment #10
gábor hojtsyComment #11
gábor hojtsyPatch looks generally good for what it does. However, the closing parenthesis is removed from the line at _locale_import_parse_plural_forms(). No wonder it does not pass testing :) Would be great to fix.