Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Empeche un staff de se sanctionner lui-meme #2850

Merged
merged 3 commits into from
Jul 16, 2015
Merged

Empeche un staff de se sanctionner lui-meme #2850

merged 3 commits into from
Jul 16, 2015

Conversation

Eskimon
Copy link
Contributor

@Eskimon Eskimon commented Jun 24, 2015

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés #2794

Un staff ne peut plus se sanctionner lui-meme par accident.

QA

Verifier qu'un staff peut bien sanctionner tout le monde (ban, LS) (sauf les robots) mais pas lui-meme.
Verifier via l'API serait bien aussi...

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 3aaf693 on Eskimon:fix-selfsanction into 92f891b on zestedesavoir:dev.

@firm1
Copy link
Contributor

firm1 commented Jun 24, 2015

Le cacher dans le template c'est bien, mais coté back-end ce n'est pas interdit, ça signifie que c'est potentiellement faisable via l'API par exemple.

@Eskimon
Copy link
Contributor Author

Eskimon commented Jun 24, 2015

Ah oui c'est tres vrai ca

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.03% when pulling 719bc1d on Eskimon:fix-selfsanction into 92f891b on zestedesavoir:dev.

@@ -428,6 +428,9 @@ def modify_profile(request, user_pk):
profile = get_object_or_404(Profile, user__pk=user_pk)
if profile.is_private():
raise PermissionDenied
if request.user.profile == profile:
messages.error(request, u"Vous ne pouvez pas vous sanctionner vous-même !")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Internationalisation !

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.03% when pulling 0ad49cc on Eskimon:fix-selfsanction into 92f891b on zestedesavoir:dev.

@Situphen Situphen added the S-BUG Corrige un problème label Jun 24, 2015
@pierre-24
Copy link
Member

Rapport de QA: pas OK. À priori, je peux toujours faire des bétises via l'API:

Je suis moi:

screenshot from 2015-06-26 21 47 03

Se bannir sois-même :

screenshot from 2015-06-26 21 46 39

(d'ailleurs, je devrais même plus être capable de faire quoique ce soit après, re-BUG)

Se mettre sois-même en lecture seule:

screenshot from 2015-06-26 21 44 43

(pareil, ça m'empèche de rien faire sur l'API)

@Eskimon
Copy link
Contributor Author

Eskimon commented Jul 16, 2015

Voili voilou, le site et l'API ont le meme comportement :)

@pierre-24
Copy link
Member

Rapport de QA; ok pour moi, cette fois :)

pierre-24 added a commit that referenced this pull request Jul 16, 2015
Empeche un staff de se sanctionner lui-meme
@pierre-24 pierre-24 merged commit a240d8e into zestedesavoir:dev Jul 16, 2015
@pierre-24 pierre-24 added this to the Version de développement milestone Jul 17, 2015
@Eskimon Eskimon deleted the fix-selfsanction branch July 17, 2015 10:50
@Eskimon Eskimon removed this from the Version 15.7 milestone Jul 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants