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

Closed
opened 2021-07-12 11:11:00 +02:00 by Darks · 1 comment
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 2021-07-12 11:11:00 +02:00
Darks self-assigned this 2021-07-12 11:11:00 +02:00
Owner

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 2021-07-14 17:05:14 +02:00
Sign in to join this conversation.
No description provided.