Développement #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.
Files
Associated revisions
dataviz: reconstruct ajax filters cell url on each call (#66956)
History
Updated by Valentin Deniaud over 2 years ago
- File 0001-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch 0001-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Benjamin Dauvergne over 2 years ago
- Status changed from Solution proposée to 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.
Updated by Valentin Deniaud over 2 years ago
- File 0001-data-add-cell-get_ajax_url-method-66956.patch 0001-data-add-cell-get_ajax_url-method-66956.patch added
- File 0002-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch 0002-dataviz-reconstruct-ajax-filters-cell-url-on-each-ca.patch added
- Status changed from En cours to 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 ?
Updated by Benjamin Dauvergne over 2 years ago
- Status changed from Solution proposée to Solution validée
Oui c'est encore mieux.
Updated by Valentin Deniaud over 2 years ago
- Status changed from Solution validée to 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)
Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
data: add cell get_ajax_url method (#66956)