[spip-dev] r14066 - branches/spip-2.0/ecrire/inc spip/ecrire/inc

Dev
1

* esj@rezo.net tapuscrivait, le 05/06/2009 09:12:

Author: esj@rezo.net
Date: 2009-06-05 09:12:41 +0200 (ven, 05 jun 2009)
New Revision: 14066

Log:
Cas tordu d'un appel de notes. Le test unitaire suivant passe à présent intégralement:

Je suis au regret de dire qu'après recherche, ce dépôt casse l'appel et le retour des notes de bas de page.

Pour être précis :
Un paragraphe dans SPIP est précédé d'une ligne vide et suivi d'une ligne vide[[Sauf les listes à puce et les tableaux]].

Donne dans l'espace privé, dans ecrire/?exec=articles&id_article=46 :

Un paragraphe dans SPIP est précédé d’une ligne vide et suivi d’une ligne vide.
Notes

Sauf les listes à puce et les tableaux

Au lieu de donner (en 14065) :
Un paragraphe dans SPIP est précédé d’une ligne vide et suivi d’une ligne vide [1].

Notes

[1] Sauf les listes à puce et les tableaux

Donc, il manque l'appel [1] dans le texte et le retour [1] devant la note.
Bug vérifié sur un site sans aucun plugin.

2

Pour être précis, alimente le jeu de tests unitaire référencé dans le log:
http://zone.spip.org/trac/spip-zone/changeset/26808/dev/tests/propre/notes.html

Committo,Ergo:Sum

3

* Committo,Ergo:sum tapuscrivait, le 18/06/2009 00:40:

New Revision: 14066
Log:
Cas tordu d'un appel de notes. Le test unitaire suivant passe à présent intégralement:

Je suis au regret de dire qu'après recherche, ce dépôt casse l'appel et le retour des notes de bas de page.

Pour être précis :

Pour être précis, alimente le jeu de tests unitaire référencé dans le log:
Connexion · GitLab

C'est fait : Connexion · GitLab
ça teste a[[b]]
Le test est OK en 14065 et faux en 14066

4

RealET a écrit :

Le test est OK en 14065 et faux en 14066

ha oui.
dans ecrire/inc/notes.php, ligne 49
il faudrait remplacer
  if (!(isset($nom)
par
   if (!($nom

puisque désormais $nom ne peut plus être NULL
mais string(0) a minima

'le' blème, c'est que du coup
on perd l'affichage de la "note appelee explicitement vide"
(test d) mais on ne perd pas son 'appel' [1] :

   bla bla bla[]
   Un paragraphe précédé et suivi d’une ligne vide[[Sauf listes]]
   ...

donne
   bla bla bla[1]
   Un paragraphe précédé et suivi d’une ligne vide[2]
   ...
   [2] Sauf listes

(testé en SPIP 2.1.0 dev SVN [14113])

faut-il commiter quand même ? (m'interroge-je)

5

non il faut rajouter 'AND $ref'
je suis dessus, mais il y a un pb en aval avec code_echappement que je ne comprends pas

Committo,Ergo:Sum

6

Committo,Ergo:sum a écrit :

je suis dessus, mais il y a un pb en aval avec code_echappement que je ne comprends pas

désormais ok avec http://trac.rezo.net/trac/spip/changeset/14114 donc
et aussi pour les tests de propre/notes.html [29239],
y compris le test 'd'

sur SPIP 2.1.0 dev SVN [14113]
et SPIP 2.0.8 SVN [14105]