Projet

Général

Profil

Development #54326

ne pas notifier les erreurs sources de données / appels webservice lors de la "prévisualisation"

Ajouté par Frédéric Péters il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Lauréline Guérin
Version cible:
-
Début:
27 mai 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Quand on fait le rendu de toutes les pages d'un formulaire sous le paramétrage de celui-ci, on n'a pas le contexte de remplissage normal, du coup des champs configurés avec des sources de données ou appels webservices peuvent se trouver avec des mauvaises URL, ce qui fera des 404, et s'il y a configuration pour être notifié d'une erreur il y aura notification.

exemple sur un gabarit qui dirait {{agendas_url}}api/agenda/{{form_var_thematique_raw}}-{{form_var_lieu_raw}}/datetimes/

Il faudrait désactiver les notifs/enregistrements d'erreur lors du rendu de cette zone.

Fichiers

0001-data_source-don-t-record-errors-on-preview-54326.patch (1,94 ko) 0001-data_source-don-t-record-errors-on-preview-54326.patch Lauréline Guérin, 27 mai 2021 16:18
0001-data_source-don-t-record-errors-on-preview-54326.patch (4,84 ko) 0001-data_source-don-t-record-errors-on-preview-54326.patch Lauréline Guérin, 27 mai 2021 17:34
0001-data_source-don-t-record-errors-on-preview-54326.patch (7,75 ko) 0001-data_source-don-t-record-errors-on-preview-54326.patch Lauréline Guérin, 28 mai 2021 11:48

Révisions associées

Révision 08bb5f73 (diff)
Ajouté par Lauréline Guérin il y a presque 3 ans

data_source: don't record errors on preview (#54326)

Historique

#1

Mis à jour par Lauréline Guérin il y a presque 3 ans

  • Assigné à mis à Lauréline Guérin
#2

Mis à jour par Lauréline Guérin il y a presque 3 ans

#3

Mis à jour par Frédéric Péters il y a presque 3 ans

Je ne pensais pas à cette prévisualisation mais à la prévisualiation des champs sous un formualaire; /backoffice/forms/231/

#4

Mis à jour par Lauréline Guérin il y a presque 3 ans

oups en effet :)
je garde quand même pour la preview data_source (admin) ?

#5

Mis à jour par Lauréline Guérin il y a presque 3 ans

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

Mis à jour par Frédéric Péters il y a presque 3 ans

Oui c'est bien utile de l'avoir également là.

#7

Mis à jour par Lauréline Guérin il y a presque 3 ans

pas super propre :/

#8

Mis à jour par Frédéric Péters il y a presque 3 ans

Il y aurait aussi les appels webservice à traiter et ça ne va pas pouvoir être géré ainsi; je me disais mais pareillement pas pas propre que le rendu "preview" pourrait être entouré d'un get_request().disable_error_notifications = True / = False.

#9

Mis à jour par Lauréline Guérin il y a presque 3 ans

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

décidément, j'ai vraiment trop lu en travers ...

#11

Mis à jour par Frédéric Péters il y a presque 3 ans

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

Mis à jour par Lauréline Guérin il y a presque 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 08bb5f73873cec6029199e8685bd323f9fd97e38
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu May 27 16:17:22 2021 +0200

    data_source: don't record errors on preview (#54326)
#13

Mis à jour par Frédéric Péters il y a presque 3 ans

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

Formats disponibles : Atom PDF