[SPIP Zone] [Spip-zone-commit] r24835 - in /_plugins_/_dev_/spipBB: branches/spip-1.9/Changelog.txt branches/spip-1.9/corps_sk/c_liste_messages.html branches/spip-1.9/plugin.xml dev/corps_sk/c_liste_messages.html

Archives Zone
1

Salut Chryjs,

Pourquoi avoir supprimé la boucle avec jointure dans le fichier
c_liste_messages.html de la version dev du plugin ? Il me semble que
celle-ci est moins "gourmande" en ressources que celle que tu as
commité pour arriver au même résultat.

++
b_b

2008/12/1 <chryjs@free.fr>:

Author: chryjs@free.fr
Date: Mon Dec 1 15:54:51 2008
New Revision: 24835

Log:
correction sur c_liste_messages

Modified:
   _plugins_/_dev_/spipBB/branches/spip-1.9/Changelog.txt
   _plugins_/_dev_/spipBB/branches/spip-1.9/corps_sk/c_liste_messages.html
   _plugins_/_dev_/spipBB/branches/spip-1.9/plugin.xml
   _plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html

Modified: _plugins_/_dev_/spipBB/branches/spip-1.9/Changelog.txt

--- _plugins_/_dev_/spipBB/branches/spip-1.9/Changelog.txt (original)
+++ _plugins_/_dev_/spipBB/branches/spip-1.9/Changelog.txt Mon Dec 1 15:54:51 2008
@@ -1,5 +1,10 @@
2008-09-21 chryjs <chryjs!@!free!.!fr>

+ * 0.6.1.3 Correction pour c_liste_messages fin de boucle pas applicable en 1.9 cf report : http://spipbb.free.fr/spip.php?page=voirsujet&id_forum=860
+
+
+2008-09-21 chryjs <chryjs!@!free!.!fr>
+
* 0.6.1.2 Parametres pour le formulaire de tri

2008-09-19 chryjs <chryjs!@!free!.!fr>

Modified: _plugins_/_dev_/spipBB/branches/spip-1.9/corps_sk/c_liste_messages.html

