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

Correction de l'historique de modération : cas d'une note nulle #4213

Merged
merged 1 commit into from
Feb 20, 2017
Merged

Correction de l'historique de modération : cas d'une note nulle #4213

merged 1 commit into from
Feb 20, 2017

Conversation

gllmc
Copy link
Member

@gllmc gllmc commented Feb 15, 2017

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) correction de #4202

Je me suis rendu compte que j'avais fait une erreur avec #4202 : pour déterminer si une action était une note de karma ou une sanction, je testais action.karma. Sauf que si la note a une valeur nulle (exemple : quand un membre modifie son pseudo), le test était faux et elle était considérée comme une sanction. Le correctif consiste donc à faire le test inverse, c'est-à-dire tester action.type, ce qui renverra toujours True dans le cas d'une sanction et False dans le cas d'une note de karma.

Au passage, si une note de karma a une valeur nulle, j'ai changé le comportement pour afficher au lieu de Modification du karma : 0 dans la colonne Action, ce qui me paraît plus judicieux.

QA

  • Appliquer une sanction à un membre et vérifier qu'elle s'affiche dans la colonne Action ;
  • Mettre une note de karma non nulle, vérifier qu'il est affiché Modification du karma : x ;
  • Mettre une note de karma de valeur nulle, vérifier qu'il est affiché .

@coveralls
Copy link

coveralls commented Feb 15, 2017

Coverage Status

Coverage remained the same at 88.367% when pulling 53b8e5b on GCodeur:correction_historique_moderation into 4612e11 on zestedesavoir:dev.

@pierre-24 pierre-24 added C-Back Concerne le back-end Django QA svp S-Régression Corrige un problème sur un composant qui fonctionnait auparavant labels Feb 15, 2017
@pierre-24
Copy link
Member

QA: OK :)

screenshot from 2017-02-20 21 23 58

@pierre-24 pierre-24 merged commit 349c645 into zestedesavoir:dev Feb 20, 2017
@pierre-24 pierre-24 added this to the Version de développement milestone Feb 20, 2017
@pierre-24 pierre-24 removed the QA svp label Feb 20, 2017
@gllmc gllmc deleted the correction_historique_moderation branch July 17, 2017 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants