r101519 - in _squelettes_/aveline/trunk

ben · Janvier 9, 2017, 10:03

Author: ben.spip@gmail.com
Date: 2017-01-05 14:04:56 +0100 (jeu. 05 janv. 2017)
New Revision: 101519

Modified:
   _squelettes_/aveline/trunk/inclure/article-resume.html
   _squelettes_/aveline/trunk/inclure/auteur_articles_note.html
   _squelettes_/aveline/trunk/inclure/breve-resume.html
   _squelettes_/aveline/trunk/inclure/evenement-resume.html
   _squelettes_/aveline/trunk/inclure/forum.html
   _squelettes_/aveline/trunk/inclure/mots_cles.html
   _squelettes_/aveline/trunk/inclure/rubrique-resume.html
   _squelettes_/aveline/trunk/inclure/site-resume.html
   _squelettes_/aveline/trunk/inclure/syndic_article-resume.html
   _squelettes_/aveline/trunk/noisettes/anythingslider_liste_articles.html
   _squelettes_/aveline/trunk/noisettes/anythingslider_liste_evenements.html
   _squelettes_/aveline/trunk/noisettes/anythingslider_liste_syndic_articles.html
   _squelettes_/aveline/trunk/noisettes/auteur-articles.html
   _squelettes_/aveline/trunk/noisettes/liste_articles.html
   _squelettes_/aveline/trunk/noisettes/liste_auteurs.html
   _squelettes_/aveline/trunk/noisettes/liste_breves.html
   _squelettes_/aveline/trunk/noisettes/liste_evenements.html
   _squelettes_/aveline/trunk/noisettes/liste_forums.html
   _squelettes_/aveline/trunk/noisettes/liste_syndic_articles.html
   _squelettes_/aveline/trunk/noisettes/mot-articles.html
   _squelettes_/aveline/trunk/noisettes/mot-breves.html
   _squelettes_/aveline/trunk/noisettes/mot-evenements.html
   _squelettes_/aveline/trunk/noisettes/mot-forums.html
   _squelettes_/aveline/trunk/noisettes/page-recherche-articles.html
   _squelettes_/aveline/trunk/noisettes/page-recherche-breves.html
   _squelettes_/aveline/trunk/noisettes/page-recherche-evenements.html
   _squelettes_/aveline/trunk/noisettes/page-recherche-forums.html
   _squelettes_/aveline/trunk/noisettes/page-recherche-syndic_articles.html
   _squelettes_/aveline/trunk/noisettes/rubriques_secteur_langue.html
   _squelettes_/aveline/trunk/noisettes/site-syndic_articles.html
Log:
Il ne faut utiliser #INCLURE que quand on a besoin de conditionner un affichage ou de filtrer le résultat en appliquant un filtre dessus. Sinon il faut toujours utiliser <INCLURE> car #INCLURE perd toutes les balises dynamiques (php inline, formulaires etc…). 2/n

Details: http://core.spip.org/projects/spip/repository/revisions/101519

larmarange · Janvier 11, 2017, 10:32

Coucou Ben,

la modif a été un peu radicale. Du coup tu as remplacé pas mal de balises #INCLURE appelées dans la partie conditionnelle d’autres balises et ça casse pas mal de noisettes.

Par ailleurs, dans un certain nombre d’autres cas, le recours à #INCLURE plutôt que avait été motivé par le fait de réduire un peu le cache vu que noizetier/aveline enchaîne déjà les inclusions en cascade. Or dans le cas où il n’y a justement pas de balises dynamiques, n’est pas forcément nécessaire.

Maintenant, s’il y a par endroit une pertinence à utiliser au lieu de #INCLURE, je n’y vois pas d’inconvénient, à condition de tester le changement au préalable et de vérifier que cela ne casse rien.

Je me permets donc, dans l’immédiat et sans animosité, d’annuler les trois derniers commits afin de rétablir le fonctionnement des sites (car actuellement on on a message d’erreur pour toute liste d’objets) qui utilise Aveline.

Si tu as identifié des noisettes particulières qui nécessite un changement (aucun squelette n’est infaillible), tu es libre de procéder aux corrections pertinentes.

Bien amicalement

Le lun. 9 janv. 2017 à 11:08, <ben.spip@gmail.com> a écrit :

Author: ben.spip@gmail.com
Date: 2017-01-05 14:04:56 +0100 (jeu. 05 janv. 2017)
New Revision: 101519

