Skip Menu |

This queue is for tickets about the XML-Feed CPAN distribution.

Report information
The Basics
Id: 20763
Status: resolved
Priority: 0/
Queue: XML-Feed

People
Owner: DAVECROSS [...] cpan.org
Requestors: gregor+debian [...] comodo.priv.at
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Problem with parsing dates in Atom feeds
Date: Sun, 30 Jul 2006 20:13:19 +0200
To: bug-XML-Feed [...] rt.cpan.org
From: gregor herrmann <gregor+debian [...] comodo.priv.at>
Hi, Joey Hess has noticed a problem with the parsing of dates in XML::Feed:Atom. The modules looks for "modified" and "issued" elements and not for "updated" and "published" elements, as required by RFC 4287. You can find the full bug report at http://bugs.debian.org/380498 Please consider applying Joey's patch which is attached to the bug report at the mentioned URL. (The bug report is against 0.08 but AFAICS the code is the same in 0.10). Thanks in advance, Gregor (Debian Perl Group) -- .''`. http://info.comodo.priv.at/ | gpg key ID: 0x00F3CFE4 : :' : debian: the universal operating system - http://www.debian.org/ `. `' member of https://www.vibe.at/ | how to reply: http://got.to/quote/ `- NP: Tom Waits: The Piano Has Been Drinking
Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

Should be fixed in 0.2
From: fschlich [...] zedat.fu-berlin.de
Hi, a review of the original patch in Debian showed that only part of the patch was taken. The missing hunk is: --- a/lib/XML/Feed/Entry/Format/Atom.pm +++ b/lib/XML/Feed/Entry/Format/Atom.pm @@ -145,7 +145,9 @@ if (@_) { $entry->{entry}->issued(DateTime::Format::W3CDTF- Show quoted text
>format_datetime($_[0])) if $_[0];
} else { - $entry->{entry}->issued ? iso2dt($entry->{entry}->issued) : undef; + return iso2dt($entry->{entry}->issued) if $entry->{entry}- Show quoted text
>issued;
+ return iso2dt($entry->{entry}->published) if $entry->{entry}- Show quoted text
>published;
+ return undef; } } Florian
Fixed in version 0.48 - currently en route to CPAN.