Setting a limit for possible suggestions will filter out many similars. As people can translate independent from each other many similar suggestions can go in. The admins have to read through all of them which makes it a long process. I suggest to limit the number of visible suggestions to not more than 15. Or another possible way: Check similarity of sentences and filter out duplicates before showing.

Comments

Gábor Hojtsy’s picture

Status: Active » Postponed (maintainer needs more info)

How the code is written, duplicates should not even be possible to be submitted. If they can be submitted, I'd consider it a bug. Also, kkaefer has an issue to support diffing between suggestions, so you can see the exact changes pretty darn fast. See #563128: Rework the translation UI. Not sure whether that covers the use cases you'd want to cover with this feature request.

Nchase’s picture

it is more about limiting the amound of translation to get in for suggestions. The worst case: We got 400 translators and everyone has translated the same string. Now someone reviews it and sees all that 400 suggestions waiting for him to be read. Kind too much I guess? Than you limit the amount to say 25 by the software. More suggestions are not accepted any more or simply rejected with a "queue full" message.

kkaefer’s picture

Status: Postponed (maintainer needs more info) » Active

Instead of setting a hard limit to the number of suggestions, we could instead opt to display a warning message when trying to submit a new suggestion when there are more than a certain number of suggestions already available for that string.

Gábor Hojtsy’s picture

Yeah, that one sounds like a decently better idea to me at least! :)

Nchase’s picture

yes, great! :)

Eirik_R’s picture

I might have missed a point here, but as far as I can tell, duplicates (for translations) are very possible. I put probably 100 or so of them into l.d.o a week ago when doing mass imports.

If you look at Norwegian Bokmål translation of «Stylesheet URL» you'll see there's a duplicate (and btw, I believe they are both wrong, but I'll leave that for later). I had at least four identical suggestions for the same string at a point, there could be more of those. The server should definately leave out suggestions (or translations) that are already in, but some of those I put in could perhaps be explained by importing very many similar strings at the same time.

Gábor Hojtsy’s picture

Yeah, then we have a bug here. The server should not allow the same suggestion for the same string multiple times.

Eirik_R’s picture

I think this could be the result of import handling strings different than "normal" translate. Using the translate screen i can not add another duplicate, whereas import allows me to add duplicates in bulk.

I understand l10n server accepts translations from remote (l10n client (++?)). Shouldn't all translations/suggestions be put through a common filter before adding them to the database?

Gábor Hojtsy’s picture

@Eirik_R: yes, a common filter would be best.

Gábor Hojtsy’s picture

I've just submitted code (#494320: Unify translation checks when saving and skip whitespace only strings) and planning to deploy fixes to localize.drupal.org to not allow duplicate submissions from .po files or remote suggestions either, just like it was for UI based translations. Whitespace only submissions are also stopped before getting into the database. Maybe that would make this a duplicate then?

Gábor Hojtsy’s picture

Title: limit number of possible suggestions » Limit number of possible suggestions

Is this still relevant given the duplicate submissions now averted? I'm still not sure this one is a very good idea.

andypost’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Category: feature » task

Suppose we should divide translation & suggestion as grouping entity to allow extend suggestion functionality

Also this could make queries lighter

'l10n_server_string'
'l10n_server_translation' - only one which used as active
'l10n_server_suggestion' - community could poll on suggestion and some action will copy it's data to active-translation table

podarok’s picture

agree to #12