Project

General

Profile

Development #29124

Afficher un message d'erreur lorsqu'une cellule ne peut pas afficher son contenu

Added by Emmanuel Cazenave 12 months ago. Updated 24 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
17 Dec 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

De #29104, une cellule qui n'arrive pas à charger son contenu affiche 'Chargement...' dans un bandeau rouge, un message d'erreur mettrait fin au suspense.

J'arrive à reproduire avec un cellule de type 'profil' : flinguer la vue 'ajax_page_cell' en y lançant une exception, on voit bien qu'on reste sur 'Chargement...'.

Lecture de combo.public.js, je vois combo_load_cell qui place le rouge mais qui ne change pas le texte (et qui devrait afficher un smiley dans la console que je n'y trouve pas).

0001-cell-display-error-message-on-ajax-call-failure-2912.patch View (2.89 KB) Lauréline Guerin, 21 Oct 2019 02:39 PM

Associated revisions

Revision be705385 (diff)
Added by Lauréline Guérin 26 days ago

cell: display error message on ajax call failure (#29124)

History

#1 Updated by Lauréline Guerin about 2 months ago

  • Assignee set to Lauréline Guerin

#2 Updated by Lauréline Guerin about 2 months ago

quelque chose comme ça ?

#3 Updated by Emmanuel Cazenave 27 days ago

  • Status changed from Solution proposée to Solution validée

Testé, ok pour moi.

Pour mémoire, ça couvre le cas où la vue ajax_page_cell plante, genre timeout lors de l'appel vers wcs.

Si wcs fait 502, c'est une autre histoire (la cellule est vide, avec ou sans ce patch), parce que ajax_page_cell ne plante pas.

#4 Updated by Lauréline Guerin 26 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit be7053850ff3c2b144904e25aed54e0f92521355
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Oct 21 14:38:29 2019 +0200

    cell: display error message on ajax call failure (#29124)

#5 Updated by Frédéric Péters 24 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF