[BUG] route : front_suivi_signalement_user_response

[numew]

Ticket

#2248

Description

Sur la fiche signalement usager lors de la soumission d'un message/document : contrôle de l'email passé en parametre afin de ne pas permettre l'enregistrement du suivi/file avec l'id d'un utilisateur n'étant pas sur le signalement

Tests

• Vérifier que la soumission fonctionne toujours quand l'email correspond à l'occupant/déclarant et que l'user est enregistré sur le suivi/document
• Vérifier que la soumission fonctionne quand le paramètre email n'est pas fournit sans que l'user soit enregistré sur le suivi/document
• Vérifier que la soumission fonctionne quand le paramètre email ne correspond ni à l'occupant ni au déclarant (ex admin-01@histologe.fr) sans que l'user soit enregistré sur le suivi/document

[hmeneuvrier]

Relecture OK, et tests OK (on a bien NULL en base si on ne mets pas d'adresse mail ou une adresse mail qui ne correspond pas au signalement)

Par contre, je pense qu'il faut corriger templates/back/signalement/view/suivis.html.twig, lignes 50 à 52, car quand le createBy du suivi est à NULL, on met soit le nom du déclarant, soit le nom de l'occupant.
(car avant c'était le cas, on n'enregistrait pas les usagers)
donc il faudrait laisser ce fonctionnement seulement pour les vieux suivis, mais pour les nouveaux suivis, il faudrait peut-être mettre "Utilisateur inconnu", non ?

C'est sorti dans https://github.com/MTES-MCT/histologe/releases/tag/1.5.7
#865

[numew]

Relecture OK, et tests OK (on a bien NULL en base si on ne mets pas d'adresse mail ou une adresse mail qui ne correspond pas au signalement)

Par contre, je pense qu'il faut corriger templates/back/signalement/view/suivis.html.twig, lignes 50 à 52, car quand le createBy du suivi est à NULL, on met soit le nom du déclarant, soit le nom de l'occupant. (car avant c'était le cas, on n'enregistrait pas les usagers) donc il faudrait laisser ce fonctionnement seulement pour les vieux suivis, mais pour les nouveaux suivis, il faudrait peut-être mettre "Utilisateur inconnu", non ?

C'est sorti dans https://github.com/MTES-MCT/histologe/releases/tag/1.5.7 #865

Ok, c'est fait pour les suivi postérieurs a 2023

[hmeneuvrier]

Relecture et test OK, mais je pense qu'il faut aussi ajouter un truc pour la liste de signalement, car là on a bien "Utilisateur inconnu" dans la fiche signalement, mmais on a "Aucun" dans la liste :

Sur la page de suivi on a aussi Aucun :

Il faudrait avoir la même chose aux 3 endroits (Utilisateur inconnu me semble plus clair que Aucun)

[numew]

Relecture et test OK, mais je pense qu'il faut aussi ajouter un truc pour la liste de signalement, car là on a bien "Utilisateur inconnu" dans la fiche signalement, mmais on a "Aucun" dans la liste :

Sur la page de suivi on a aussi Aucun :

Il faudrait avoir la même chose aux 3 endroits (Utilisateur inconnu me semble plus clair que Aucun)

Ok, c'est harmonisé

[hmeneuvrier]

OK pour moi

[emilschn]

Mes tests sont ok.

Par contre, je suis un peu perplexe sur Utilisateur inconnu, parce que ça vient quand même du formulaire de suivi usager.
C'est le genre de truc qui va forcément remonter par les partenaires :)

Un avis @mathildepoulpux @arnaudchouk ?
Le contexte : il peut arriver (rarement) qu'on ne sache pas si c'est l'occupant ou le déclarant qui crée un suivi, via la page de suivi usager.
Que doit-on afficher dans ce cas ?

[mathildepoulpux]

@emilschn j'opterais pour "Occupant ou déclarant" dans ce cas.
"Utilisateur inconnu" ça va faire peur je pense

[arnaudchouk]

ah oui c est sur que ca va interroger
est ce qu on sait dans quel cas ca se presente ? pourquoi ne peut on savoir de qui il s agit ?

[numew]

ah oui c est sur que ca va interroger est ce qu on sait dans quel cas ca se présente ? pourquoi ne peut on savoir de qui il s agit ?

@arnaudchouk
Ca arrive si on accède à son signalement via un lien du type "/suivre-mon-signalement/51f75e1b-01eb-3e5e-8b2d-2ffeb5b7d94f?from=adresse@email.comr" sans la partie adresse email. Ce qui est possible uniquement en modifiant l'URL à la main ou peut être sur des lien envoyé il y'a fort longtemps - en gros, le cas ne devrait quasiment pas se présenter-

[sonarcloud]

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[emilschn]

@numew bon ça marche mais... suite à cette modification, ça marque "occupant ou déclarant" même si on est identifié (par exemple en tant qu'admin).
D'une certaine manière, ça devrait pas se produire, donc je ne pense pas que ce soit grave...

Je suis pour valider comme ça.
OK ?
@hmeneuvrier aussi ?

[hmeneuvrier]

Oui moi ça me va comme ça, je merge