--- _plugins_/_dev_/spipBB/branches/spip-1.9/corps_sk/c_liste_messages.html (original)
+++ _plugins_/_dev_/spipBB/branches/spip-1.9/corps_sk/c_liste_messages.html Mon Dec 1 15:54:51 2008
@@ -26,7 +26,7 @@
               </tr>
               [(#REM) un peu lourd mais le critère {plat} dÃ(c)conne avec {id_secteur}]
               <BOUCLE_s(ARTICLES){id_secteur=#ID_RUBRIQUE}>
- <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article} />
+ <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article}></BOUCLE_a>
               </BOUCLE_s>

               <BOUCLE_liste_mess(FORUMS){id_auteur}{!doublons messages}{plat}{!par date}{pagination #CONFIG{spipbb/fixlimit}}>

Modified: _plugins_/_dev_/spipBB/branches/spip-1.9/plugin.xml

--- _plugins_/_dev_/spipBB/branches/spip-1.9/plugin.xml (original)
+++ _plugins_/_dev_/spipBB/branches/spip-1.9/plugin.xml Mon Dec 1 15:54:51 2008
@@ -1,5 +1,5 @@
<plugin>
- <version>0.6.1.2</version>
+ <version>0.6.1.3</version>
       <nom>
               <multi>
                       [en]SpipBB: Management of SPIP 1.9 forums

Modified: _plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html

--- _plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html (original)
+++ _plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html Mon Dec 1 15:54:51 2008
@@ -24,8 +24,12 @@
               <tr>
                       <th colspan="2"><:spipbb:message_s:></th>
               </tr>
+ [(#REM) c: 1/12/8 report b_b : un peu lourd mais le critère {plat} déconne avec {id_secteur}]
+ <BOUCLE_s(ARTICLES){id_secteur=#ID_RUBRIQUE}>
+ <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article} />
+ </BOUCLE_s>

- <BOUCLE_liste_mess(FORUMS spip_articles){id_auteur}{id_secteur=#ID_RUBRIQUE}{plat}{!par date}{pagination #CONFIG{spipbb/fixlimit}}>
+ <BOUCLE_liste_mess(FORUMS){id_auteur}{!doublons messages}{plat}{!par date}{pagination #CONFIG{spipbb/fixlimit}}>

               <tr class="inter entete_sujet">
                       <td id="forum#ID_FORUM" colspan="2">

_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit

2

Salut Bruno !!!

bruno bergot a écrit :

Salut Chryjs,

Pourquoi avoir supprimé la boucle avec jointure dans le fichier
c_liste_messages.html de la version dev du plugin ? Il me semble que
celle-ci est moins "gourmande" en ressources que celle que tu as
commité pour arriver au même résultat.

++
b_b

D'abord, comme je n'en ai pas eu l'occasion, je tiens à en profiter pour te remercier pour tes ajouts/corrections/etc sur ce plugin !

Je comprends ton émoi concernant ce commit mais t'invite à une relecture plus attentive. En effet, je ne l'ai _pas_ supprimé !

Bien au contraire je l'ai mis dans la branche appropriée : la branche dev du plugin lui-même. Il se trouve que par mégarde il semble que tu l'avait mise dans la branche dédiée à SPIP 1.9 or cette optimisation est incompatible avec la 1.9 d'où un bug report.

J'ai donc :
- reporté ton commit vers la branche dev (SPIP 2.0). Je ne sais plus pourquoi je n'ai pas fait de move... Peut être par inadvertance.

>> +++ _plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html Mon Dec 1 15:54:51 2008
>> + <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article} />

- repris ton code pour l'adapter à la version 1.9
>> +++ _plugins_/_dev_/spipBB/branches/spip-1.9/corps_sk/c_liste_messages.html Mon Dec 1 15:54:51 2008
>> - <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article} />
>> + <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article}></BOUCLE_a>

Mais j'ai pu faire une erreur malencontreuse !

--chryjs

3

Re Chryjs,

D'abord, comme je n'en ai pas eu l'occasion, je tiens à en profiter pour te
remercier pour tes ajouts/corrections/etc sur ce plugin !

De rien c'est normal :slight_smile:

Je comprends ton émoi concernant ce commit mais t'invite à une relecture
plus attentive. En effet, je ne l'ai _pas_ supprimé !

En fait je parlais de la modification du fichier
_plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html dans ton
commit :

http://trac.rezo.net/trac/spip-zone/changeset/24835#file3

J'avais déjà "optimisé" cette boucle avec une jointure pour la version dev :

<BOUCLE_liste_mess(FORUMS
spip_articles){id_auteur}{id_secteur=#ID_RUBRIQUE}{plat}{!par
date}{pagination #CONFIG{spipbb/fixlimit}}>

Cela évitait d'utiliser les doublons et les deux boucles pour pallier
le bug du critère {plat} avec {id_secteur}.

Voilou et merci pour le boulot que tu fais sur ce plugin.

++

Bien au contraire je l'ai mis dans la branche appropriée : la branche dev du
plugin lui-même. Il se trouve que par mégarde il semble que tu l'avait mise
dans la branche dédiée à SPIP 1.9 or cette optimisation est incompatible
avec la 1.9 d'où un bug report.

J'ai donc :
- reporté ton commit vers la branche dev (SPIP 2.0). Je ne sais plus
pourquoi je n'ai pas fait de move... Peut être par inadvertance.

+++ _plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html Mon Dec 1
15:54:51 2008
+ <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article} />

- repris ton code pour l'adapter à la version 1.9

+++
_plugins_/_dev_/spipBB/branches/spip-1.9/corps_sk/c_liste_messages.html Mon
Dec 1 15:54:51 2008
- <BOUCLE_a(FORUMS){doublons messages}{plat}{id_article} />
+ <BOUCLE_a(FORUMS){doublons
messages}{plat}{id_article}></BOUCLE_a>

Mais j'ai pu faire une erreur malencontreuse !

--chryjs

4

bruno bergot a écrit :

[...]
En fait je parlais de la modification du fichier
_plugins_/_dev_/spipBB/dev/corps_sk/c_liste_messages.html dans ton
commit :

http://trac.rezo.net/trac/spip-zone/changeset/24835#file3

J'avais déjà "optimisé" cette boucle avec une jointure pour la version dev :

Gloupppppppps j'ai mal "transporté" ta modif. Erreur donc. Faut que je revert !

Cela évitait d'utiliser les doublons et les deux boucles pour pallier
le bug du critère {plat} avec {id_secteur}.

Yes !

Voilou et merci pour le boulot que tu fais sur ce plugin.

Oui enfin si je casse pas le travail des autres :stuck_out_tongue: