In lot of case, there can be minor typos like missing "." at the end of the row or use of wrong quotation marks in some languages etc. In these case, if the moderator corrects the string he takes the authorship too which is not fair.

What I suggest is a checkbox button for every string what indicates the ownership. This should be checked by default (the moderator should uncheck it if he corrects the useless translation).

A more complex case can be a 3 stage select box for those cases where the suggestion is corrects an existing translation. Who should be the author of the string? The moderator could deciede.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Takafumi’s picture

+1 for this feature.

nevergone’s picture

+1
I need :)

asrob’s picture

+1

Takafumi’s picture

Status: Active » Needs review
FileSize
9.76 KB
3.18 KB

Please review this patch.

Gábor Hojtsy’s picture

Title: "Keep the original translator" checkbox » Add a "keep original translator" checkbox
Status: Needs review » Needs work

@Takafumi: I don't understand why does this patch lists all the usernames for all people who have active suggestions in a dropdown to pick? That seems to be too much to offer here. What's the use case?

Takafumi’s picture

@Gábor: l10n_server allows two or more suggestions, but even in that case, only one suggestion can be edited. So, all usernames who have active suggestions will have to be listed, and then, moderator can determine a translator.

Gábor Hojtsy’s picture

Is it that you pick the suggestion closest to what you'd accept, right? Then why would you pick anybody else but you or the suggestor to take credit? I'm not getting this.

Takafumi’s picture

@Gábor: You might want to imagine the case with the first post.

Gábor Hojtsy’s picture

Status: Needs work » Needs review

Ah, you mean this one:

A more complex case can be a 3 stage select box for those cases where the suggestion is corrects an existing translation. Who should be the author of the string? The moderator could deciede.

That indeed seems like what you've attempted to solve, right.

droplet’s picture

can we make it easier ? compare different between 2 suggestions automatic. if changes less than X%, make it as "minor changes"(whatever you want to call). and it need to keep a records for their contributors works too. I would like to be a Top minor contributors showing on L.D.O pages.

Status: Needs review » Needs work

The last submitted patch, l10n_server_1.patch, failed testing.

palik’s picture

+1, subscribing :)

droplet’s picture

recalling people to focus on it :)
I can see my point getting up a lot by minor fixing a "."

#4, #9
I don't get any idea of this 2 point :(

for now, it may have
original translator
suggestion translator
ME

if I submit a new string / approve a suggestion, translator change to ME/suggestion translator.
if I want to keep "original translator as translator", it should provide ONE option only.

so we need to provide a CLICK BOX only ??

danillonunes’s picture

I don't know if an selectbox or checkbox is required.

I think that every time I click "Edit a copy" icon, the translation can be saved as "Suggestion by xyz, Edited by abc.". If I want make a new translation without one previous suggestion, I click on <New translation>.

kulcsi’s picture

FileSize
123.31 KB
117.63 KB
147.96 KB

The main translator in most of the cases can be calculated based on the number of new/modified characters commited by the translators. The translation should be automatically attributed to the person who have modified the most no. of characters.
We should give the opportunity to the moderator to change the default value of the "attributed to" metadata.

wundo’s picture

subscribing

SebCorbin’s picture

Version: 6.x-1.x-dev » 6.x-3.x-dev
Assigned: Unassigned » SebCorbin

I would also like to see this implemented. This is quite easy, so I'll do the code.

So we have currently two possibilities :

  • A checkbox ticked by default, that the moderator unticks if he wants the suggestion attributed to himself
  • A script that detects when a characters string is enough edited to be attributed to the moderator

Both ways, the original contributor comes from the "Edit a copy" button.

Which one do you like? (We can also combine the two)

SebCorbin’s picture

Status: Needs work » Needs review
FileSize
5.72 KB

First patch, first impressions: this module is a mess :(
FAPI is hardly respected, I had to follow this mess to make a quick patch, but I'd better see the translation.inc file rewritten...

Please review this patch (which is against 6.x-3.x-dev)

droplet’s picture

@SebCorbin,

I edit my own translation and also show this:
"Re-author this suggestion to droplet"

another cool way is make "User Name" to be ajax editable, let admin change to any username.
(anyway, now is enough for 95% cases)

another problem, edit textfield keeps moving, its quite hard to cancel the checkbox:
#1228398: Keep textarea expanded when editing suggestion

SebCorbin’s picture

I don't agree with the fact to put any username for two reasons:

  1. Why would it be useful to put someone who doesn't have actually suggested something
  2. This would require more AJaX-based code for a too little feature

I totally agree that the current code is enough for 95% cases.

I'll look into the other issue.

Status: Needs review » Needs work

The last submitted patch, l10n_server_reauthor_632120_18.patch, failed testing.

SebCorbin’s picture

Status: Needs work » Needs review
FileSize
77.34 KB
8.48 KB

Re-rolled, adding a permission, and a history status.

Screenshot_30_01_13_17_14-5.png

Test should be bypassed by specifying type after name attribute on input element.

Status: Needs review » Needs work

The last submitted patch, 632120-22-re_author_suggestion.patch, failed testing.

SebCorbin’s picture

Status: Needs work » Needs review
FileSize
7.41 KB

Messed with the tests and forgot an empty()

Gábor Hojtsy’s picture

Code changes look good. I don't have time to test in the foreseeable future, so don't hold up on that :)

Pomliane’s picture

Status: Needs review » Needs work
FileSize
23.61 KB

Works as expected but the string associated to the checkbox could probably be improved: something like "Author this suggestion and assign to [user]", maybe?

SebCorbin’s picture

Version: 6.x-3.x-dev » 7.x-1.x-dev
Issue summary: View changes
Status: Needs work » Needs review

Testing on 7.x-1.x