Bug #78997
pas de réévaluation de la validité d'un champ après qu'une valeur y ait été posée par géocodage
0%
Description
Un champ carte et dessous des champs numéro / nom de la rue / commune, configurés comme obligatoire, si je vais d'abord dans ces champs et que j'en sors en les laissant vides, ils sont stylés avec l'info "! champ obligatoire". Si ensuite je clique sur la carte et que ça remplit les champs en question avec l'adresse à la position cliquée, l'info "! champ obligatoire" reste affichée.
Révisions associées
Historique
Mis à jour par Robot Gitea il y a 10 mois
- Statut changé de Nouveau à Solution proposée
Corentin Sechet (csechet) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/459
- Titre : js: refresh live validation on geolocation prefill (#78997)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/459/files
Mis à jour par Corentin Séchet il y a 10 mois
- Assigné à changé de Corentin Séchet à Benjamin Dauvergne
C'est un peu galère : les évènement JS purs (envoyés via element.dispatchEvent) sont bien gérés par jQuery ($element.on(...)). Dans l'autre sens en revanche, non, parce que jQuery ajoute une couche de gestion des évènements par-dessus le système d’évènements natifs. Donc la suppression de jQuery implique que n'importe où on ajoute un element.addHandler, on migre les $element.trigger correspondants vers dispatchEvent.
La manière "propre" de faire aurait été de migrer toute la fonction geoloc_prefill vers du JS natif, j'imagine. Mais je vais faire ma mauvaise tête : on pourra se permettre ce genre de refactoring quand on aura des tests unitaires JS, en l'état je trouve ça dangereux.
Je n'ai pas fait le tour en détail, mais un grep trigger('change') permet de voir les autres endroits où le problème est potentiellement présent.
J'ai changé le handler 'blur' vers un 'change' pour la mise à jour de la validation live, parce que ça faisait double emploi. Mais là dessus je veux bien une relecture parcimonieuse parce que le comportement n'est pas équivalent.
Mis à jour par Robot Gitea il y a 10 mois
- Statut changé de Solution proposée à En cours
Thomas Jund (tjund) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :
Mis à jour par Frédéric Péters il y a 10 mois
- Assigné à changé de Benjamin Dauvergne à Corentin Séchet
Mis à jour par Robot Gitea il y a 10 mois
- Statut changé de Solution proposée à En cours
Thomas Jund (tjund) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 10 mois
- Statut changé de Solution proposée à Solution validée
Thomas Jund (tjund) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 10 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/459
- Titre : js: refresh live validation on geolocation prefill (#78997)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/459/files
Mis à jour par Transition automatique il y a 10 mois
- Statut changé de Résolu (à déployer) à Solution déployée
js: refresh live validation on geolocation prefill (#78997)