devs
/
PCv5
6
10
Fork 3
Code Issues 24 Pull Requests Releases Wiki Activity
Labels Milestones
New Issue

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

Closed
opened 2 years ago by Darks · 1 comments
Darks commented 2 years 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 :)
👍 1
Darks added the bug label 2 years ago
Darks self-assigned this 2 years ago
Lephenixnoir commented 2 years ago
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 2 years ago
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
dev
preprod
new_editor
improve-entropy
widgets
master
poc/tabtitle
api/admin/members
Labels
Apply labels
Clear labels
bug
Something is not working duplicate
This issue or pull request already exists easy
The fix is considered as easy to make enhancement
New feature help wanted
Need some help invalid
Something is wrong performance
Something is slow, not optimized proposal
Feature to be discussed, RFC question
More information is needed security
This issue is a security breach warning
It works, but for how long? wontfix
This won't be fixed
No Label bug duplicate easy enhancement help wanted invalid performance proposal question security warning wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
Darks
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: devs/PCv5#101
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(<nil>)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes