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

[BO - Liste signalement][Back] Implémentation des filtres manquants #2421

Merged
merged 6 commits into from
Apr 19, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Apr 5, 2024

Ticket

#2410
#2423

Description

Récap et routes

Champ PR Lien
Territoires http://localhost:8080/bo/list/signalements/?page=1&territoires[]=13
Recherche http://localhost:8080/bo/list/signalements/?page=1&searchTerms=2022
Statut http://localhost:8080/bo/list/signalements/?page=1&status=ferme
Commune / EPCI http://localhost:8080/bo/list/signalements/?page=1&communes[]=Marseille&communes[]=Gex

http://localhost:8080/bo/list/signalements/?page=1&communes[]=13002

http://localhost:8080/bo/list/signalements/?page=1&epcis[]=244400503
Etiquettes http://localhost:8080/bo/list/signalements/?page=1&etiquettes[]=1&etiquettes[]=7
Date de dépôt avec les deux dates http://localhost:8080/bo/list/signalements/?page=1&dateDepotDebut=2023-03-01&dateDepotFin=2023-04-01
Date de dépôt avant la date de dépôt fin http://localhost:8080/bo/list/signalements/?dateDepotFin=2023-04-01
Date de dépôt après la date de dépôt début http://localhost:8080/bo/list/signalements/?page=1&dateDepotDebut=2023-03-01
Procédure estimée http://localhost:8080/bo/list/signalements/?page=1&procedure=rsd
Partenaires affectés http://localhost:8080/bo/list/signalements/?page=1&partenaires[]=22
http://localhost:8080/bo/list/signalements/?page=1&partenaires[]=AUCUN
Statut de la visite http://localhost:8080/bo/list/signalements/?page=1&visiteStatus=Planifi%C3%A9e
Type de dernier suivi http://localhost:8080/bo/list/signalements/?page=1&typeDernierSuivi=automatique
Date du dernier suivi http://localhost:8080/bo/list/signalements/?page=1&dateDernierSuiviDebut=2023-01-01&dateDernierSuiviFin=2023-12-31
Date du dernier suivi début http://localhost:8080/bo/list/signalements/?page=1&dateDernierSuiviDebut=2023-01-01
Date du dernier suivi fin http://localhost:8080/bo/list/signalements/?page=1&dateDernierSuiviFin=2023-12-31
Statut de l'affectation http://localhost:8080/bo/list/signalements/?page=1&statusAffectation=cloture_un_partenaire
Criticité http://localhost:8080/bo/list/signalements/?page=1&criticiteScoreMin=5&criticiteScoreMax=6
Déclarant http://localhost:8080/bo/list/signalements/?page=1&typeDeclarant=locataire
Nature du parc http://localhost:8080/bo/list/signalements/?page=1&natureParc=public
Allocataire http://localhost:8080/bo/list/signalements/?page=1&allocataire=caf
Allocataire oui http://localhost:8080/bo/list/signalements/?page=1&allocataire=oui
Allocataire non http://localhost:8080/bo/list/signalements/?page=1&allocataire=non
Enfants de - 6 ans http://localhost:8080/bo/list/signalements/?page=1&enfantsM6=non
Situation http://localhost:8080/bo/list/signalements/?page=1&situation=attente_relogement

Changements apportés

  • Ajout des filtres manquants (en gras dans les récap des routes) :
    • Procédure estimée
    • Type de dernier suivi
    • Date du dernier suivI
    • Statut de l'affectation
    • Situation
  • Prise en charge de la valeur non_renseigne pour les filtres suivantes :
    • nature parc
    • allocataire
    • enfant moins de 6 ans
  • Prise en charge d'une liste de communes provenant d'une EPCI (mise à jour du type commune dans le DTO)
  • Ajout petit jeu de donnée EPCI (fixtures)
  • Centralisation du nettoyage des communes avec arrondissements

Chargement des noms des EPCI dans la table commune

Pré-requis

Tests

@sfinx13 sfinx13 force-pushed the feature/2410-filter-missing branch from 0382aeb to 470242e Compare April 5, 2024 11:28
@sfinx13 sfinx13 marked this pull request as ready for review April 5, 2024 13:55
@sfinx13 sfinx13 force-pushed the feature/2410-filter-missing branch from db3d4b6 to e7a8403 Compare April 6, 2024 11:44
@sfinx13 sfinx13 force-pushed the feature/2410-filter-missing branch from e7a8403 to 0033cbb Compare April 6, 2024 11:54
src/Service/Signalement/SearchFilter.php Outdated Show resolved Hide resolved
src/Service/Signalement/SearchFilter.php Show resolved Hide resolved
src/Service/Signalement/SearchFilter.php Outdated Show resolved Hide resolved
src/Service/Signalement/SearchFilter.php Show resolved Hide resolved
src/Command/LoadEpciCommand.php Outdated Show resolved Hide resolved
src/Command/LoadEpciCommand.php Outdated Show resolved Hide resolved
src/Repository/CommuneRepository.php Outdated Show resolved Hide resolved
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.

J'ai mis des remarques sur la commande.

J'ai quelques questions/remarques :

  • On ne peux plus rechercher sur plusieurs territoires en même temps ?

  • Il manque la recherche par EPCI

  • On ne permet pas de recherche avec seulement une date de début ou de fin (plutot que les deux) ?

  • Idem pour la date de dernier suivi + prendre en compte la remarque d'helene et faire plutot le même système que pour la date de début/fin

  • Il faut penser à remettre l’ordre de tri par défaut que l'on avait auparavant (dernier date de suivi/création en premier)

Merci pour la description de la PR facilitant bien les tests.

src/Command/LoadEpciCommand.php Outdated Show resolved Hide resolved
src/Command/LoadEpciCommand.php Outdated Show resolved Hide resolved
src/Command/LoadEpciCommand.php Outdated Show resolved Hide resolved
Copy link
Collaborator Author

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

@numew réponses

On ne peux plus rechercher sur plusieurs territoires en même temps ?

Je l'ai ajouté pour la nouvelle liste

Il manque la recherche par EPCI

Oui, j'attendais un retour de Mathlde, je l'ai donc ajouté

On ne permet pas de recherche avec seulement une date de début ou de fin (plutot que les deux) ?

Ajusté

Idem pour la date de dernier suivi + prendre en compte la remarque d'helene et faire plutot le même système que pour la date de début/fin

Ajusté

Il faut penser à remettre l’ordre de tri par défaut que l'on avait auparavant (dernier date de suivi/création en premier)

régression corrigé

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/Service/Signalement/SearchFilter.php Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Apr 19, 2024

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

Relecture et retests OK

@hmeneuvrier hmeneuvrier merged commit 766c549 into develop Apr 19, 2024
2 of 3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/2410-filter-missing branch April 26, 2024 10:27
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

5 participants