Skip Menu |

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

Report information
The Basics
Id: 5024
Status: resolved
Priority: 0/
Queue: XML-RSS

People
Owner: ABH [...] cpan.org
Requestors: cpan [...] thechaz.net
Cc:
AdminCc:

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



Subject: permaLink attribute
Hi, I submitted a patch to report the isPermaLink attribute in guid in $item->{'permaLink'}, over at: http://sourceforge.net/tracker/index.php?func=detail&aid=880926&group_id=66921&atid=516236 I wasn't sure which of these places was the most appropriate to submit patches, so apologies if this duplication was unnecessary.
On Sun Jan 25 14:37:37 2004, guest wrote: Show quoted text
> Hi, > > I submitted a patch to report the isPermaLink attribute in guid in > $item->{'permaLink'}, over at: > http://sourceforge.net/tracker/index.php?
func=detail&aid=880926&group_id=66921&atid=516236 Show quoted text
> > I wasn't sure which of these places was the most appropriate to submit > patches, so apologies if this duplication was unnecessary.
The RT site would have been. The sourceforge tracker is down (or at least the link above doesn't work). Can you post the patch (and a test case) here? - ask
From: cpan [...] thechaz.net
On Sat Mar 11 19:49:27 2006, ABH wrote: Show quoted text
> The RT site would have been. The sourceforge tracker is down (or at > least the link above > doesn't work). > > Can you post the patch (and a test case) here? > > > - ask
ask, I don't know if this counts as a "test case" or not, but I attached the patch, an example RSS file, and an example script demonstrating when the guid permalink attribute is true, false, and missing. Chaz
From: cpan [...] thechaz.net
Hm. I don't see those files via the web nor the email notification I got, so I'm going to give a link to them: http://www.thechaz.net/xmlrss/rss.txt http://www.thechaz.net/xmlrss/XML-RSS.patch http://www.thechaz.net/xmlrss/rss.xml Sorry if these files were actually posted and I'm just not seeing them. This is my first time using RT. Chaz
Subject: Re: [rt.cpan.org #5024] permaLink attribute
Date: Fri, 21 Apr 2006 23:01:02 -0700
To: bug-XML-RSS [...] rt.cpan.org
From: Ask Bjørn Hansen <ask [...] perl.org>
On Apr 21, 2006, at 9:34 PM, Guest via RT wrote: Show quoted text
> Sorry if these files were actually posted and I'm just not seeing > them. > This is my first time using RT.
Thanks Chaz, it looks good. I can easily turn your demonstration script into a test script. I have to prepare for the MySQL conference next week, so it'll be a week or two before I can look at it so please leave the files on your server until then. (Or attach them to the ticket via the webinterface) Thanks again! - ask -- http://www.askbjoernhansen.com/
From: cpan [...] thechaz.net
On Sat Apr 22 02:01:51 2006, ask@perl.org wrote: Show quoted text
> > On Apr 21, 2006, at 9:34 PM, Guest via RT wrote: >
> > Sorry if these files were actually posted and I'm just not seeing > > them. > > This is my first time using RT.
> > Thanks Chaz, it looks good. I can easily turn your demonstration > script into a test script. > > I have to prepare for the MySQL conference next week, so it'll be a > week or two before I can look at it so please leave the files on your > server until then. (Or attach them to the ticket via the webinterface) > > Thanks again! > > > - ask
Take your time! Just let me know when it's safe to take them down.
On Sat Apr 22 02:01:51 2006, ask@perl.org wrote: Show quoted text
> > On Apr 21, 2006, at 9:34 PM, Guest via RT wrote: >
> > Sorry if these files were actually posted and I'm just not seeing > > them. > > This is my first time using RT.
> > Thanks Chaz, it looks good. I can easily turn your demonstration > script into a test script.
Attached is a patch that does that and everything else that is needed for the patch integration. Regards, Shlomi Fish Show quoted text
> > I have to prepare for the MySQL conference next week, so it'll be a > week or two before I can look at it so please leave the files on
your Show quoted text
> server until then. (Or attach them to the ticket via the
webinterface) Show quoted text
> > Thanks again! > > > - ask >

Message body is not shown because it is too large.

Fixed a release or two ago.