r12385 - in spip: ecrire/action ecrire/inc prive prive/rss

Author: esj@rezo.net
Date: 2008-08-24 23:42:57 +0200 (dim, 24 aoû 2008)
New Revision: 12385

Log:
Passage en squelettes des flux RSS de l'espace privé construits par {{{action/rss.php}}}.

Les squelettes se trouvent dans un repértoire spécifique {{{prive/rss}}} et ont actuellement le délai de cache par défaut, c'est sans doute à revoir, je ne mets pas encore sur la branche 2.

Les flux RSS qui travaillent sur plusieurs boucles ont imposé d'interpoler du PHP dans le squelette pour le tri final. Afin de ne pas alourdir celui-ci encore plus, {{{texte_backend}}} transcode a présent l'apostrophe en {{{'}}} ce qui dispense de truffer de {{{addslashes}}} le dit squelette.

Les URLS délivrant ces flux restent les mêmes que précédemment. Il serait bon d'évacuer du répertoire {{{action}} le script {{{rss.php}}} qui n'est pas une action sur la base de données, mais il faudrait alors assurer le suivi des URL, par htaccess ou autres. Tant qu'à faire, il faudrait en profiter pour évacuer les cas particulier où le paramètre {{{op}}} n'est pas égal au nom du squelette construisant le flux.

Added:
   spip/prive/rss/
   spip/prive/rss/a_suivre.html
   spip/prive/rss/forum_article.html
   spip/prive/rss/forum_breve.html
   spip/prive/rss/forum_rubrique.html
   spip/prive/rss/forum_syndic.html
   spip/prive/rss/forum_thread.html
   spip/prive/rss/forums.html
   spip/prive/rss/messagerie.html
   spip/prive/rss/revisions.html
   spip/prive/rss/signatures.html
Modified:
   spip/ecrire/action/rss.php
   spip/ecrire/inc/filtres.php
   spip/ecrire/inc/suivi_versions.php

Details: http://trac.rezo.net/trac/spip/changeset/12385

Diras-tu encore après ça que tu ne vois pas l'utilité des boucles (POUR) permettant de boucler sur un tableau et que tu n'en a jamais eu le besoin ?

Cédric

Le 24 août 08 à 23:43, esj@rezo.net a écrit :

Author: esj@rezo.net
Date: 2008-08-24 23:42:57 +0200 (dim, 24 aoû 2008)
New Revision: 12385

Log:
Passage en squelettes des flux RSS de l'espace privé construits par {{{action/rss.php}}}.

Les squelettes se trouvent dans un repértoire spécifique {{{prive/rss}}} et ont actuellement le délai de cache par défaut, c'est sans doute à revoir, je ne mets pas encore sur la branche 2.

Les flux RSS qui travaillent sur plusieurs boucles ont imposé d'interpoler du PHP dans le squelette pour le tri final. Afin de ne pas alourdir celui-ci encore plus, {{{texte_backend}}} transcode a présent l'apostrophe en {{{'}}} ce qui dispense de truffer de {{{addslashes}}} le dit squelette.

Les URLS délivrant ces flux restent les mêmes que précédemment. Il serait bon d'évacuer du répertoire {{{action}} le script {{{rss.php}}} qui n'est pas une action sur la base de données, mais il faudrait alors assurer le suivi des URL, par htaccess ou autres. Tant qu'à faire, il faudrait en profiter pour évacuer les cas particulier où le paramètre {{{op}}} n'est pas égal au nom du squelette construisant le flux.

Added:
   spip/prive/rss/
   spip/prive/rss/a_suivre.html
   spip/prive/rss/forum_article.html
   spip/prive/rss/forum_breve.html
   spip/prive/rss/forum_rubrique.html
   spip/prive/rss/forum_syndic.html
   spip/prive/rss/forum_thread.html
   spip/prive/rss/forums.html
   spip/prive/rss/messagerie.html
   spip/prive/rss/revisions.html
   spip/prive/rss/signatures.html
Modified:
   spip/ecrire/action/rss.php
   spip/ecrire/inc/filtres.php
   spip/ecrire/inc/suivi_versions.php

Details: http://trac.rezo.net/trac/spip/changeset/12385

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

Le 25 août 08 à 11:20, cedric.morin@yterium.com a écrit :

Diras-tu encore après ça que tu ne vois pas l'utilité des boucles (POUR) permettant de boucler sur un tableau et que tu n'en a jamais eu le besoin ?

Bah oui parce que je ne vois pas en quoi cela m'aurait aidé ici.

Emmanuel

A écrire la même chose sans php, donc avec le bénéfice du cache :stuck_out_tongue:

Le 25 août 08 à 11:23, Committo,Ergo:sum a écrit :

Le 25 août 08 à 11:20, cedric.morin@yterium.com a écrit :

Diras-tu encore après ça que tu ne vois pas l'utilité des boucles (POUR) permettant de boucler sur un tableau et que tu n'en a jamais eu le besoin ?

Bah oui parce que je ne vois pas en quoi cela m'aurait aidé ici.

Emmanuel

Le 25 août 08 à 11:28, cedric.morin@yterium.com a écrit :

A écrire la même chose sans php, donc avec le bénéfice du cache :stuck_out_tongue:

Si tu arrives à réécrire ces squelettes sans PHP je serais convaincu, mais je suis sceptique. Le pb est de faire des Select dans plusieurs tables puis d'interclasser les résultats. Ta boucle sur un tableau exige que ce tableau soit préalablement rempli, comment vas-tu faire sinon par du PHP ? Et qu'est-ce qui va etre plus simple que la structure actuelle qui n'est pas très différente d'un squelette sans PHP. S'il n'y avait pas cet interclassement, on aurait:

<BOUCLE1(T1)>R1</BOUCLE1>
<BOUCLE2(T2)>R2</BOUCLE2>

Là on a:

< ?php
<BOUCLE1(T1)>$r[#DATE] .= 'R1'</BOUCLE1>
<BOUCLE2(T2)>$r[#DATE] .= 'R2'</BOUCLE2>
rsort($r);
echo join('',$r);
? >

Tu peux vraiment faire plus simple ?

En revanche, j'ai oublié des texte_backend à certains endroits, Fil a raison sur les pbs de sécu que ça pose, je vais blinder.

Emmanuel