[spip-dev] r19693 - in branches/spip-2.1/ecrire: inc public

* esj@rezo.net tapuscrivait, le 11/07/2012 11:33:

Author: esj@rezo.net
Date: 2012-07-11 11:33:23 +0200 (mer, 11 jui 2012)
New Revision: 19693

Log:
r18982 n'avait corrigé le bug sur PARAMETRES_FORUM qu'incomplètement, en ne tenant pas compte de la boucle nommée "0".

Par ailleurs, afin que le raccourci pour les tables puisse produire du XHTML Basic, introduction d'une constante _RACCOURCI_THEAD qui ne produit les verbeux Thead et Tbody que si elle vaut True. Du coup, ce raccourci devient capable de produire des tables ayant plusieurs lignes de Th (typiquement une en haut et une en bas), ce qui est nettement plus utile que de forcer les verbosités sus-nommées.

Modified:
    branches/spip-2.1/ecrire/inc/texte.php
    branches/spip-2.1/ecrire/public/balises.php

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

Maleureusement, se commit introduit un effet de bord indésirable constaté sur SoyezCréateurs.
J'ai vérifié en revenant au code d'avant cette modification.

[(#PARAMETRES_FORUM|url_reponse_forum)] dans une boucle ARTICLES renvoie avec ce commit id_rubrique de l'article en plus de id_article
Ce qui fait que le retour du forum se fait sur la rubrique et non sur l'article.
[(#PARAMETRES_FORUM{#SELF}|url_reponse_forum)] permet de contourner le bug, mais ce n'est pas propre.

* RealET tapuscrivait, le 17/08/2012 17:45:

* esj@rezo.net tapuscrivait, le 11/07/2012 11:33:

Author: esj@rezo.net
Date: 2012-07-11 11:33:23 +0200 (mer, 11 jui 2012)
New Revision: 19693

Log:
r18982 n'avait corrigé le bug sur PARAMETRES_FORUM qu'incomplètement,
en ne tenant pas compte de la boucle nommée "0".

Par ailleurs, afin que le raccourci pour les tables puisse produire du
XHTML Basic, introduction d'une constante _RACCOURCI_THEAD qui ne
produit les verbeux Thead et Tbody que si elle vaut True. Du coup, ce
raccourci devient capable de produire des tables ayant plusieurs
lignes de Th (typiquement une en haut et une en bas), ce qui est
nettement plus utile que de forcer les verbosités sus-nommées.

Modified:
    branches/spip-2.1/ecrire/inc/texte.php
    branches/spip-2.1/ecrire/public/balises.php

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

Maleureusement, se commit introduit un effet de bord indésirable
constaté sur SoyezCréateurs.
J'ai vérifié en revenant au code d'avant cette modification.

[(#PARAMETRES_FORUM|url_reponse_forum)] dans une boucle ARTICLES renvoie
avec ce commit id_rubrique de l'article en plus de id_article
Ce qui fait que le retour du forum se fait sur la rubrique et non sur
l'article.
[(#PARAMETRES_FORUM{#SELF}|url_reponse_forum)] permet de contourner le
bug, mais ce n'est pas propre.

Et le contournement provisoire :

Oups, manquait un S dans un nom de variable. Corrigé par r19823.

Committo,Ergo:Sum

* Committo,Ergo:sum tapuscrivait, le 19/08/2012 17:54:

* esj@rezo.net tapuscrivait, le 11/07/2012 11:33:

Author: esj@rezo.net
Date: 2012-07-11 11:33:23 +0200 (mer, 11 jui 2012)
New Revision: 19693

Log:
r18982 n'avait corrigé le bug sur PARAMETRES_FORUM qu'incomplètement, en ne tenant pas compte de la boucle nommée "0".

Maleureusement, se commit introduit un effet de bord indésirable constaté sur SoyezCréateurs.

Oups, manquait un S dans un nom de variable. Corrigé par r19823.

Merci !
Là où ça va être plus embêtant pour certains, c'est que l'URL calculée contient id_rubrique pour les forums postés durant la présence de ce bug.

Pourquoi ?
Parce que la table spip_forum contient dans la colonne id_rubrique le n° de la rubrique de l'article (au lieu de 0 normalement).

UPDATE `spip_forum` SET `id_rubrique`=0 WHERE `id_article`>0 AND `id_rubrique`>0 permet de réparer.

(PS : je renvois ce message qui n'est pas passé hier sur gmane).