Skip Menu |

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

Report information
The Basics
Id: 46897
Status: open
Priority: 0/
Queue: XML-RSS
People
Owner: Nobody in particular
Requestors: GLORYBOX [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri Jun 12 15:10:32 2009 GLORYBOX [...] cpan.org - Ticket created
Subject: Access category element's domain attribute
It's not a bug actually, but lack of functionality. There is no way to get category element "domain" attribute. Probably it's good idea to represent categories as a hashref: { 'domain' => [ 'category1', 'category2', 'category3' ], 'domain2' => [ 'category4', 'category5', 'category6' ], } What do you think?
  Thu Jun 25 17:20:43 2009 GLORYBOX [...] cpan.org - Correspondence added
I've prepared patch implementing described functionality. Would you review it? Are you interested?
  Thu Jun 25 17:20:46 2009 GLORYBOX [...] cpan.org - Status changed from 'new' to 'open'
  Thu Jun 25 19:22:14 2009 SHLOMIF [...] cpan.org - Correspondence added
Hi GLORYBOX! Thanks for your interest in XML::RSS. I'm sorry for not replying earlier. See below for my response. On Fri Jun 12 15:10:32 2009, GLORYBOX wrote: Show quoted text
> It's not a bug actually, but lack of functionality. > There is no way to get category element "domain" attribute. > > Probably it's good idea to represent categories as a hashref: > { > 'domain' => [ 'category1', 'category2', 'category3' ], > 'domain2' => [ 'category4', 'category5', 'category6' ], > } > > What do you think?
Regarding this functionality - it seems like it may break backwards compatibility so please make it an optional feature (see ->parse's %options argument, etc.). Also make sure new tests are added and that all existing tests pass. Regards, -- Shlomi Fish