Plural forms displayed as "Array" in Moderate queue
Stored translation - displayed well
Suggested - as "Array" word

Very, Very unusable

Comments

gábor hojtsy’s picture

StatusFileSize
new92.31 KB

Illustrating the issue

gábor hojtsy’s picture

Status: Active » Fixed
StatusFileSize
new1.19 KB

Uh the problem is not actually in the display but the saving of suggestions in some way, we have 105 bogus suggestions/translations:

mysql> select count(*) from l10n_community_translation where translation = 'Array';
+----------+
| count(*) |
+----------+
|      105 | 
+----------+
1 row in set (0.32 sec)

This covers many languages:

mysql> select count(*), language from l10n_community_translation where translation = 'Array' group by language;
+----------+----------+
| count(*) | language |
+----------+----------+
|       11 | es       | 
|        3 | fr       | 
|       14 | hu       | 
|        5 | ne       | 
|       29 | pl       | 
|        2 | tr       | 
|       37 | uk       | 
|        4 | zh-hant  | 
+----------+----------+
8 rows in set (0.77 sec)

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.

gábor hojtsy’s picture

Posted service notification at http://localize.drupal.org/node/264

podarok’s picture

Status: Fixed » Needs work
Issue tags: +l10n-server-uk
StatusFileSize
new32.4 KB
new794 bytes

looks like not fixed 8(
there is testing file in attachements

Only local images are allowed.

gábor hojtsy’s picture

Status: Needs work » Fixed
StatusFileSize
new802 bytes

Uhm, 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.

podarok’s picture

thanks....
really fixed!

Status: Fixed » Closed (fixed)
Issue tags: -l10n-server-uk

Automatically closed -- issue fixed for 2 weeks with no activity.