La liste des derniers topics du menu contient des topics privés #101

Closed
opened 1 year ago by Darks · 1 comments
Darks commented 1 year ago
Owner

La liste des derniers topics recense les topics privés pour tout le monde.

Trois solutions à ça :

  • accepter le comportement et laisser des 403 pour les gens qui n'ont pas accès aux topics (rien à faire)
  • supprimer de la liste les topics privés et utiliser le futur système de topic non-lu pour être informé (facile)
  • rendre dynamique la liste suivant l'utilisateur (moins évident)

Je suis plutôt partant pour la dernière option, bien qu'elle soit peut-être un peu tricky à mettre en place.

Techniquement, tout se passe dans cette requête SQL : https://gitea.planet-casio.com/devs/PCv5/src/branch/dev/app/processors/menu.py#L16

J'essaie de travailler dessus au plus vite, ça me permettra de me remettre dans le bain de la v5 :)

La liste des derniers topics recense les topics privés pour tout le monde. Trois solutions à ça : - accepter le comportement et laisser des 403 pour les gens qui n'ont pas accès aux topics (rien à faire) - supprimer de la liste les topics privés et utiliser le futur système de topic non-lu pour être informé (facile) - rendre dynamique la liste suivant l'utilisateur (moins évident) Je suis plutôt partant pour la dernière option, bien qu'elle soit peut-être un peu tricky à mettre en place. Techniquement, tout se passe dans cette requête SQL : https://gitea.planet-casio.com/devs/PCv5/src/branch/dev/app/processors/menu.py#L16 J'essaie de travailler dessus au plus vite, ça me permettra de me remettre dans le bain de la v5 :)
Darks added the
bug
label 1 year ago
Darks self-assigned this 1 year ago

J'approuve la dernière solution, c'est le comportement correct à mon goût.

Suggestion pour que ça reste pas trop compliqué (et performant) : sélectionner 20 topics et ensuite filtrer, c'est pas demain la veuille que 10 topics privés seront actifs en même temps.

Je sais pas si t'as vu, mais la méthode qui factorise ça c'est forum.is_default_accessible() pour les invités et member.can_access_forum() sinon.

J'approuve la dernière solution, c'est le comportement correct à mon goût. Suggestion pour que ça reste pas trop compliqué (et performant) : sélectionner 20 topics et ensuite filtrer, c'est pas demain la veuille que 10 topics privés seront actifs en même temps. Je sais pas si t'as vu, mais la méthode qui factorise ça c'est `forum.is_default_accessible()` pour les invités et `member.can_access_forum()` sinon.
Darks closed this issue 1 year ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: devs/PCv5#101
Loading…
There is no content yet.