Bug #24742 for XML-RSS: guid not being preserved over parse save

Fri Feb 02 13:46:50 2007 magnus [...] alum.mit.edu - Ticket created

Subject:	guid not being preserved over parse save
Date:	Fri, 2 Feb 2007 13:46:21 -0500
To:	bug-XML-RSS [...] rt.cpan.org
From:	Daniel Risacher <magnus [...] alum.mit.edu>

If I do this: use XML::RSS; my $rss = XML::RSS->new(); $rss->parsefile("/home/magnus/www/phc.xml"); $rss->save("/home/magnus/www/phc-test.xml"); Then all the guids of my items come out as '1' in the resulting file: [m5] ~$ diff -u ~/www/phc.xml ~/www/phc-test.xml|grep guid| tail -2 -<guid isPermaLink="true">http://www.risacher.org/mp3a/mp3/phc/phc-20070127.mp3</guid> +<guid isPermaLink="true">1</guid> This is suboptimal.

Wed Feb 14 14:58:48 2007 SHLOMIF [...] cpan.org - Correspondence added

On Fri Feb 02 13:46:50 2007, magnus@alum.mit.edu wrote: Show quoted text

> If I do this: > > use XML::RSS; > my $rss = XML::RSS->new(); > $rss->parsefile("/home/magnus/www/phc.xml"); > $rss->save("/home/magnus/www/phc-test.xml"); > > Then all the guids of my items come out as '1' in the resulting file: > > [m5] ~$ diff -u ~/www/phc.xml ~/www/phc-test.xml|grep guid| tail -2 > -<guid isPermaLink="true">http://www.risacher.org/mp3a/mp3/phc/phc- > 20070127.mp3</guid> > +<guid isPermaLink="true">1</guid> > > This is suboptimal. >

I cannot reproduce with the XML-RSS trunk but I added a regression test for this in revision 9103 just in case the bug returns. Can you please svn checkout http://svn.perl.org/modules/XML-RSS/trunk/ and see if the bug still exists there? Please also attach the file you used as input. Ask, I suggest you close the bug once a new stable version of XML-RSS is released. Regards, Shlomi Fish

Wed Feb 14 14:58:49 2007 The RT System itself - Status changed from 'new' to 'open'

Mon Feb 19 15:40:50 2007 ABH [...] cpan.org - Status changed from 'open' to 'resolved'

Tue Feb 20 15:41:43 2007 magnus [...] alum.mit.edu - Correspondence added

Subject:	Re: [rt.cpan.org #24742] guid not being preserved over parse save
Date:	Tue, 20 Feb 2007 15:41:03 -0500
To:	bug-XML-RSS [...] rt.cpan.org
From:	Daniel Risacher <magnus [...] alum.mit.edu>

In version 1.22, the error was on line 1551 of XML/RSS.pm where $item->{$guid} should have been $item->{guid} The issue is resolved in the SVN version. On Wed, 14 Feb 2007 14:58:50 -0500, "Shlomi Fish via RT" <bug-XML-RSS@rt.cpan.org> wrote: Show quoted text

> > <URL: http://rt.cpan.org/Ticket/Display.html?id=24742 > > > On Fri Feb 02 13:46:50 2007, magnus@alum.mit.edu wrote:

>> If I do this: >> >> use XML::RSS; >> my $rss = XML::RSS->new(); >> $rss->parsefile("/home/magnus/www/phc.xml"); >> $rss->save("/home/magnus/www/phc-test.xml"); >> >> Then all the guids of my items come out as '1' in the resulting file: >> >> [m5] ~$ diff -u ~/www/phc.xml ~/www/phc-test.xml|grep guid| tail -2 >> -<guid isPermaLink="true">http://www.risacher.org/mp3a/mp3/phc/phc- >> 20070127.mp3</guid> >> +<guid isPermaLink="true">1</guid> >> >> This is suboptimal. >>

> > I cannot reproduce with the XML-RSS trunk but I added a regression test > for this in revision 9103 just in case the bug returns. Can you please > svn checkout http://svn.perl.org/modules/XML-RSS/trunk/ and see if the > bug still exists there? > > Please also attach the file you used as input. > > Ask, I suggest you close the bug once a new stable version of XML-RSS is > released. > > Regards, > > Shlomi Fish

Tue Feb 20 15:41:45 2007 The RT System itself - Status changed from 'resolved' to 'open'

Thu Feb 22 17:56:55 2007 ABH [...] cpan.org - Status changed from 'open' to 'resolved'