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 - Date d'entrée] Correction du type de champ pour la date d'entrée du logement dans le formulaire NDE #3507

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Jan 2, 2025

Ticket

#3435

Description

Depuis le nouveau formulaire, on enregistre la vraie date d'entrée dans le logement.
Malheureusement, la modale pour éditer la NDE écrase cette donnée avec des dates arbitraires (01/01/1970 et 02/01/2023) qui servaient à savoir si on était avant ou après 2023.
Heureusement, on a conservé la donnée enregistrée à la base dans le formulaire.

Changements apportés

  • migration pour remplacer la colonne date_entree si elle est égale aux dates arbitraires, et qu'on a une date réelle d'enregistrée
  • remplacement des boutons radio par un champ de saisie (identifique à celui de la modale d'édition annexe
  • enregistrement dans le json de type_composition_logement pour la sauvegarder aux deux endroits

Tests

  • Vérifier que l'édition d'un signalement fonctionne dans la modale NDE, et que l'affichage est toujours conforme
  • Vérifier que l'édition d'un signalement fonctionne dans la modale Informations logement, et que l'affichage est toujours conforme
  • Modifier la date d'entrée à la main en BDD pour avoir une des deux dates arbitraires de NDE
  • make execute-migration name=Version20250102154204 direction=up
  • Vérifier que la date du json est bien reprise

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.

migration KO pour moi( mais pas nécessaire)
J'ai mis une question dans le code, j'en ai une autre :

On ne modifie pas le "Entrée dans le logement" sur la partie NDE (je n'ai pas d'avis, je me pose la question) ?
Screenshot 2025-01-03 at 17-55-22 #2025-1 Signalement - Histologe

migrations/Version20250102154204.php Show resolved Hide resolved
src/Manager/SignalementManager.php Show resolved Hide resolved
@emilschn
Copy link
Collaborator Author

emilschn commented Jan 6, 2025

migration KO pour moi( mais pas nécessaire) J'ai mis une question dans le code, j'en ai une autre :

On ne modifie pas le "Entrée dans le logement" sur la partie NDE (je n'ai pas d'avis, je me pose la question) ? Screenshot 2025-01-03 at 17-55-22 #2025-1 Signalement - Histologe

Ca me semble compliqué parce que pour certains, on n'a qu'une fausse date :)
Sauf si on fait un if sur les dates "par défaut" ? et si c'est pas le cas, on met la vraie date ?

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.

Ok

src/Manager/SignalementManager.php Show resolved Hide resolved
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.

Dans la partie Informations sur le logement, les 2 dates ne sont pas formatées de la même manière :
image

src/Manager/SignalementManager.php Show resolved Hide resolved
@@ -487,7 +484,8 @@ public function updateFromInformationsLogementRequest(
->setBailDpeBail($informationsLogementRequest->getBailDpeBail())
->setBailDpeEtatDesLieux($informationsLogementRequest->getBailDpeEtatDesLieux())
->setBailDpeDpe($informationsLogementRequest->getBailDpeDpe())
Copy link
Collaborator

Choose a reason for hiding this comment

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

Dans la modale NDE, je met "non" pour DPE, dans l'information logement c'est toujours "oui" (et, si je met ensuite "ne sait pas" dans information logement, ça reste à "non" dans la partie NDE)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok c'est corrigé !

Copy link
Collaborator

Choose a reason for hiding this comment

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

je ne vois pas les changements, cc'est aussi un bug de rafraichissement ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

En fait, si je change dans la modale NDE, ce n'est pas répercucté dans l'onglet Foyer / Informations sur le logement.

Mais l'inverse fonctionne bien

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Trop bizarre... Je vais voir

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, j'ai compris : dans ce sens, c'est quand le DPE était false (et donc empty) que ça ne marchait pas... c'est corrigé !

@emilschn emilschn force-pushed the fix/3435-date-entree-logement branch from 084d55e to c35ae9c Compare January 8, 2025 10:01
Copy link

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.

3 participants