[BO - signalement] Possibilité d'ajouter des photos et rapport à une visite, et affichage de ces photos, mise à jour des suivis, catégorisation des types de documents #2343

hmeneuvrier · 2024-03-18T14:16:52Z

Ticket

Description

Possibilité d'ajouter des photos de visite et un rapport de visite.
Affichage des photos de visite en mode album.
Séparation des documents, en fonction de leur type, entre les documents sur la situation (en faut de page) et les documents sur les procédures (en bas de page)
Ajustement des textes et de la visibilité des suivis d'ajout et de suppression de fichier

Changements apportés

Création d'un nouveau twig pour la modale d'upload de rapport et photos de visites
Mise à jour des twigs liés aux visites et liés aux photos
Mise à jour du form_upload_documents.js pour l'utiliser pour la momdale d'upload de rapport et de photos de visites
Dans l'enum DocumentType, création de nouvelles listes pour séparer les types de documents en fonction de situation ou procédure
Création d'une fonction getAllPhotosOrdered dans le signalementManager (pour l'affichage dans l'album photo)
Dans SignalementFileController, on met à jour l'intervention et la description d'un fichier si elles existent
Dans l'entité Intervention, création d'une fonction getRapportDeVisite (car avant seuls les rapports de visite avaient l'id d'une intervention, ce qui conditionnait l'affichage des boutons et du rapport dans la partie Visite. Or maintenant, les photos de visites ont également l'id de l'intervention)
Mise à jour de la création de suivis dans le SuiviManager
Mise à jour des suivis de suppression dans le SignalementFileController

Pré-requis

Tests

Faire les tests suivants sur un signalement fait avec le nouveau formulaire et un signalement fait avec l'ancien.
⚠️ attention de faire sur des signalements dans un territoire avec des partenaires de visite et d'affecter ces partenaires pour les tests sur les visites !

Ajouter des photos de la situation. Si nouveau formulaire vérifier qu'on peut choisir le désordre. Si ancien, non. Vérifier le texte du suivi créé et l'affichage des photos dans le haut de la page
Ajouter des documents de la situation. Vérifier que le choix des types est bien limité suivant les specs. Vérifier que les fichiers s'affichent en haut de la page, que le texte du suivi est bon et qu'il est partagé à l'usager.
En bas de page, ajouter des documents partenaires. Vérifier que le choix des types est bien limité suivant les specs. Vérifier que les fichiers s'affichent en bas de la page, que le texte du suivi est bon, et que ce suivi n'est pas partagé à l'usager.
Créer une visite effectuée dans le passé, sans ajouter un rapport de visite. Vérifier qu'on peut ajouter un rapport de visite a posteriori, et qu'on peut ajouter des photos de visite. Ajouter un rapport de visite, vérifier qu'il n'y a pas besoin de choisir le type, qu'il est affecté d'office. Vérifier que le suivi est partagé à l'usager et que le texte est bon. Vérifier qu'on a ensuite un bouton "Voir le rapport de visite" qui ouvre le bon document.
Ajouter des photos de la visite, vérifier qu'on peut ajouter une description. Vérifier que les photos s'affichent bien en miniature sous la visite. Vérifier que le suivi est partagé à l'usager et que le texte est bon.
Vérifier qu'on peut ouvrir les photos de visite en mode album. L'album ouvre toutes les photos liées au signalement, mais elles sont bien classées en fonction de leur type indépendamment de leur id. S'il y a plusieurs visites, elles sont bien classées par visite. Vérifier que dans l'album photo on a la description ajoutée à la photo de visite
Editer les types ou les descriptions des photos et des documents, vérifier que ça fonctionne bien.

hmeneuvrier · 2024-03-18T14:20:45Z

assets/controllers/back_signalement_view/form_upload_documents.js

+            <div class="fr-col-3">                
+                <input type="text" id="file-description" name="file[description]"
+                required="required" class="fr-input" placeholder="Description de l'image">
+                <input type="hidden" id="file-id" name="file[id]">


Ce champ de description n'est pas très pratique, mais je ne vois pas comment faire mieux sachant que chaque fichier ajouté est sur une fr-grid-row.
Des idées ?

Qu'entends tu par "pas très pratique" ? (j'ai pas encore testé)

Éventuellement le mettre sous chaque bloc sur toue la longueur (comme le file-error) mais pas sur que ce soit mieux, moi ca me va en l'état.

templates/back/signalement/view/visites/visites-buttons.html.twig

assets/controllers/back_signalement_edit_file/back_signalement_edit_file.js

src/Entity/Enum/DocumentType.php

hmeneuvrier · 2024-03-21T20:57:23Z

src/Entity/Enum/DocumentType.php

    {
        return [
+            self::PHOTO_SITUATION->name => self::PHOTO_SITUATION->label(),
+            self::AUTRE->name => self::AUTRE->label(),


J'ai mis AUTRE dans la liste des situations, mais c'est un peu bizarre... je m'explique, le type AUTRE est le type par défaut qui a été renseigné pour tous les documents et photos déjà existants avant la mise à jour des types de documents le mois dernier. Ce qui veut dire que tous les documents AUTRE dans les fiches de signalements vont apparaitre dans le haut de la fiche de signalement.
Ce qui veut aussi dire que quand on ajoute un document partenaire en bas, on ne peut pas lui affecter le type AUTRE si on ne trouve pas le type que l'on souhaite.

@mathildepoulpux surtout pour toi ce commentaire

assets/controllers/back_signalement_edit_file/back_signalement_edit_file.js

assets/controllers/back_signalement_view/form_upload_documents.js

sfinx13 · 2024-03-26T11:20:47Z

assets/controllers/back_signalement_view/form_upload_documents.js

+            <div class="fr-col-3">                
+                <input type="text" id="file-description" name="file[description]"
+                required="required" class="fr-input" placeholder="Description de l'image">
+                <input type="hidden" id="file-id" name="file[id]">


Qu'entends tu par "pas très pratique" ? (j'ai pas encore testé)

assets/controllers/back_signalement_view/form_upload_documents.js

src/Manager/SignalementManager.php

templates/back/signalement/view/photos-documents.html.twig

templates/back/signalement/view/visites/visite-item.html.twig

src/Manager/SuiviManager.php

src/Controller/Back/SignalementController.php

numew

Quelques remarques

assets/controllers/back_signalement_view/form_upload_documents.js

numew · 2024-03-26T13:38:36Z

assets/controllers/back_signalement_view/form_upload_documents.js

+            <div class="fr-col-3">                
+                <input type="text" id="file-description" name="file[description]"
+                required="required" class="fr-input" placeholder="Description de l'image">
+                <input type="hidden" id="file-id" name="file[id]">


Éventuellement le mettre sous chaque bloc sur toue la longueur (comme le file-error) mais pas sur que ce soit mieux, moi ca me va en l'état.

src/Controller/Back/SignalementFileController.php

src/Manager/SignalementManager.php

templates/back/signalement/view/visites/visites-buttons.html.twig

src/Manager/SuiviManager.php

templates/back/signalement/view/visites/visites-buttons.html.twig

numew

Test KO sur la visibilité des documents par l'usager

numew

Il reste un soucis : On peux envoyer plusieurs rapport de visite via la modale d'ajout du rapport de visite. Hors on a ensuite que le bouton permettant d'en visualiser un seul (idem quand on en supprime un c'est de façon unitaire)
Je ne sait pas comment on peux corriger ça

hmeneuvrier · 2024-03-29T16:42:46Z

Il reste un soucis : On peux envoyer plusieurs rapport de visite via la modale d'ajout du rapport de visite. Hors on a ensuite que le bouton permettant d'en visualiser un seul (idem quand on en supprime un c'est de façon unitaire) Je ne sait pas comment on peux corriger ça

@mathildepoulpux qu'en penses-tu ?

mathildepoulpux · 2024-04-03T10:02:51Z

Il reste un soucis : On peux envoyer plusieurs rapport de visite via la modale d'ajout du rapport de visite. Hors on a ensuite que le bouton permettant d'en visualiser un seul (idem quand on en supprime un c'est de façon unitaire) Je ne sait pas comment on peux corriger ça

@mathildepoulpux qu'en penses-tu ?

Bonne question. Déjà est-ce que ça arrive qu'on en envoie plusieurs pour une même visite ?
Parce que si c'est marginal j'aurais tendance à permettre un seul rapport par visite. Sinon, on peut réfléchir à une solution !

hmeneuvrier · 2024-04-04T09:03:48Z

Il reste un soucis : On peux envoyer plusieurs rapport de visite via la modale d'ajout du rapport de visite. Hors on a ensuite que le bouton permettant d'en visualiser un seul (idem quand on en supprime un c'est de façon unitaire) Je ne sait pas comment on peux corriger ça

@mathildepoulpux qu'en penses-tu ?

Bonne question. Déjà est-ce que ça arrive qu'on en envoie plusieurs pour une même visite ? Parce que si c'est marginal j'aurais tendance à permettre un seul rapport par visite. Sinon, on peut réfléchir à une solution !

@mathildepoulpux @numew j'ai vérifié sur Metabase, ça n'arrive jamais.
Du coup je limite à 1

numew · 2024-04-04T09:06:02Z

Il reste un soucis : On peux envoyer plusieurs rapport de visite via la modale d'ajout du rapport de visite. Hors on a ensuite que le bouton permettant d'en visualiser un seul (idem quand on en supprime un c'est de façon unitaire) Je ne sait pas comment on peux corriger ça

@mathildepoulpux qu'en penses-tu ?

Bonne question. Déjà est-ce que ça arrive qu'on en envoie plusieurs pour une même visite ? Parce que si c'est marginal j'aurais tendance à permettre un seul rapport par visite. Sinon, on peut réfléchir à une solution !

@mathildepoulpux @numew j'ai vérifié sur Metabase, ça n'arrive jamais. Du coup je limite à 1

Oui enfin, je pens qu'actuellement c'est impossible on risque pas de trouver des cas :) mais je suis d'accord

hmeneuvrier · 2024-04-04T09:33:42Z

@mathildepoulpux
Du coup si on limite à 1 pour le rapport ça nécessite pas mal d'ajustements, et je vois 3 solutions possibles :

on n'utilise pas du tout cette modale (car quel est son intérêt dans ce cas, vu que le type est prédéfini qu'il n'y a qu'un document, qu'on n'a pas de description) et on a juste un bouton qui ouvre un explorateur du système pour aller chercher un seul fichier.
on fait une modale spécifique, et dans ce cas comment elle est ? Quel est son comportement ?
on adapte la modale actuelle, mais il faut spécifier, on laisse la drop Area ou pas ? Une fois un fichier choisi qu'est-ce qui se passe ?

mathildepoulpux · 2024-04-04T09:54:10Z

@mathildepoulpux Du coup si on limite à 1 pour le rapport ça nécessite pas mal d'ajustements, et je vois 3 solutions possibles :

on n'utilise pas du tout cette modale (car quel est son intérêt dans ce cas, vu que le type est prédéfini qu'il n'y a qu'un document, qu'on n'a pas de description) et on a juste un bouton qui ouvre un explorateur du système pour aller chercher un seul fichier.

on fait une modale spécifique, et dans ce cas comment elle est ? Quel est son comportement ?

on adapte la modale actuelle, mais il faut spécifier, on laisse la drop Area ou pas ? Une fois un fichier choisi qu'est-ce qui se passe ?

La première solution me semble être la plus simple non ?

hmeneuvrier · 2024-04-05T08:04:25Z

@mathildepoulpux Du coup si on limite à 1 pour le rapport ça nécessite pas mal d'ajustements, et je vois 3 solutions possibles :

on n'utilise pas du tout cette modale (car quel est son intérêt dans ce cas, vu que le type est prédéfini qu'il n'y a qu'un document, qu'on n'a pas de description) et on a juste un bouton qui ouvre un explorateur du système pour aller chercher un seul fichier.

on fait une modale spécifique, et dans ce cas comment elle est ? Quel est son comportement ?

on adapte la modale actuelle, mais il faut spécifier, on laisse la drop Area ou pas ? Une fois un fichier choisi qu'est-ce qui se passe ?

La première solution me semble être la plus simple non ?

@mathildepoulpux Finalement, on reprend la modale d'édition du rapport de visite. Ca évite de complexifier le code, et c'est aussi simple pour l'utilisateur

sonarcloud · 2024-04-05T08:04:54Z

Quality Gate failed

Failed conditions
11.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

numew

Ok

sfinx13

Lecture et test OK

hmeneuvrier commented Mar 18, 2024

View reviewed changes

templates/back/signalement/view/visites/visites-buttons.html.twig Show resolved Hide resolved

hmeneuvrier commented Mar 18, 2024

View reviewed changes

assets/controllers/back_signalement_edit_file/back_signalement_edit_file.js Show resolved Hide resolved

hmeneuvrier force-pushed the feature/2198-bo-signalement-ajout-photos-visite branch from 8abbc9e to 89a59ff Compare March 21, 2024 09:03

hmeneuvrier commented Mar 21, 2024

View reviewed changes

src/Entity/Enum/DocumentType.php Outdated Show resolved Hide resolved

hmeneuvrier commented Mar 21, 2024

View reviewed changes

hmeneuvrier marked this pull request as ready for review March 21, 2024 21:05

hmeneuvrier assigned emilschn, sfinx13 and numew Mar 21, 2024

hmeneuvrier requested review from emilschn, sfinx13 and numew March 25, 2024 10:57

sfinx13 requested changes Mar 26, 2024

View reviewed changes

sfinx13 reviewed Mar 26, 2024

View reviewed changes

src/Manager/SuiviManager.php Outdated Show resolved Hide resolved

src/Manager/SuiviManager.php Outdated Show resolved Hide resolved

sfinx13 reviewed Mar 26, 2024

View reviewed changes

src/Controller/Back/SignalementController.php Outdated Show resolved Hide resolved

hmeneuvrier force-pushed the feature/2198-bo-signalement-ajout-photos-visite branch from 451a4ea to c1da34b Compare March 26, 2024 13:50

numew reviewed Mar 26, 2024

View reviewed changes

hmeneuvrier force-pushed the feature/2198-bo-signalement-ajout-photos-visite branch from 5c36f6a to 4a27845 Compare March 27, 2024 09:43

numew reviewed Mar 29, 2024

View reviewed changes

src/Manager/SuiviManager.php Show resolved Hide resolved

templates/back/signalement/view/visites/visites-buttons.html.twig Outdated Show resolved Hide resolved

numew reviewed Mar 29, 2024

View reviewed changes

hmeneuvrier force-pushed the feature/2198-bo-signalement-ajout-photos-visite branch from feee85d to df02357 Compare March 29, 2024 13:33

numew reviewed Mar 29, 2024

View reviewed changes

hmeneuvrier force-pushed the feature/2198-bo-signalement-ajout-photos-visite branch from e81be20 to 587792a Compare April 4, 2024 09:04

hmeneuvrier force-pushed the feature/2198-bo-signalement-ajout-photos-visite branch from 587792a to 6c94f6a Compare April 4, 2024 09:13

hmeneuvrier marked this pull request as draft April 4, 2024 09:24

hmeneuvrier added 10 commits April 4, 2024 17:10

add rapport and photos visites #2198

51f3706

wip suivi and miniature #2198

da87b78

sort photos and add description #2198

8c2bb60

edit photo description #2198

0a8eb36

select document between procedure and situation #2198

376607c

suivis #2198

f721e3f

edit description on nall photos #2198

edc735b

changes based on comments #2198

0086586

channges based on comments #2198

b72e252

changes based on comments #2198

52b0855

hmeneuvrier force-pushed the feature/2198-bo-signalement-ajout-photos-visite branch from 6c94f6a to 52b0855 Compare April 4, 2024 15:10

only one file for Rapport de visite #2198

bb6e648

hmeneuvrier marked this pull request as ready for review April 5, 2024 08:04

numew approved these changes Apr 5, 2024

View reviewed changes

sfinx13 approved these changes Apr 5, 2024

View reviewed changes

sfinx13 merged commit 47c1526 into develop Apr 5, 2024
2 of 3 checks passed

hmeneuvrier deleted the feature/2198-bo-signalement-ajout-photos-visite branch April 11, 2024 13:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BO - signalement] Possibilité d'ajouter des photos et rapport à une visite, et affichage de ces photos, mise à jour des suivis, catégorisation des types de documents #2343

[BO - signalement] Possibilité d'ajouter des photos et rapport à une visite, et affichage de ces photos, mise à jour des suivis, catégorisation des types de documents #2343

hmeneuvrier commented Mar 18, 2024 •

edited

Loading

hmeneuvrier Mar 18, 2024 •

edited

Loading

sfinx13 Mar 26, 2024

numew Mar 26, 2024 •

edited

Loading

hmeneuvrier Mar 26, 2024

hmeneuvrier Mar 21, 2024

hmeneuvrier Mar 26, 2024

sfinx13 Mar 26, 2024

numew left a comment

numew Mar 26, 2024 •

edited

Loading

numew left a comment •

edited

Loading

numew left a comment

hmeneuvrier commented Mar 29, 2024

mathildepoulpux commented Apr 3, 2024

hmeneuvrier commented Apr 4, 2024

numew commented Apr 4, 2024

hmeneuvrier commented Apr 4, 2024

mathildepoulpux commented Apr 4, 2024

hmeneuvrier commented Apr 5, 2024

sonarcloud bot commented Apr 5, 2024

numew left a comment

sfinx13 left a comment

[BO - signalement] Possibilité d'ajouter des photos et rapport à une visite, et affichage de ces photos, mise à jour des suivis, catégorisation des types de documents #2343

[BO - signalement] Possibilité d'ajouter des photos et rapport à une visite, et affichage de ces photos, mise à jour des suivis, catégorisation des types de documents #2343

Conversation

hmeneuvrier commented Mar 18, 2024 • edited Loading

Ticket

Description

Changements apportés

Pré-requis

Tests

hmeneuvrier Mar 18, 2024 • edited Loading

Choose a reason for hiding this comment

sfinx13 Mar 26, 2024

Choose a reason for hiding this comment

numew Mar 26, 2024 • edited Loading

Choose a reason for hiding this comment

hmeneuvrier Mar 26, 2024

Choose a reason for hiding this comment

hmeneuvrier Mar 21, 2024

Choose a reason for hiding this comment

hmeneuvrier Mar 26, 2024

Choose a reason for hiding this comment

sfinx13 Mar 26, 2024

Choose a reason for hiding this comment

numew left a comment

Choose a reason for hiding this comment

numew Mar 26, 2024 • edited Loading

Choose a reason for hiding this comment

numew left a comment • edited Loading

Choose a reason for hiding this comment

numew left a comment

Choose a reason for hiding this comment

hmeneuvrier commented Mar 29, 2024

mathildepoulpux commented Apr 3, 2024

hmeneuvrier commented Apr 4, 2024

numew commented Apr 4, 2024

hmeneuvrier commented Apr 4, 2024

mathildepoulpux commented Apr 4, 2024

hmeneuvrier commented Apr 5, 2024

sonarcloud bot commented Apr 5, 2024

Quality Gate failed

numew left a comment

Choose a reason for hiding this comment

sfinx13 left a comment

Choose a reason for hiding this comment

hmeneuvrier commented Mar 18, 2024 •

edited

Loading

hmeneuvrier Mar 18, 2024 •

edited

Loading

numew Mar 26, 2024 •

edited

Loading

numew Mar 26, 2024 •

edited

Loading

numew left a comment •

edited

Loading