[spip-dev] [2.1 SVN 16347] Flux de syndication non récupéré

Dev
1

Bonjour,

Je viens de constater que http://www.eglise.catholique.fr/gateway/syndication.php?key=actualite&value=all ne peux pas/plus être syndiqué par SPIP.

Erreur affichée 10 fois à l'inscription du site :

Warning: date() expects parameter 2 to be long, string given in /ecrire/genie/syndic.php on line 174

Pourtant, il est considéré comme valide par
http://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fwww.eglise.catholique.fr%2Fgateway%2Fsyndication.php%3Fkey%3Dactualite%26value%3Dall

Mais en le regardant, il y a des CDATA douteux :
<pubDate><![CDATA[Wed, 06 Oct 2010 10:55:56 +0200]]></pubDate>
<lastBuildDate><![CDATA[Wed, 06 Oct 2010 10:55:56 +0200]]></lastBuildDate>

On peut dire que c'est une erreur du flux, non ?

-- RealET

2

* RealET tapuscrivait, le 06/10/2010 11:11:

Bonjour,

Je viens de constater que
Site portail de l'Église catholique en France
ne peux pas/plus être syndiqué par SPIP.

Erreur affichée 10 fois à l'inscription du site :

Warning: date() expects parameter 2 to be long, string given in
/ecrire/genie/syndic.php on line 174

Pourtant, il est considéré comme valide par
http://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fwww.eglise.catholique.fr%2Fgateway%2Fsyndication.php%3Fkey%3Dactualite%26value%3Dall

Mais en le regardant, il y a des CDATA douteux :
<pubDate><![CDATA[Wed, 06 Oct 2010 10:55:56 +0200]]></pubDate>
<lastBuildDate><![CDATA[Wed, 06 Oct 2010 10:55:56 +0200]]></lastBuildDate>

On peut dire que c'est une erreur du flux, non ?

Ceci dit, la personne qui m'a signalé ça indique qu'une modification de SPIP corrige le problème :
ligne 137 du fichier ecrire/inc/syndic.php
preg_match(',<(pubdate)>([^<]*)<,Uims',$item, $match))
         {
             if (preg_match ("@\@\@\@SPIP_CDATA(.*)\@\@\@@i", $match[2],$la_date_tmp ))
             {
                 $match[2]=$echappe_cdata[$la_date_tmp[1]];
             }
             $la_date = my_strtotime($match[2]);
         }

-- RealET

3

Bin non, c'est valide, les CDATA ici... Pas utile, mais valide.

-Nicolas

4

On peut dire que c'est une erreur du flux, non ?

Hum... Ce bug a déjà été corrigé dans la branche dev il y a 11 mois :
http://core.spip.org/trac/spip/changeset/14753

Comme je l'avais oublié, j'ai tout refait... Du coup patch encore plus
général ; cette fois, dans l'extension "sites" de la branche dev.

-- Fil

5

* Fil tapuscrivait, le 06/10/2010 22:42:

On peut dire que c'est une erreur du flux, non ?

Hum... Ce bug a déjà été corrigé dans la branche dev il y a 11 mois :
http://core.spip.org/trac/spip/changeset/14753

Comme je l'avais oublié, j'ai tout refait... Du coup patch encore plus
général ; cette fois, dans l'extension "sites" de la branche dev.
Connexion · GitLab

Et pour la branche stable 2.1 ?

-- RealET

6

Et pour la branche stable 2.1 ?

Hé bien tu dois pouvoir copier/coller les deux fichiers. Mais je n'ai
pas envie de prendre le risque que ça casse d'autres syndications, et
tant qu'il n'y a pas plus de jeux de tests, je m'abstiendrai de
reporter.

-- Fil