Produits Entr'ouvert » Combo

Il faut déclencher le trigger lors du success en général (juste après le $button.attr('disabled', null);

Bon, je suis pas à l'aise en JS, mais je pense que quand tu déclenches un trigger, tu peux le faire sur un objet et envoyer des données. Ici tu le fais sur tout le document et sans rien préciser, et on voit dans le code de #8454 que tu fais un vaste « $(window).on('load combo:cellform-reload', function() { $('div.combo-cell-map').each(function() ... »

Donc j'aurais plutôt imaginé un $form.parents('div.cell').trigger('combo:combo-cell-saved'); ajouté après la ligne 173 ($button.attr('disabled', null);)

Et dans ce cas, tu dois récupérer le signal avec quelque chose dans ce genre dans le code de #8454 :
$('.combo-cell-map').on('combo:combo-cell-saved', et que vazy redessine la map)

(mais vraiment je suis pas à l'aise, je préférerai l'avis d'un tiers, j'ai juste l'impression qu'avec ta technique, on redessine toute les cartes trouvées sur la page, et pas juste celle qu'on vient d'enregistrer)

Serghei Mihai a écrit :

D'accord avec toi, mais à mon avis signal doit être envoyé après que le formulaire soit rechargé, pour que la fonction qui écoute le trigger puisser réagir.

En fait ce qui me "dérange" c'est que ça va pas être envoyé si y'a du ckeditor, donc je parlais surtout d'envoyer le signal en dehors du «if (data.indexOf('ckeditortype') == -1)». Mais ça peut effectivement être fait à la fin du "success".

Bon je termine sur "combo:cellform-reload", pour moi un nom de signal ce n'est pas le nom de l'action qui doit être faite (ça c'est le receveur qui décide) mais plutôt l'action qui vient d'être faite. Donc ici plutôt "combo:cellform-saved"

(la description du ticket peut-elle être étendue sur un exemple d'application ? c'est assez abstrait pour le moment).

Frédéric Péters a écrit :

(la description du ticket peut-elle être étendue sur un exemple d'application ? c'est assez abstrait pour le moment).

En fait j'ai demandé à Serghei d'extraire ce signal du patch de la cellule carte sur #8454. On y voit donc là bas un exemple d'utilisation de ce signal, qui s'occupe de relancer la machinerie leaflet après que l'utilisateur a cliqué sur "enregistré" ; sinon la carte devient un vide. Et comme Serghei n'a pas trouvé de signal existant, il propose d'ajouter celui-ci.

Je pense que j'aurais une préférence pour qu'une partie <script> présente dans un ajax_whatever_form.html d'une cellule soit automatiquement exécuté.

Ça demande une modification à PageEditCellView pour qu'il regarde si un template spécifique existe (plutôt que prendre ajax_cell_form.hmtl en toute situation), et à cell_form.html (utilisé pour le rendu initial) pour également utiliser celui-ci (plutôt qu'hardcoder form.as_p).

Je pense que ce n'est pas trop compliqué et que ça nous laissera du code plus facile à maintenir (pas toujours facile de suivre les événements js).

Alternativement l'approche prise dans #7344 (pour la cellule gallerie, pas intégrée) était d'utiliser l'existant attribut manager_form_template pour avoir le propre template de rendu, avec le <script> dedans (mais c'était il y a longtemps, pas sûr du fonctionnement avec le rechargement ajax via le bouton save, pas sur en fait que le bouton save ait été utile dans cette cellule.

Cela étant, vu qu'il y a déjà dix commentaires pour un patch d'une ligne (parce que je dirais de virer la construction $.Event() et d'utiliser une chaine directement dans l'appel .trigger()), peut-être est-il bon d'en rester là.

Ack

(mais oui pour passer à terme à la solution de Fred qui ajouter en plus de la souplesse dans l'affichage des espace de configuration des cellules)