Development #56734
Les sources de données automatiques (agenda) n'enregistrent ni ne notifient les erreurs
0%
Description
Voir par exemple https://demarches.mesdemarches06.fr/backoffice/settings/data-sources/46/
Ça cache notamment les erreurs de champ avec _display/_structure vide à la soumission.
Le chemin dans le code c'est : ItemField.get_display_value() -> NamedDataSource.get_display_value() -> NamedDataSource.get_structured_value() -> NamedDataSource.get_value_by_id() -> request_json_items() -> get_json_from_url() et là :
if data_source:
get_publisher().record_error(
error_summary,
context='[DATASOURCE]',
notify=data_source.get('notify_on_errors'),
record=data_source.get('record_on_errors'),
)
Les deux étant à False, ça ne fait rien. Par défaut je dirais bien d'enregistrer, et je serai pour aussi que NamedDataSource.record_on_errors soit à True par défaut aussi.
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Sujet changé de Les sources de données automatiques (agenda) n'enregistre ni ne notifient les erreurs à Les sources de données automatiques (agenda) n'enregistrent ni ne notifient les erreurs
Mis à jour par Frédéric Péters il y a plus de 2 ans
Ça cache notamment les erreurs de champ avec _display/_structure vide à la soumission.
Mais l'erreur serait plutôt à ce niveau-là, plutôt qu'essayer de la rattraper par un effet de bord ici, non ?
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Frédéric Péters a écrit :
Ça cache notamment les erreurs de champ avec _display/_structure vide à la soumission.
Mais l'erreur serait plutôt à ce niveau-là, plutôt qu'essayer de la rattraper par un effet de bord ici, non ?
Je n'essaie de rien rattraper du tout, j'essaie surtout d'avoir des informations. Si le référentiel foire une fois sur deux, en dehors de l'effet à la soumission ça a aussi un effet en général dans l'ergonomie du formulaire ou les champs listes seront vides, donc ça peut être une information intéressante, même si à la longue ça nous sera signalé directement.
Mais bien sûr qu'il faut corriger le code de soumission de w.c.s. pour ne pas perdre des données; on peut faire l'un et l'autre je pense.
Mis à jour par Benjamin Dauvergne il y a plus de 2 ans
Voilà j'ai ouvert #56824 concernant le fond du problème.
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/918
- Titre : WIP: misc: configure agenda data sources to notify errors (#56734)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/918/files
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne (bdauvergne) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 4 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/918
- Titre : misc: configure agenda data sources to notify errors (#56734)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/918/files
Mis à jour par Transition automatique il y a 4 mois
- Statut changé de Résolu (à déployer) à Solution déployée
misc: configure agenda data sources to notify errors (#56734)