Plural forms displayed as "Array" in Moderate queue
Stored translation - displayed well
Suggested - as "Array" word
Very, Very unusable
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | more-string-checks.patch | 802 bytes | gábor hojtsy |
| #4 | drupal-all-uk.po | 794 bytes | podarok |
| #4 | array_issue.jpg | 32.4 KB | podarok |
| #2 | import-plural-Array.patch | 1.19 KB | gábor hojtsy |
| #1 | Suggestions.png | 92.31 KB | gábor hojtsy |
Comments
Comment #1
gábor hojtsyIllustrating the issue
Comment #2
gábor hojtsyUh the problem is not actually in the display but the saving of suggestions in some way, we have 105 bogus suggestions/translations:
This covers many languages:
Investigation turns out that the trim function can be called with an array, and converts the string to an Array. Fixed this immediately to overcome any future issues. Regular translation input on the UI was not a problem, since it checks for strings before calling this function. The l10n_remote feature with l10n_client was not an issue, because l10n_client lacks plural translations still. Anyway, solved it in this API function, so it is surely not reappearing.
This nicely underlines that automated tests for imports would be timely to add. Hopefully solved for now though.
Also, dropped the 105 Array suggestions.Contacted the teams instead.Comment #3
gábor hojtsyPosted service notification at http://localize.drupal.org/node/264
Comment #4
podaroklooks like not fixed 8(
there is testing file in attachements
Comment #5
gábor hojtsyUhm, you are perfectly right. I've assumed I did not make the same mistake at two places. Oh, I did. Sorry again. Fixed, tested, works for me.
Comment #6
podarokthanks....
really fixed!