Modified:
squelettes/aveline/trunk/inclure/article-resume.html
squelettes/aveline/trunk/inclure/auteur_articles_note.html
squelettes/aveline/trunk/inclure/breve-resume.html
squelettes/aveline/trunk/inclure/evenement-resume.html
squelettes/aveline/trunk/inclure/forum.html
squelettes/aveline/trunk/inclure/mots_cles.html
squelettes/aveline/trunk/inclure/rubrique-resume.html
squelettes/aveline/trunk/inclure/site-resume.html
squelettes/aveline/trunk/inclure/syndic_article-resume.html
squelettes/aveline/trunk/noisettes/anythingslider_liste_articles.html
squelettes/aveline/trunk/noisettes/anythingslider_liste_evenements.html
squelettes/aveline/trunk/noisettes/anythingslider_liste_syndic_articles.html
squelettes/aveline/trunk/noisettes/auteur-articles.html
squelettes/aveline/trunk/noisettes/liste_articles.html
squelettes/aveline/trunk/noisettes/liste_auteurs.html
squelettes/aveline/trunk/noisettes/liste_breves.html
squelettes/aveline/trunk/noisettes/liste_evenements.html
squelettes/aveline/trunk/noisettes/liste_forums.html
squelettes/aveline/trunk/noisettes/liste_syndic_articles.html
squelettes/aveline/trunk/noisettes/mot-articles.html
squelettes/aveline/trunk/noisettes/mot-breves.html
squelettes/aveline/trunk/noisettes/mot-evenements.html
squelettes/aveline/trunk/noisettes/mot-forums.html
squelettes/aveline/trunk/noisettes/page-recherche-articles.html
squelettes/aveline/trunk/noisettes/page-recherche-breves.html
squelettes/aveline/trunk/noisettes/page-recherche-evenements.html
squelettes/aveline/trunk/noisettes/page-recherche-forums.html
squelettes/aveline/trunk/noisettes/page-recherche-syndic_articles.html
squelettes/aveline/trunk/noisettes/rubriques_secteur_langue.html
squelettes/aveline/trunk/noisettes/site-syndic_articles.html
Log:
Il ne faut utiliser #INCLURE que quand on a besoin de conditionner un affichage ou de filtrer le résultat en appliquant un filtre dessus. Sinon il faut toujours utiliser car #INCLURE perd toutes les balises dynamiques (php inline, formulaires etc…). 2/n

Details: http://core.spip.org/projects/spip/repository/revisions/101519

spip-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-commit
dev: http://trac.rezo.net/trac/spip/

–

Joseph LARMARANGE

Demographer PhD at IRD • Ceped (UMR 196 Paris Descartes - IRD)

Tel. (Ceped): +33 1 76 53 34 75

Cell. phone (France) : +33 6 62 06 51 82

Cell. Phone (Côte d’Ivoire): +225 89 40 59 33

Skype: joseph.larmarange
Ceped, 45 rue des Saints-Pères, 75006 Paris, France

cerdic · Janvier 11, 2017, 10:43

Salut Joseph

Joseph a écrit :

Par ailleurs, dans un certain nombre d'autres cas, le recours à #INCLURE
plutôt que <INCLURE> avait été motivé par le fait de réduire un peu le
cache vu que noizetier/aveline enchaîne déjà les inclusions en cascade.

