Re: [Spip-zone-commit] r119082 - in _plugins_/saisies/trunk

oui et non :)

Comme d’habitude tout est toujours histoire de compromis…
Et donc les options sont :

 - insérer le JS sur toutes les pages du sites : dispendieux, puisque ça veut dire que ce JS sera intégré sur tous les sites de la terre qui utilisent le plugin saisies, sur toutes les pages du sites, alors qu’il n’est utilisé que sur quelques formulaires de quelques sites

 - insérer le JS sur les pages concernées -> définitivement ce qu’il faut faire
du coup deux méthodes possibles :
    - on gère dans affichage_final pour voir les pages où le JS est requis via un strpos sur affichage_final (ce qui est fait). Ça a certes un petit coup perfo, mais très minime car un strpos sur du texte c’est très rapide. Du coup pour toutes les pages qui n’ont pas besoin de ce JS (99,9%) c’est indolore. Pour les quelques pages qui utilisent ce JS c’est plus couteux, mais ça reste marginal
    - on insère du JS dans la saisie, qui sera inséré avec chaque saisie de ce type donc, et un mécanisme de loader qui gère les cas multiples proprement (aka la première saisie de la page doit charger le JS et uniquement elle, et se charger d’initialiser toutes les saisies concernées).
C’est aussi possible, mais plus complexe à mettre en place et à maintenir. Il y a un bon exemple sur gis, dont j’avais refait le JS sur ce modèle là.
Ici l’avantage est que tout est en cache, pas de travail supplémentaire sur le serveur, mais au prix d’une complexité accrue sur le JS et la maintenance et c’est très facile de le péter…


--
Cédric
Le 16 déc. 2019 à 14:26 +0100, [email protected], a écrit :
> Author: [email protected]
> Date: 2019-12-16 13:26:27 +0000 (Mon, 16 Dec 2019)
> New Revision: 119082
>
> Added:
> _plugins_/saisies/trunk/inclure/generer_saisies_fonctions.php
> _plugins_/saisies/trunk/javascript/afficher_si.js
> Removed:
> _plugins_/saisies/trunk/javascript/afficher_si.js.html
> _plugins_/saisies/trunk/javascript/afficher_si.js_fonctions.php
> Modified:
> _plugins_/saisies/trunk/inclure/generer_saisies.html
> _plugins_/saisies/trunk/paquet.xml
> Log:
> perf : eviter les js en squelette lorsque pas necessaire (note en passant : faudra aussi voir a supprimer ces tests sur affichage_final, c'est bien bouffeur
>
>
> Details: https://zone.spip.net/trac/spip-zone/changeset/119082
>
> _______________________________________________
> [email protected] - https://listes.rezo.net/mailman/listinfo/spip-zone-commit