Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Synthèse/Validation] Question : une observation épinglée est visible de tous #2507

Closed
mvergez opened this issue Apr 20, 2023 · 4 comments

Comments

@mvergez
Copy link
Contributor

mvergez commented Apr 20, 2023

Dans le cadre d'une prestation pour l'Agence Régionale de la Biodiversité île de France, nous nous sommes demandés si les épingles sur les observations fonctionnaient comme prévu.

En effet, il semblerait que les autres utilisateurs puissent voir les observations épinglées par un utilisateur. Pour reproduire :

  • Se connecter en tant qu'admin
  • Épingler une observation
  • Se déconnecter puis se connecter en tant qu'agent ou autre utilisateur
  • Rechercher les observations épinglées (via le filtre avancé "Est épinglé")
  • L'observation épinglée plus haut apparait

Est-ce le fonctionnement prévu ?
En re-parcourant la PR : #1843, il semblerait que non :

Un utilisateur ne peut voir que les épingles qu'il a créé lui-même

Merci d'avance pour vos éclaircissements !

@mvergez mvergez changed the title Question : une observation épinglée est visible de tous [Synthèse/Validation] Question : une observation épinglée est visible de tous Apr 20, 2023
@bouttier
Copy link
Contributor

bouttier commented Apr 20, 2023

En effet il manque l’id_role dans le filtre :

pin_filter = self.model.reports.any(
TReport.report_type.has(BibReportsTypes.type == "pin")
)

@camillemonchicourt
Copy link
Member

En effet, les épingles sont censées être individuelles. Voir #1819
Donc c'est un soucis si ce n'est pas répercuté dans la recherche de la Synthèse.

@mvergez
Copy link
Contributor Author

mvergez commented Apr 20, 2023

Top merci pour vos retours, j'ai fait une PR qui règle le souci

@camillemonchicourt
Copy link
Member

Corrigé dans la 2.12.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants