Development #72263
base_adresse: retourner l'id recherché sur une recherche d'adresse par id
Début:
12 décembre 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Cf. ticket client #72257
Recherche pas ID pour une adresse dont la casse à changer:- l'ancien ID :
13213_3680~43.325554~5.430226~Avenue de FraisVallon 13013 Marseille
- la recherche fonctionne:
- mais l'adresse a un nouvel id:
13213_3680~43.325555~5.430234~Avenue de frais vallon 13013 Marseille
- et w.c.s compare l'id recherché avant de répondre, donc boum
Le plus simple pour pallier à cela serait de renvoyer l'id qu'on a reçu plutôt que l'id qu'on recrée, ça fournira de la stabilité aux clients de l'API (on pourrait retourner une clé old_id et adapter w.c.s. mais beaucoup de travail pour pas grand chose).
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus d'un an
renvoyer l'id qu'on a reçu plutôt que l'id qu'on recrée
Pouce bleu !
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-base_adresse-set-result-id-to-lookup-id-72263.patch 0001-base_adresse-set-result-id-to-lookup-id-72263.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Fichier 0001-base_adresse-set-result-id-to-lookup-id-72263.patch 0001-base_adresse-set-result-id-to-lookup-id-72263.patch ajouté
Avec un test.
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 479a455cac44ed223a4fd1a936f7f05b93ed7839 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Mon Dec 12 10:10:05 2022 +0100 base_adresse: set result id to lookup id (#72263) base_adresse data source ids are not canonical, many can match the same adress. But clients of the API does not handle this well, so it's better to always return the same id event if we know it has changed.
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
base_adresse: set result id to lookup id (#72263)
base_adresse data source ids are not canonical, many can match the same
adress. But clients of the API does not handle this well, so it's better
to always return the same id event if we know it has changed.