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

Fix #2275 : uniformisation du nombre de messages d'un membre (forum) #2285

Merged
merged 1 commit into from
Feb 25, 2015
Merged

Fix #2275 : uniformisation du nombre de messages d'un membre (forum) #2285

merged 1 commit into from
Feb 25, 2015

Conversation

gustavi
Copy link
Contributor

@gustavi gustavi commented Feb 11, 2015

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

QA

  • Modérer des messages sur un membre
  • Vérifier avec des droits staff et un simple membre si le nombre de messages postés correspond bien à ceux affichés en cliquant sur le ce nombre.

@Fulbert
Copy link
Contributor

Fulbert commented Feb 11, 2015

Voilà ce que j'ai fait :

  • créé un nouveau sujet avec le compte user
  • masqué le premier message du sujet, celui écrit par le compte user, avec le compte admin

Avec le compte admin :

  • le compteur de message posté ET de sujet créé affiche le nombre correct (en incluant les messages masqués).

Avec le compte user :

  • le compteur de message posté est correct (il n'inclut pas les messages masqués, qui ne sont pas affichés sur la page Messages postés par user
  • le compteur de sujet créé inclut le sujet créé, même si le premier message est masqué. La page Sujets créés par user liste bien le sujet, mais affiche l'extrait du message qui devrait être masqué. Même un utilisateur lambda (ïtrema par exemple) peut voir cet extrait.

Du coup ça pose une question : faut-il compter et afficher un sujet créé si le premier message est masqué ?
Et ça soulève un bug : on peut voir un extrait d'un message masqué si celui-ci est le premier message d'un sujet, en allant sur la page Sujets créés par [le créateur du sujet].

J'ai testé de masquer un message seulement, je sais pas ce qu'il se passe si on déplace un sujet vers un forum corbeille.

@Eskimon Eskimon added C-Back Concerne le back-end Django S-BUG Corrige un problème labels Feb 12, 2015
@Fulbert
Copy link
Contributor

Fulbert commented Feb 25, 2015

Je pense que ça peut être mergé.

@Eskimon
Copy link
Contributor

Eskimon commented Feb 25, 2015

Yep avec les modifs de hier ca me semble bon d'apres ton message

Eskimon added a commit that referenced this pull request Feb 25, 2015
Fix #2275 : uniformisation du nombre de messages d'un membre (forum)
@Eskimon Eskimon merged commit 04f80e5 into zestedesavoir:dev Feb 25, 2015
@gustavi gustavi deleted the fix-2275 branch December 9, 2015 21:47
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-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants