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

[FO/BO - Signalement] Mise à jour des types de documents #2221

Merged
merged 7 commits into from
Feb 12, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Feb 6, 2024

Ticket

#2155
#2203
#2202

Description

Mise à jour des types de documents dans les possibilités actuelles
Ajouter une colonne description pour les fichiers (utilisée pour les photos de visite)
Migration pour passer les documents et photos existantes en type "Autre"
Mise à jour du uploadedBy sur les fichiers lors de la création de signalement et de l'ajout de suivis
Mise à jour de l'affichage du bouton "supprrimer le fichier" en fonction des specs https://github.com/MTES-MCT/histologe/wiki/Ajout-documents#bo---supprimer-les-documents-et-photos

Changements apportés

  • Création d'une migration
  • Mise à jour des controllers
  • Mise à jour des loaders de fixtures
  • Mise à jour de l'entité File pour ajouter la description et des fonctions isUsagerFile et isPartnerFile
  • Mise à jour de la FileFactory
  • Mise à jour des FileManager et InterventionMmanager
  • Mise à jour du SignalementFileProcessor
  • Mise à jour des tests

Filtres listing signalement - Gestion de doc - FO (1)
Filtres listing signalement - Gestion de doc - BO

Pré-requis

docker compose exec -ti histologe_phpfpm php bin/console doctrine:migrations:execute --down 'DoctrineMigrations\Version20240205143843'
docker compose exec -ti histologe_phpfpm php bin/console doctrine:migrations:execute --up 'DoctrineMigrations\Version20240205143843'

Tests

  • Vérifier la table files en base qu'il y ait bien la colonne description et que tous les fichiers aient un document_type AUTRE s'ils n'avaient pas de document_type
  • Faire un signalement en locataire depuis le nouveau formulaire en ajoutant des photos de désordres, et tous les documents demandés (bail, état des lieux, dpe, diagnostique plomb dans le désordre logement/sécurité) -> vérifier en base le document_type des photos et documents, et le uploaded_by_id
  • Dans le BO, ajouter photo et document, et vérifier le document_type en base
  • Dans le BO, ajouter une visite dans le passé avec un fichier rapport de visite et vérifier le document_type en base
  • Faire un suivi visible à l'usager
  • En usager, depuis la page de suivi usager, ajouter photo et document, et vérifier le type en base (et le uploaded_by_id)
  • Sur un ancien signalement, ajouter des photos et documents avec plusieurs agents puis vérifier qu'on ne peut supprimer que les documents partenaires, que si le signalement est en cours, et que si soit on est admin territoire ou super admin, soit on est agent mais c'est notre fichier
  • Idem sur un nouveau signalement

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quelques commentaires ; pas encore testé !

src/Controller/Back/SignalementFileController.php Outdated Show resolved Hide resolved
src/Entity/File.php Show resolved Hide resolved
@hmeneuvrier hmeneuvrier force-pushed the feature/2155-update-types-documents branch from e028737 to ff5ec10 Compare February 8, 2024 16:39
src/Entity/File.php Outdated Show resolved Hide resolved
src/Entity/File.php Show resolved Hide resolved
@hmeneuvrier
Copy link
Collaborator Author

hmeneuvrier commented Feb 9, 2024

  • A la création du signalement, mettre l'id de l'usager dans les fichiers
  • Sur la page de suivi, mettre l'id de l'usager dans les fichiers
  • Changer l'affichage de la page signalement pour ne plus se baser que sur null (peut-être garder null pour les anciens signalements)
  • On garde tel quel avec les rôles pour déterminer si c'est un document usager ou partenaire

@hmeneuvrier hmeneuvrier force-pushed the feature/2155-update-types-documents branch from ff5ec10 to 2ad795f Compare February 9, 2024 11:21
Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests et relecture OK

src/Entity/File.php Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Feb 12, 2024

Quality Gate Passed Quality Gate passed

Issues
5 New issues

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

See analysis details on SonarCloud

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK lecture et tests

@emilschn emilschn merged commit b89fbc6 into develop Feb 12, 2024
3 checks passed
@sfinx13 sfinx13 deleted the feature/2155-update-types-documents branch February 13, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants