Bug #8108
Utiliser "synchronous" (plutôt que "ajax") comme indicateur qu'on veut un rendu synchrone
0%
Description
Pour le moment le rendu synchrone d'une cellule a lieu quand le context a "ajax: True"; ça vient du fait que le rendu synchrone, on décide de l'utiliser quand on fait le rendu pour un appel ajax (où l'asynchrone vient du navigateur).
Mais quand on fait le rendu momo, on doit aussi faire du rendu synchrone et là ça fait bizarre d'avoir le paramètre appelé "ajax".
Ici je propose de plutôt appeler ça "synchronous".
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-misc-use-synchronous-as-context-key-to-request-synch.patch 0001-misc-use-synchronous-as-context-key-to-request-synch.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 8 ans
Moi j'ai pas compris... pour moi, ajax=True veut dire asynchone, et pas synchrone. Non ?
Mis à jour par Frédéric Péters il y a plus de 8 ans
ajax=True ça veut dire "oh ça va servir dans le rendu ajax d'une cellule", mais le "rendu ajax d'une cellule", c'est justement du rendu synchrone (il y a eu un premier temps où de l'html a été envoyé au navigo, qui fait une requête vers l'url du rendu ajax, qui fait le rendu désiré, qui est ensuite affiché dans le navigo).
Le "oh ça va servir dans le rendu ajax d'une cellule", avec Momo, il devient "oh ça va servir dans le rendu ajax d'une cellule ou dans le rendu pour momo".
Plutôt que ça, donc, on change en "synchronous: True", qui dit qu'on veut le rendu du contenu de la cellule, point, peu importe le temps que ça prenne. (et surtout ne nous donne pas un wrapper qui demandera un appel http derrière, merci).
Mis à jour par Thomas Noël il y a plus de 8 ans
J'avais donc compris le ajax=True à l'envers, preuve que ton patch est doublement intéressant. Go.
Mis à jour par Frédéric Péters il y a plus de 8 ans
Il suit #6842, si quelqu'un pouvait y regarder, ce serait super.
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 cfbf3bf6a7d842a84a6f1dcee7c39565e2d14e34
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Aug 24 14:59:46 2015 +0200
misc: use "synchronous" as context key to request synchronous rendering (#8108)
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: use "synchronous" as context key to request synchronous rendering (#8108)