Development #66956
dataviz, éviter de concaténer l'url de rafraîchissement ajax
0%
Description
La querystring se retrouve recopiée une fois de plus à chaque appel, la méthode de construction de l'URL est foireuse.
Fichiers
Révisions associées
dataviz: reconstruct ajax filters cell url on each call (#66956)
Historique
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch 0001-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Solution proposée à En cours
J'aurait été plus à l'aise si tu mettais en cache l'URL d'origine que de la reconstruire entièrement avec du template comme c'est fait dans public/templates/combo/placeholder.html.
Donc plutôt :
// Keep base ajax_cell_url if (chart_cell.ajax_cell_url == undefined) { chart_cell.ajax_cell_url = chart_cell.data('ajax-cell-url'); } new_url = chart_cell.ajax_cell_url + '?filters_cell_id=' + $('body').data('filters-cell-id') + '&' + $(this).serialize();
Si la durée de vie de l'objet chart_cell ne correspond pas à ce qu'on souhaite faire, le stocker ailleurs.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-data-add-cell-get_ajax_url-method-66956.patch 0001-data-add-cell-get_ajax_url-method-66956.patch ajouté
- Fichier 0002-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch 0002-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch ajouté
- Statut changé de En cours à Solution proposée
Benjamin Dauvergne a écrit :
Si la durée de vie de l'objet chart_cell ne correspond pas à ce qu'on souhaite faire, le stocker ailleurs.
Aucune idée et le sens de mon patch était plutôt d'aller vers moins de dépendance à ce qui se passe dans le JS, ça te va sans la duplication de code ?
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Oui c'est encore mieux.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit f3c3e3301e2331300753b0e0d3648ee77b21dd00 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon Jul 4 17:02:11 2022 +0200 dataviz: reconstruct ajax filters cell url on each call (#66956) commit 0996d0923a372e87b05fccc9d56ae37b906a7b61 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Jul 5 10:54:02 2022 +0200 data: add cell get_ajax_url method (#66956)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
data: add cell get_ajax_url method (#66956)