C'est une MAUVAISE pratique. Ça ne réduit rien, au contraire ça augmente le volume de cache puisque le meme résultat de calcul est stocké 2 fois : une fois dans son propre cache et une fois dans le cache de l'appelant (et une fois de plus par niveau d'#INCLURE imbriqué).

De plus tu ne peux pas présager de ce que les utilisateurs vont éventuellement mettre dans des surcharges de noisette.

Il faut donc impérativement mettre <INCLURE> partout où c'est possible, c'est à dire *partout* sauf en cas d'affichage conditionnel
[ avant(#INCLURE)apres]
ou de filtre
[(#INCLURE|filtrer]

Il y a peut-être eu un peu de casse lors du remplacement systématique, à corriger, mais en l'état le squelette pose plein de problèmes de cache qui obligent les utilisateurs à mettre du #CACHE{0} ce qui est catastrophique.

Je propose donc que tu fasses un remontée d'erreur que l'on corrige, plutôt qu'on s'amuse à revert et rerevert ad libitum sans corriger aucun des problèmes.

--
Cédric

ben · Janvier 11, 2017, 10:53

Hello Joseph, pour compléter ce que dit cedric : j’ai d’abord pensé modifier dans mon coin, puis on s’est dit que non autant que ça profite à tout le monde .

Le problème initial posé était en effet de palier au fait de mettre un #CACHE{0} pour résoudre un problème de rafraîchissement de noisette .

J’ai bien fait attention sur cette série de modif laborieuse d’essayer de ne rien casser mais j’ai ptet merdé sur une/quelques noisettes.
Si tu as des exemples qui ne marchent pas on regarde, je corrige et je répercute sur les autres.
En tout cas depuis la modif, le site sur lequel j’ai appliqué n’a pas d’erreur et je viens de vérifier sur le forum de contrib https://contrib.spip.net/Aveline-v2 pas de remontée non plus.

Ben.

ben · Janvier 11, 2017, 11:31

Hello Comme dit dans le mail précédent : ce n’est pas une expression régulière mais mes yeux ( et avec l’âge l’erreur est possible )

je suis preneur d’un exemple précis si tu as un site de test ou alors me transférer un site de test et je le monte.

Ben.

larmarange · Janvier 11, 2017, 2:21

Autant pour moi et un grand merci Cédric pour les précisions techniques.

Serait-il possible de préciser, en SPIP 3.0, la situation syntaxique suivante [(#ENV{toto}|oui) #INCLURE ] ?

Il me semble que c’est également un cas de figure où le recours à #INCLURE est nécessaire. Ou bien ai-je tout faux ? Ce cas de figure est très fréquent dans Aveline (cf. https://zone.spip.org/trac/spip-zone/changeset/101519/squelettes/aveline)

Cdlt

Joseph

cerdic · Janvier 11, 2017, 2:34

Joseph a écrit :

Autant pour moi et un grand merci Cédric pour les précisions techniques.

Serait-il possible de préciser, en SPIP 3.0, la situation syntaxique
suivante `[(#ENV{toto}|oui) #INCLURE ]` ?

Il me semble que c'est également un cas de figure où le recours à
#INCLURE est nécessaire.

Pas du tout non : sur cette écriture la condition repose sur #ENV{toto} et le résultat de l'inclusion n'a pas besoin d'être connu. On peut(doit) donc tout à fait mettre
[(#ENV{toto}|oui) <INCLURE> ]
dans ce cas

Cédric

larmarange · Janvier 11, 2017, 2:38

OK merci pour la précision.
Ben, j’essaie de revert le revert et d’identifier les cas problématiques, comme par exemple :

[(#ENV{utiliser_logo_article_rubrique}|non)[(#LOGO_ARTICLE||image_reduire{[(#ENV{taille_max_logo}|sinon{150})]}|sinon{
#INCLURE{fond=inclure/article-resume-image}{env}})]]

Cdlt

Le mer. 11 janv. 2017 à 15:34, Cédric Morin <cedric@yterium.com> a écrit :

Joseph a écrit :

Autant pour moi et un grand merci Cédric pour les précisions techniques.

Serait-il possible de préciser, en SPIP 3.0, la situation syntaxique
suivante [(#ENV{toto}|oui) #INCLURE ] ?

Il me semble que c’est également un cas de figure où le recours à
#INCLURE est nécessaire.

Pas du tout non : sur cette écriture la condition repose sur #ENV{toto}
et le résultat de l’inclusion n’a pas besoin d’être connu. On peut(doit)
donc tout à fait mettre
[(#ENV{toto}|oui) ]
dans ce cas

Cédric

—

–

Joseph LARMARANGE

Demographer PhD at IRD • Ceped (UMR 196 Paris Descartes - IRD)

Tel. (Ceped): +33 1 76 53 34 75

Cell. phone (France) : +33 6 62 06 51 82

Cell. Phone (Côte d’Ivoire): +225 89 40 59 33

Skype: joseph.larmarange
Ceped, 45 rue des Saints-Pères, 75006 Paris, France

larmarange · Janvier 11, 2017, 3:06

Ok

je viens de ré-appliquer les commits de Ben et ai revu l’ensemble de ces modifs pour corriger quelques erreurs de parenthésages et quelques cas où #INCLURE reste impératif.

J’ai fait quelques tests sur des sites tournant avec Aveline. Pas rencontré de message d’erreur (au moins pour le moment).

Cdlt

Le mer. 11 janv. 2017 à 15:38, Joseph <joseph@larmarange.net> a écrit :

OK merci pour la précision.
Ben, j’essaie de revert le revert et d’identifier les cas problématiques, comme par exemple :
[(#ENV{utiliser_logo_article_rubrique}|non)[(#LOGO_ARTICLE||image_reduire{[(#ENV{taille_max_logo}|sinon{150})]}|sinon{
#INCLURE{fond=inclure/article-resume-image}{env}})]]
Cdlt

Le mer. 11 janv. 2017 à 15:34, Cédric Morin <cedric@yterium.com> a écrit :

Joseph a écrit :

Autant pour moi et un grand merci Cédric pour les précisions techniques.

Serait-il possible de préciser, en SPIP 3.0, la situation syntaxique
suivante [(#ENV{toto}|oui) #INCLURE ] ?

Il me semble que c’est également un cas de figure où le recours à
#INCLURE est nécessaire.

Pas du tout non : sur cette écriture la condition repose sur #ENV{toto}
et le résultat de l’inclusion n’a pas besoin d’être connu. On peut(doit)
donc tout à fait mettre
[(#ENV{toto}|oui) ]
dans ce cas

Cédric

—

–

Joseph LARMARANGE

Demographer PhD at IRD • Ceped (UMR 196 Paris Descartes - IRD)

Tel. (Ceped): +33 1 76 53 34 75

Cell. phone (France) : +33 6 62 06 51 82

Cell. Phone (Côte d’Ivoire): +225 89 40 59 33

Skype: joseph.larmarange
Ceped, 45 rue des Saints-Pères, 75006 Paris, France

–

Joseph LARMARANGE

Demographer PhD at IRD • Ceped (UMR 196 Paris Descartes - IRD)

Tel. (Ceped): +33 1 76 53 34 75

Cell. phone (France) : +33 6 62 06 51 82

Cell. Phone (Côte d’Ivoire): +225 89 40 59 33

Skype: joseph.larmarange
Ceped, 45 rue des Saints-Pères, 75006 Paris, France

cerdic · Janvier 11, 2017, 3:49

Super, merci beaucoup Joseph, désolé pour le temps perdu !
--
Cédric

Joseph a écrit :

Ok

je viens de ré-appliquer les commits de Ben et ai revu l'ensemble de ces
modifs pour corriger quelques erreurs de parenthésages et quelques cas
où #INCLURE reste impératif.

J'ai fait quelques tests sur des sites tournant avec Aveline. Pas
rencontré de message d'erreur (au moins pour le moment).

Cdlt

Le mer. 11 janv. 2017 à 15:38, Joseph <joseph@larmarange.net
<mailto:joseph@larmarange.net>> a écrit :

    OK merci pour la précision.
    Ben, j’essaie de revert le revert et d’identifier les cas
    problématiques, comme par exemple :

    |[(#ENV{utiliser_logo_article_rubrique}|non)[(#LOGO_ARTICLE||image_reduire{[(#ENV{taille_max_logo}|sinon{150})]}|sinon{
    #INCLURE{fond=inclure/article-resume-image}{env}})]]
    |

    Cdlt

    Le mer. 11 janv. 2017 à 15:34, Cédric Morin <cedric@yterium.com
    <mailto:cedric@yterium.com>> a écrit :

        Joseph a écrit :
         > Autant pour moi et un grand merci Cédric pour les précisions
        techniques.
         >
         > Serait-il possible de préciser, en SPIP 3.0, la situation
        syntaxique
         > suivante `[(#ENV{toto}|oui) #INCLURE ]` ?
         >
         > Il me semble que c'est également un cas de figure où le recours à
         > #INCLURE est nécessaire.

        Pas du tout non : sur cette écriture la condition repose sur
        #ENV{toto}
        et le résultat de l'inclusion n'a pas besoin d'être connu. On
        peut(doit)
        donc tout à fait mettre
        [(#ENV{toto}|oui) <INCLURE> ]
        dans ce cas

        Cédric

    —


    --
    *Joseph LARMARANGE*
    Demographer PhD atIRD <http://www.ird.fr/>•Ceped
    <http://www.ceped.org/> (UMR 196 Paris Descartes - IRD)

    Tel. (Ceped): +33 1 76 53 34 75 <tel:01%2076%2053%2034%2075>
    Cell. phone (France) : +33 6 62 06 51 82 <tel:06%2062%2006%2051%2082>
    Cell. Phone (Côte d'Ivoire): +225 89 40 59 33
    <tel:+225%2089%2040%2059%2033>
    Skype: joseph.larmarange
    Ceped, 45 rue des Saints-Pères, 75006 Paris, France

--
*Joseph LARMARANGE*
Demographer PhD atIRD <http://www.ird.fr/>•Ceped
<http://www.ceped.org/> (UMR 196 Paris Descartes - IRD)

Tel. (Ceped): +33 1 76 53 34 75
Cell. phone (France) : +33 6 62 06 51 82
Cell. Phone (Côte d'Ivoire): +225 89 40 59 33
Skype: joseph.larmarange
Ceped, 45 rue des Saints-Pères, 75006 Paris, France