Development #46031
lever une erreur quand get_value_by_id() donne quelque chose avec un id différent
0%
Description
Il y a des sources de données qui se trouvent configurées pour interroger wcs avec "Paramètre d’identifiant" configuré avec "id", alors que non ce paramètre n'existe pas.
Le résultat c'est que le paramètre est ignoré et plouf on récupère toute la liste, dont on prend le premier élément,
items = request_json_items(url, self.data_source)
if not items: # None or empty list are not valid
return None
if hasattr(request, 'datasources_cache'):
request.datasources_cache[url] = items
return items[0]
(ça aussi ça pourrait être discuté, et exiger qu'il y ait un seul élément).
et plouf les données ne correspondent pas à ce qui a été sélectionné.
Je serais pour ajouter une vérification ici, comme quoi l'id des données obtenues correspond bien à l'id demandé, et si ce n'est pas le cas LoggedError.record() et return None.
Fichiers
Demandes liées
Révisions associées
wf: accept empty values in mapping of edit card action (#46031)
Historique
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0001-datasources-check-item-id-46031.patch 0001-datasources-check-item-id-46031.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 3 ans
Tu peux rebaser ta branche sur master, ça corrigera l'erreur tox/pytest qui empêche les tests de se faire (#46252)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a plus de 3 ans
commit fa4a58429d49e4ebe69aa85e607861eb936f6625
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Tue Sep 1 14:34:52 2020 +0200
datasources: check item id (#46031)
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Lié à Bug #47974: les sources JSON qui renvoient des id sous forme d'entiers ne fonctionnement plus en autocomplétion ajouté
datasources: check item id (#46031)