Utilise permission_required pour la gestion des droits – module member

[gllmc]

Q	R
Type de modification	refactorisation
Ticket(s) (issue(s)) concerné(s)	#4181

Cette pull request modifie le fichier de vues du module member pour utiliser le décorateur permission_required au lieu d'une condition comme montré dans #4181.

QA

Vérifier que les pages/actions suivantes ne sont accessibles qu'aux membres du staff :

• sanction d'un membre (pour essayer sans avoir accès au formulaire : Empêche un membre non staff d'en bannir un autre #4047)
• modification du profil (je parle de la vue settings_mini_profile, pas de la page des paramètres permettant à un membre de changer ses propres paramètres)
• recherche de double-comptes via l'IP
• modification du karma (même principe que pour la sanction d'un membre)

Le comportement attendu est le suivant :

• redirection vers la page de connexion pour un utilisateur non authentifié
• 403 pour un membre n'ayant pas les droits
• accès à la page pour un membre ayant les droits

[coveralls]

Coverage decreased (-0.003%) to 88.36% when pulling 5028c4a on GCodeur:decorateur_perms_member into 54e11ef on zestedesavoir:dev.

[pierre-24]

Je pense que tu peux étendre cette idée à un certains nombre de modules, non ? :)

[artragis]

mais c'est mieux de le faire petit à petit. En tout cas gros +1 pour l'amélioration de la base de code :)

[gllmc]

En effet, je compte faire ça module par module afin d'éviter de rendre la QA trop longue (à voir cependant si c'est vraiment nécessaire de tester toutes les pages à la main, les tests couvrent ça normalement). :)

[DevHugo]

Nan, c'est pas la peine, on teste une fois pour voir si la méthode marche et après, on merge les PR en série.

[pierre-24]

Et je confirme que ça fonctionne. Au boulot, @GCodeur ;)