Hello,
Est-ce qu’il ne manque pas un test d’existence de la constante avant de faire le define à la L424 ?
Tant qu’on y est, est-ce qu’il n’y aura pas moyen de rendre ce JS plus facile à maintenir/lire avec une combo #PRODUIRE/#INCLURE (et puis un petit coup de |compacte éventuellement) ? On aurai alors un squelette SPIP, ce qui me semble déjà beaucoup mieux que de faire cracher du JS à PHP.
Merci !
Le 13.12.16 à 06:38, Phenix a écrit :
Hello,
Est-ce qu’il ne manque pas un test d’existence de la constante avant de
faire le |define| à la L424 ?
je ne vois pas ce que tu veux dire 
Tant qu’on y est, est-ce qu’il n’y aura pas moyen de rendre ce JS plus
facile à maintenir/lire avec une combo #PRODUIRE/#INCLURE (et puis un
petit coup de |compacte éventuellement) ? On aurai alors un squelette
SPIP, ce qui me semble déjà beaucoup mieux que de faire cracher du JS à PHP.Merci !
sans doute, mais hier soir j'avais autre chose à faire 
--
Maïeul
Le 13/12/2016 à 06:38, Phenix a écrit :
Tant qu’on y est, est-ce qu’il n’y aura pas moyen de rendre ce JS plus
facile à maintenir/lire avec une combo #PRODUIRE/#INCLURE (et puis un
petit coup de |compacte éventuellement) ? On aurai alors un squelette
SPIP, ce qui me semble déjà beaucoup mieux que de faire cracher du JS à PHP.
Je trouverais ça presque aussi pire.
À mon avis il faudrait surtout reconvevoir le code de cette fonctionnalité pour que le maximum de choses soit dans du javascript parfaitement générique et statique.
Par exemple en mettant les conditions dans des data-truc="condition", et que le JS générique se débrouille ensuite avec ça.
(C'est une piste possible.)
--
RastaPopoulos