Produits Entr'ouvert » Combo

J'ai regardé un peu ce ticket, as expected c'est pas évident.

• sélection d'un formulaire
• wcs renvoie les données + les sous filtres liés au formulaire
• on enregistre ces sous filtres
• on les affiche sur le formulaire backoffice de la cellule

• sélection d'un formulaire sur la cellule filtre
• le clic sur rafraîchir déclenche le rechargement de toutes les cellules, normal
• chaque cellule récupère les données (normal) + les sous filtres qu'elle met en cache (parce qu'on ne peut pas juste faire un save() comme dans le scénario backoffice)
  • cette mise en cache nécessite un ID unique généré au chargement de la page
• la cellule filtre détecte que ça y est tout le monde s'est rafraîchi, puis se rafraîchi elle-même en allant lire le cache des sous filtres de chaque cellule et en en déduisant les sous filtres à afficher

Côté style JS, } else if(url.includes('?')) { je serais assez pour l'esapce après le if. (il y en a deux autres).

dataviz-uuid, pas formellement un uuid, pour éviter la confusion peut-être appeler ça différemment ?

combo/public/static/js/combo.public.js

  } else if (url.includes('?')) {
    qs = '&';

C'est bizarre, window.location.search c'est la querystring (et son point d'interrogation), donc si ça existe c'est le if au-dessus, et si elle n'existe pas (et on tape un & derrière) et sinon c'est le else dessous (et on part d'une chaine avec juste "?").

Ce qu'on se trouve ici avoir c'est un data-ajax-cell-url qui contiendrait un point d'interrogation, ce qui arrive via

      new_url = chart_cell.data('ajax-cell-url')  + '?filters_cell_id=' + $('body').data('filters-cell-id') + '&' + $(this).serialize()

(un espace en trop devant le premier +).

mais ça vient donc mal se combiner en présence d'un window.location.search, où on va se trouver avec ...?filters_cell_id=XXX&plop=plop + le '?la-query-string&'.

Il me semble qu'avec l'introduction de la possibilité de query string dans data-ajax-cell-url il faut gérer un cas supplémentaire, dans mon faux js :

  if (window.location.search and '?' dans l'url) {
    qs = '&' + window.location.search[1:] + '&';
  } else if (window.location.search and not '?' dans l'url) {
    qs = window.location.search + '&';
  } elif ('?' dans l'url)
    qs = '&';
  } else {
    qs = '?';
  }

(par exemple)

Automatic expiration