entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

dataviz: automatically refresh cells on filter change (#72465) #21

Merged
vdeniaud merged 1 commits from wip/72465-dataviz-tenter-le-rafraichisseme into main 2023-01-16 09:37:23 +01:00
Conversation 5 Commits 1 Files Changed 2 +16 -3
2 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
combo/apps/dataviz/templates/combo/chart-filters.html
View File

@ -23,11 +23,12 @@
<script>
$(function () {
const form = $('#chart-filters');
var loaded_cell_count = 0;
if (!$('body').data('filters-cell-id')) {
$('body').data('filters-cell-id', Math.random().toString(36).slice(2, 7));
var loaded_cell_count = 0;
function load_filters_cell_last() {
if (++loaded_cell_count == $('div.chartngcell').length) {
combo_load_cell($('.chart-filters-cell'));
@ -54,7 +55,8 @@
end_field.parent().hide();
}
}).change();
$('#chart-filters').submit(function(e) {
form.submit(function(e) {
e.preventDefault();
$(window).trigger('combo:refresh-graphs');
chart_cell = $(this).parents('.cell');
@ -62,6 +64,12 @@
new_url = ajax_cell_url + '?filters_cell_id=' + $('body').data('filters-cell-id') + '&' + $(this).serialize();
chart_cell.data('ajax-cell-url', new_url);
});
tjund commented 2023-01-04 17:46:58 +01:00
Outdated
Review
Copy Link

Pourquoi ne pas simplement ?

$('form#chart-filters').change(function()

Pourquoi ne pas simplement ? `$('form#chart-filters').change(function()`
vdeniaud commented 2023-01-05 10:24:19 +01:00
Outdated
Review
Copy Link

Par mimétisme avec ce qui est fait côté BO, effectivement j'ai enlevé et ça fonctionne.

(côté BO ça crée une boucle infinie, pas 100% sûr à cause de quoi)

Par mimétisme avec ce qui est fait côté BO, effectivement j'ai enlevé et ça fonctionne. (côté BO ça crée une boucle infinie, pas 100% sûr à cause de quoi)
tjund commented 2023-01-05 12:03:15 +01:00
Outdated
Review
Copy Link

$('form#chart-filters') utilisé 3x.
Pour des questions de performance, ne pas parser 3x le dom. Parser 1x et stocker dans 1 `const form = $('#chart-filters');

`$('form#chart-filters')` utilisé 3x. Pour des questions de performance, ne pas parser 3x le dom. Parser 1x et stocker dans 1 `const form = $('#chart-filters');
tjund commented 2023-01-05 12:04:49 +01:00
Outdated
Review
Copy Link

(côté BO ça crée une boucle infinie, pas 100% sûr à cause de quoi)

Je n'arrive pas à reproduire, c'est du côté de combo sur la cellule graph ?

> (côté BO ça crée une boucle infinie, pas 100% sûr à cause de quoi) Je n'arrive pas à reproduire, c'est du côté de combo sur la cellule graph ?
vdeniaud commented 2023-01-05 12:12:31 +01:00
Outdated
Review
Copy Link

Modif appliquée

Modif appliquée
vdeniaud commented 2023-01-05 12:13:40 +01:00
Outdated
Review
Copy Link

C'est côté paramétrage de la cellule, sélectionner n'importe quelle donnée qui ne soit pas bijoe (genre « Démarches : Nombre de demandes »).

C'est côté paramétrage de la cellule, sélectionner n'importe quelle donnée qui ne soit pas bijoe (genre « Démarches : Nombre de demandes »).
form.change(function() {
tjund commented 2023-01-04 17:41:04 +01:00
Outdated
Review
Copy Link

Je pense que c'est mieux d'utiliser l'event submit sur le form plutôt que de simuler un click sur le bouton :

$('form#chart-filters').submit();

Je pense que c'est mieux d'utiliser l'event submit sur le form plutôt que de simuler un click sur le bouton : `$('form#chart-filters').submit();`
tjund commented 2023-01-04 17:44:04 +01:00
Outdated
Review
Copy Link

je pense qu'il est même possible de récupérer le form d'un champs sans faire intervenir de dom :

this.form.submit()

je pense qu'il est même possible de récupérer le form d'un champs sans faire intervenir de dom : `this.form.submit()`
vdeniaud commented 2023-01-05 10:22:17 +01:00
Outdated
Review
Copy Link

Changé en faveur de submit(), par contre this.form.submit() (ou this.submit()) ne fonctionne pas, car ça soumet le formulaire alors qu'on ne veut pas vraiment (il y a un e.preventDefault() dans le handler).

Changé en faveur de submit(), par contre this.form.submit() (ou this.submit()) ne fonctionne pas, car ça soumet le formulaire alors qu'on ne veut pas vraiment (il y a un e.preventDefault() dans le handler).
if(loaded_cell_count == 0) {
form.submit();
}
});
});
</script>
{% endblock %}

5
combo/apps/dataviz/templates/combo/chartngcell_form.html
View File

@ -29,5 +29,10 @@
end_field_template.parent().hide();
}
}).change();
$('select, input', 'div#panel-dataviz_chartngcell-{{ cell.pk }}-general').change(function() {
$('div#cell-dataviz_chartngcell-{{ cell.pk }} button.save').click();
});
});
</script>