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)



Subject: Access category element's domain attribute
Download (untitled) / with headers
text/plain 313b
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?
I've prepared patch implementing described functionality. Would you review it? Are you interested?
Download (untitled) / with headers
text/plain 751b
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


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.