Re: [Devel] r489 - branches/generic

[Ploc <plub2010 AT acampado.net>]

Salut à tous,

Devant l'absence de correctif sur le trunk au sujet de la validité du flux
rss de l'agenda du libre, j'ai corrigé un problème dans la branche generic.

J'ai utilisé FeedValidator pour valider le flux et détecter d'éventuel
problème :
<http://feedvalidator.org/check.cgi?url=http%3A%2F%2Fwww.agendadulibre.org%2Frss.php%3Fregion%3Dall>.

Le premier problème auquel je me suis attaqué est celui de la date dc:date
qui est techniquement valide mais fonctionnellement incorrecte. En effet,
pour l'agenda du libre, cette date correspond à la date de début
d'évènement. Or, selon les documents suivants, il apparait que cette date
doit fonctionnellement correspondre à la date de publication de l'item dans
le flux rss.

- <http://feedvalidator.org/docs/warning/ImplausibleDate.html>
- <http://www.rssboard.org/rss-profile#element-channel-item-pubdate>
-
<http://cyber.law.harvard.edu/rss/rss.html#ltpubdategtSubelementOfLtitemgt>

L'évènement publié par l'agenda étant dans le futur, cette date était dans
le futur. Or, selon tous client rss, cette ne peut qu'être légèrement dans
le passée, puisque l'élément a été publié il y a peu de temps. C'est à mon
pour cela que les clients rss affichaient des dates bidons (genre 1er
janvier 1970) en lieu et place de la date réelle.

Corrigé dans la branche generic.

Ploc


On Tue,  9 Nov 2010 20:06:06 +0100 (CET), svn AT agendadulibre.org wrote:
> Author: ploc
> Date: Tue Nov  9 20:06:04 2010
> New Revision: 489
> 
> Log:
> The field <dc:date> is linked to the pubDate, the publication date and
> time of the item, not the event start date as it is now. This is why
some
> rss feed readers were ignoring this date as the publication date cannot
be
> in the futur.
> 
> Modified:
>    branches/generic/rss.php
> 
> Modified: branches/generic/rss.php
>
==============================================================================
> --- branches/generic/rss.php  Mon Nov  8 21:29:01 2010        (r488)
> +++ branches/generic/rss.php  Tue Nov  9 20:06:04 2010        (r489)
> @@ -166,7 +166,7 @@
>    echo "   <dc:identifier>" . $event->id . "@" . $mailDomain .
>    "</dc:identifier>\n";
>    $evt_timezone = ereg_replace("^\+([0-9][0-9])([0-9][0-9])$",
"+\\1:\\2",
>                              date('O', $start_timestamp));
> -  $evt_date     = date('Y-m-d\TH:i', $start_timestamp);
> +  $evt_date     = str_replace(" ", "T", $event->decision_time);
>    echo "   <dc:date>" . $evt_date . $evt_timezone . "</dc:date>\n";
>  
>    if ($longitude && $latitude)
> _______________________________________________
> Devel mailing list
> Devel AT agendadulibre.org
> http://agendadulibre.org/cgi-bin/mailman/listinfo/devel