fix(policy): Fix scope in policies

[aminedhobb]

Contexte

On a un bug qui est apparu suite à la refacto de la méthode current_agent_department_organisations dans #2404 .

En effet, policy_scope(current_department.organisations) retourne toutes les organisations de l'agent et pas celle du département seulement 😬 .

Ça a pour conséquence que toutes les organisations de tous les départements sont retournées dans la liste:

Heureusement il n'y a que nous (les super admins) qui sommes sur plusieurs territoires donc le bug n'a à priori pas eu d'impact.

Solution

Je répare la manière dont est construit le scope erroné dans l'OrganisationPolicy ainsi que dans les autres policies en faisant appel à la variable scope pour prendre en compte la requête passée en argument de la méthode policy_scope.

[Holist]

Ca m'a l'air bon. Je me demande si il faudrait ajouter des tests pour les autres scopes que tu modifies ici (CategoryConfiguration et Department et MessageConfiguration) mais c'est peut être too much sachant que ce que tu fais c'est le fonctionnement attendu du resolve du scope. D'ailleurs il y a ArchivePolicy qui semble problématique également, il y a une raison pour ne pas l'avoir modifié aussi ?

[aminedhobb]

Ca m'a l'air bon. Je me demande si il faudrait ajouter des tests pour les autres scopes que tu modifies ici (CategoryConfiguration et Department et MessageConfiguration) mais c'est peut être too much sachant que ce que tu fais c'est le fonctionnement attendu du resolve du scope.

En fait ces scopes ne sont jamais appelés dans l'appli sur autre chose que sur la table toute entière, donc ce que j'ai fait ne change rien (c'est d'ailleurs pour ça que les tests existants n'ont pas fail) mais ça prévient les futures erreurs.

D'ailleurs il y a ArchivePolicy qui semble problématique également, il y a une raison pour ne pas l'avoir modifié aussi ?

Bien vu merci 🙌 !! La méthode n'était même pas déclarée correctement 😕 et n'était pas utilisée, je l'ai enlevée ici 754a9cf

[Holist]

👍