[Mono-dev] monologue and RSS.NET.dll
Wade Berrier
wberrier at novell.com
Fri Feb 9 16:49:32 EST 2007
Good idea. Patch attached. If no one objects, I'll commit.
Wade
On Fri, 2007-02-09 at 09:02 -0200, Rafael Teixeira wrote:
> I vote yes, but it would be nice to tag the error in the author's list
> to the right, as when the feed can't be read.
>
> :)
>
> On 2/8/07, Wade Berrier <wberrier at novell.com> wrote:
> > Hi,
> >
> > If RSS.NET.dll (DateTime.Parse()) can't parse a date it uses
> > DateTime.MinValue in it's place.
> >
> > This was happening for lupus' feed, making his old posts always show up
> > on monologue.
> >
> > The maintainer of advogato fixed the date format in his rss feeds so
> > this shouldn't happen anymore.
> >
> > I'm thinking it's a bad idea that feeds with invalid date formats become
> > permalinks (monologue's worker):
> >
> > if (i.PubDate == DateTime.MinValue || i.PubDate >= minPubDate) {
> >
> > Ok to remove 'i.PubDate == DateTime.MinValue ||' ?
> >
> > Wade
> >
> > _______________________________________________
> > Mono-devel-list mailing list
> > Mono-devel-list at lists.ximian.com
> > http://lists.ximian.com/mailman/listinfo/mono-devel-list
> >
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: monologue_invalid_dates.patch
Type: text/x-patch
Size: 1877 bytes
Desc: not available
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20070209/205f0082/attachment.bin
More information about the Mono-devel-list
mailing list