[spip-dev] Erreur dans la validation de SPIP

Dev
1

Bonjour,

SPIP avec son analyse XML trouve une erreur à la 3e ligne de ceci :
001
002 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
003 <html xmlns="http://www.w3.org/1999/xhtml" lang="fr" xml:lang="fr" dir="ltr">

Est-ce que c'est la ligne vide au début ?

2

* Jacques PYRAT tapotait, le 24/02/2007 18:43:

Bonjour,

SPIP avec son analyse XML trouve une erreur à la 3e ligne de ceci :
001
002 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd&quot;&gt;
003 <html xmlns="http://www.w3.org/1999/xhtml&quot; lang="fr" xml:lang="fr" dir="ltr">

Est-ce que c'est la ligne vide au début ?

C'était bien ça.
Le validateur du W3C n'en tient pas compte, celui de SPIP si.

3

Pour ma part, je ne reproduis pas ce comportement.
La regexp appliquée en debut de texte est la suivante, dont il est clair qu'elle accepte beaucoup de choses dont un saut de ligne en début de document:

define('_REGEXP_DOCTYPE',
  '/^((<[?][^>]*>)*\s*(<!--.*?-->)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>]*)>\s*/');

Il vaut toujours mieux lire le code incriminé plutot que de répondre à ses propres questions en faisant foi à un outil opaque qui a déjà fait la preuve de ses limitations.

Committo,Ergo:Sum

4

* Committo,Ergo:sum tapotait, le 24/02/2007 19:14:

define('_REGEXP_DOCTYPE',
  '/^((<[?][^>]*>)*\s*(<!--.*?-->)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>] *)>\s*/');

Il vaut toujours mieux lire le code incriminé plutot que de répondre à ses propres questions en faisant foi à un outil opaque qui a déjà fait la preuve de ses limitations.

Après lecture, mon rapport de bug était insuffisant : ce code ne supporte pas les espace avant le doctype.
Le W3C si.

5

* Committo,Ergo:sum tapotait, le 24/02/2007 19:14:

define('_REGEXP_DOCTYPE',
  '/^((<[?][^>]*>)*\s*(<!--.*?-->)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>]
*)>\s*/');

Il vaut toujours mieux lire le code incriminé plutot que de répondre
à ses propres questions en faisant foi à un outil opaque qui a déjà
fait la preuve de ses limitations.

Après lecture,

Il faut aussi lire complétement le mail ci-dessus, où figurait la phrase:

Pour ma part, je ne reproduis pas ce comportement.

ce code ne
supporte pas les espace avant le doctype.
Le W3C si.

Cours de RegExp: \s* veut dire un nombre quelconque d'espaces, il est précédé et suivi d'une autre étoile,le tout encore mis en étoile, et une regexp doit toujours chercher le plus long segment initial. Donc d'une part ça doit marcher, et d'autre part je confirme que ça marche effectivement sur ma version de PHP, savoir 4.4.4.

mon rapport de bug était insuffisant

Il reste toujours insuffisant en ce qu'il ne dit pas sur quelle version de PHP ça ne marche pas, puisqu'il a été précisé qu'il en existe au moins une où ça marche.

Committo,Ergo:Sum

6

* Committo,Ergo:sum tapotait, le 26/02/2007 14:32:

* Committo,Ergo:sum tapotait, le 24/02/2007 19:14:

define('_REGEXP_DOCTYPE',
  '/^((<[?][^>]*>)*\s*(<!--.*?-->)*)*<!DOCTYPE\s+(\w+)\s+(\w+)\s*([^>]
*)>\s*/');

Il vaut toujours mieux lire le code incriminé plutot que de répondre
à ses propres questions en faisant foi à un outil opaque qui a déjà
fait la preuve de ses limitations.

Après lecture,

Il faut aussi lire complétement le mail ci-dessus, où figurait la phrase:

Pour ma part, je ne reproduis pas ce comportement.

ce code ne
supporte pas les espace avant le doctype.
Le W3C si.

Cours de RegExp: \s* veut dire un nombre quelconque d'espaces, il est précédé et suivi d'une autre étoile,le tout encore mis en étoile, et une regexp doit toujours chercher le plus long segment initial. Donc

Oups, pourquoi est-ce que j'ai cru que \s* signifiait saut de lignes en nombre quelque ?

d'une part ça doit marcher, et d'autre part je confirme que ça marche effectivement sur ma version de PHP, savoir 4.4.4.

En php 5.2.1 ça n'a pas marché chez moi.

mon rapport de bug était insuffisant

Il reste toujours insuffisant en ce qu'il ne dit pas sur quelle version de PHP ça ne marche pas, puisqu'il a été précisé qu'il en existe au moins une où ça marche.

PS : je ne peux plus tester : j'ai mis à jour le squelette pour ne plus avoir le bug.