Skip to content

[agent] fix active users scope in users index

Vincent Agnano requested to merge fix/active-users-scope-for-agents into master

Created by: adipasquale

PR très discutable !

pendant que je bossais sur la fusion des usagers, j'ai fait des manips interdites et me suis retrouvé dans des états incohérents. Ces deux petits changements m'ont paru utiles à ce moment, mais ce sont en fait des sécurités redondantes avec ce qui existe déjà. On peut considérer ça comme du code "en trop", ou des sécurités utiles, je ne suis pas hyper sûr.

  1. filtrer sur les usagers active (càd pas soft deleted) dans les listes d'usagers coté agent. c'est deja fait indirectement via le filtre du Policy Scope des users qui filtre les usagers appartenant a l'orga courante. ca ne devrait pas se produire qu'un usager soit soft deleted mais continue a appartenir a une orga, donc c'est redondant mais quand meme plus securisant

  2. validation sur les users que responsible_id != id. ne devrait pas se produire non plus, mais quand ca se produit pour une raison ou une autre c'est le gros bazar avec des boucles infinies, donc sympa a eviter :)

Merge request reports