Bug #7996
Le menu latéral n'apparait pas si un des jsonp de COMBO_KNOWN_SERVICES n'est pas récupérable
0%
Description
Dans ce code:
$.ajax({url: element.backoffice_menu_url, xhrFields: { withCredentials: true }, async: true, dataType: 'jsonp', crossDomain: true, success: function(data) { element.data = data; check_all_done(); }, error: function(error) { console.log('bouh', error); element.data = Array(); check_all_done(); } } );
le calback error
n'est jamais appelé car jQuery ne gère pas les erreurs sur du jsonp cross-domain (voir http://api.jquery.com/jQuery.ajax/ sur le paramètre error
et http://stackoverflow.com/questions/19035557/jsonp-request-error-handling#answer-19207855 pour des contournements).
Coté le w.c.s. le bug n'est pas visible car w.c.s. semble construire son menu latéral tout seul sans passer par create_menu_items()
de publik.js
(un breakpoint ne s'arrête jamais dedans pourtant je vois bien un menu).
Je me dis qu'on pourrait tenter une construction progressive du menu, en laissant les entrées en cours de chargement avec un logo pulsatile (comment traduire throbber ? qui n'existe même pas dans un dico d'anglais) puis développer l'entrée quand c'est terminé.
Fichiers
Historique
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-gadjo-add-progressive-rendering-of-publik-menu-7996.patch 0001-gadjo-add-progressive-rendering-of-publik-menu-7996.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 8 ans
Mis à jour par Thomas Noël il y a plus de 8 ans
Ça m'a l'air bon mais j'ai pas testé en vrai, na.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit de36956369124b119510431835c3d57bbb1d52bf Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Sep 16 18:52:15 2015 +0200 gadjo: add progressive rendering of publik menu (#7996)
Mis à jour par Frédéric Péters il y a plus de 8 ans
Et validé installé sur https://demo-agent.combo.dev.entrouvert.org/