Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy’s picture

Status: Needs review » Needs work

The OGUR migration issue at #652038: Migrate permissions to og_user_roles, instead of our own code resulted in direct use of user_access() all around, so this will need a reroll. Additionally to that, I'd have the following remarks:

- let's not hardcode access for declining own suggestions, some users might not want that; instead break out "moderate own suggestions" to "decline own suggestions" and "approve own suggestions"
- your solution handles this on the individual AJAX output, but not on the mass moderation screen; allowing people to mass-decline their suggestions would have this nice side effect to have access to the moderation page and have an overview of all outstanding suggestions one has in the system, so they can directly vet their contributions there, so it would be vital to make this more granular workflow work there

hass’s picture

Great, this helps people removing their own typos.

hass’s picture

Marked #331051: Delete own suggestion as duplicate of this issue.

Gábor Hojtsy’s picture

Title: Allow declining own translations for everyone » Add permission to allow declining own translations

Retitled to be more accurate in terms of what it does.

Gábor Hojtsy’s picture

Or to be more precise, what it would be good to do :)

Pasqualle’s picture

Status: Needs work » Needs review
FileSize
12.73 KB

implemented the requirements in comment #1

I guess some other required modifications are needed:
additional update functions for
l10n_community.install
l10n_groups.install

additional test
l10n_community.test

Gábor Hojtsy’s picture

Status: Needs review » Needs work

From the looks of the moderate.inc code, this patch allows people who can moderate others and decline their own suggestions to also approve their own suggestions due to how the conditionals are formed.

Generally, I'd suggest to postpone these big patches which touch considerable amounts of code in translate.inc to after we switched over to the 2.x branch, which introduces significant changes in the translate.inc code.

Pasqualle’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
tobiasb’s picture

Status: Needs work » Needs review
FileSize
5 KB
tobiasb’s picture

droplet’s picture

Status: Needs review » Needs work

it only add & changed to a new permission. That means

moderate own suggestions = decline own suggestions (in some sections)

so everyone could decline, also approve their own suggestions.

droplet’s picture

Status: Needs work » Needs review
FileSize
1.75 KB
tobiasb’s picture

FileSize
1.93 KB

Removed hunks.

Magnus’s picture

Status: Needs review » Reviewed & tested by the community

This would be a nice feature. Often users submit a suggestion and one minute later makes a new one. If the user could decline the faulty suggestions, then there is less work for the administrators.

I applied the patch in #13 and it works great!

hass’s picture

Go for it, I'd like to decline some of my wrong suggestions, I've added some years ago.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, l10n_server-652460.patch, failed testing.

droplet’s picture

Version: 6.x-2.x-dev » 6.x-3.x-dev
Issue tags: +Novice

Nothing new, just a reroll.

I bet it is v3 now or D7 first... anything getting it commit is great :)

Sutharsan’s picture

Status: Needs work » Needs review
Issue tags: -Novice
FileSize
1.54 KB

Re-rolled the patch for 6.x-3.x-dev

Sutharsan’s picture

And a D7 patch too. I've marked it do-not-test as it will fail on this D6 branch.

SebCorbin’s picture

FileSize
4.27 KB

Added tests, will be committed if testbot is ok.

Status: Needs review » Needs work

The last submitted patch, 652460-20-l10n_server.patch, failed testing.

Gábor Hojtsy’s picture

Issue summary: View changes
izus’s picture

Hi,
any update on this ?
it can be critical to have the right to delete "own non validated suggestions" especially if by mistake some personal data unrelated with source string was suggested. Once submitted, we have no way to correct the mistake :/

ohmdesbois’s picture

This would seem very useful to easily improve the quality of suggestion imports and facilitate the work of validators to give simple contributors the right to cancel their own suggestions.
It seems that there is a role of "self-moderator". If this is the case, why not assign it automatically or in any case facilitate its request.
Thank you