Projet

Général

Profil

Development #76376

base adresse : forcer la recherche sur type=housenumber

Ajouté par Thomas Noël il y a environ un an. Mis à jour il y a 12 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 avril 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

La recherche par l'API BAN a un paramètre "type" qui permet de dire le type d'objet recherche. Et apparement ça marche maintenant (ce n'était pas le cas lors de #59633)

type : type de résultat trouvé
*    housenumber : numéro « à la plaque »
*    street : position « à la voie », placé approximativement au centre de celle-ci
*    locality : lieu-dit
*    municipality : numéro « à la commune »

Si on ne précise rien, l'API renvoie un peu de tout... et donc typiquement si on tape "de gaulle" ça ne renvoie que des type=street, des rues... et pas des adresses. Pire : parfois on tape une adresse complète et ça renvoie quand même des rues. Et ça fait planter des formulaires.

Comme on a un connecteur qui est prévu pour ne renvoyer que des adresses, je propose d'envoyer type=housenumber par défaut.


Fichiers

Révisions associées

Révision 84cd5195 (diff)
Ajouté par Benjamin Dauvergne il y a 12 mois

base_adresse: add parameter type=housenumber to prevent users from picking street addresses (#76376)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ un an

Mauvaise idée je pense, si on prend le cas de Nanterre si on met housenumber ça ne renvoie juste plus rien :/ Normalement addok est assez malin pour renvoyer des housenumber si on a mis un chiffre et sinon la rue en premier. Par contre on pourrait filtrer tout ce qui n'est pas housenumber ou street.

#2

Mis à jour par Thomas Noël il y a environ un an

Benjamin Dauvergne a écrit :

Mauvaise idée je pense, si on prend le cas de Nanterre si on met housenumber ça ne renvoie juste plus rien :/

En fait c'est une coïncidence, la BAN sur Nanterre est complétement plantée : elle n'a plus une seule adresse, uniquement des "street"... Je crois qu'on a déjà vu un tel plantage chez un autre client.

Pour le coup, en envoyant le type=housenumber ça permet d'immédiatement voir le bug BAN.

Normalement addok est assez malin pour renvoyer des housenumber si on a mis un chiffre et sinon la rue en premier. Par contre on pourrait filtrer tout ce qui n'est pas housenumber ou street.

Je n'ai aucune confiance dans la logique de addok et de sa recherche, ça donne souvent du grand n'importe quoi.

Moi je voudrais qu'on obtienne toujours uniquement des "housenumber". Ce sont les adresses au sens commun du terme. Le fait que la BAN renvoie de temps en temps des "street" fait planter silencieusement pas mal de nos formulaires ou workflows à geoloc.

#3

Mis à jour par Benjamin Dauvergne il y a environ un an

Vu en jabber avec toto, on aurait une idée pour laisser les rues explorables tout en ne permettant que la sélection d'adresses :
  • marqués tous les enregistrements de type autre que "housenumber" avec "disabled": True ainsi ils ne seront pas sélectionnables, mais seront visibles;
  • si il n'y a aucun enregistrement de type "housenumber" dans la liste mais qu'elle est non vide, alors on va en plus ajouter un choix non sélectionnable en début de liste intitulé "Veuillez indiquez un numéro dans la rue".
#4

Mis à jour par Benjamin Dauvergne il y a environ un an

  • Assigné à mis à Benjamin Dauvergne
#5

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à En cours

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Benjamin Dauvergne il y a environ un an

Et en utilisant une propriété "children" vide pour l'indication plutôt que "disabled".

#9

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de En cours à Solution proposée
#10

Mis à jour par Benjamin Dauvergne il y a environ un an

À noter pour le/la relecteur/relectrice, ça n'a pas d'impact sur l'existant puisqu'il faut explicitement ajouter type=housenumber aux appels.

#11

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution proposée à En cours

Thomas NOËL (tnoel) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

#13

Mis à jour par Benjamin Dauvergne il y a environ un an

  • Statut changé de En cours à Solution proposée
#14

Mis à jour par Robot Gitea il y a 12 mois

  • Statut changé de Solution proposée à Solution validée

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#15

Mis à jour par Robot Gitea il y a 12 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#16

Mis à jour par Transition automatique il y a 12 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#17

Mis à jour par Transition automatique il y a 10 mois

Automatic expiration

Formats disponibles : Atom PDF