Subject: | Improvement suggestion: generating Category tags with optional domain attribute |
Date: | Mon, 23 Jun 2014 13:01:03 -0700 |
To: | "bug-XML-FeedPP [...] rt.cpan.org" <bug-XML-FeedPP [...] rt.cpan.org> |
From: | Simon Rosenthal <simon_rosenthal [...] yahoo.com> |
This is for XML--FeedPP Version 0.43.
It would be nice if you could extend the $item->category() method to include a ‘domain’ attribute, as specified by the RSS 2.0 spec (see http://cyber.law.harvard.edu/rss/rss.html#ltcategorygtSubelementOfLtitemgt ).
From a quick look at the code, I’d suggest doing something like
$item->category_with_domain($text, $domaintext)
Also, it would be helpful to document that you add multiple <Category> elements to a feed item by passing an arrayref to $item->category(), and not by making multiple calls to that method (only the last one is kept if you do that).
thanks!
-Simon