Some feeds are not parsed by simplefeed (items are not created). Up so far:

http://news.nationalgeographic.com/index.rss
http://www.nature.com/news/rss.rdf
http://sciencenow.sciencemag.org/rss/current.xml
http://www.sciam.com/xml/sciam.xml

At least the last one is a 'body is required' error; some items in this feed only have a title. The other ones fail for no apparent reason and the other drupal based aggregators had no problems with them.

Comments

m3avrck’s picture

m3avrck commented

Ok, I've fixed feed items bodies being required, but the others not sure.

Try here: http://www.simplepie.org/demo/

If they fail there, that is an issue with them so file a bug report over there :-P

I've updated to latest trunk in the release so that might have fixed it too.

FiReaNGeL’s picture

FiReaNGeL commented

All feeds pass the simplepie demo test, and all feeds still fail for some reason. I'll try to investigate this weekend.

m3avrck’s picture

m3avrck commented

Strange -- must be during the node creation process for whatever reason, hmm... maybe something is failing validation still? Ping me on #drupal to help debug.

FiReaNGeL’s picture

FiReaNGeL commented

Ok, the Scientific American feed is now OK since you fixed the 'no description' problem. About the three other feeds:

$feed->data is containing all the necessary data, but for some reason, $feed->get_item_quantity() reports 0 in all three cases. So thats why they're not updating, and its clearly a simplepie bug.

Opened an 'issue' on their support forum: http://simplepie.org/support/viewtopic.php?pid=2707#p2707

For me its not a showstopper, but I guess getting it fixed as soon as possible will benefit everyone :)

m3avrck’s picture

m3avrck commented

Thanks fireangel! Hopefully we can get that straightened out with the SimplePie guys.

micheleannj’s picture

micheleannj commented

Can you give a list of feed types currently supported / not supported? I noticed that /rss/ isn't working (only get titles) but atom.xml seems fine. I'd like to roll this module out to my beta users but it would be nice to document what's currently working.

Again, lemme know if I can help out in any way.

`maj

FiReaNGeL’s picture

FiReaNGeL commented

Every feed type is supported as far as I know, its just that I stumbled (up so far) on three specific feeds that were failing.

m3avrck’s picture

m3avrck commented

Everything is supported :-p

http://simplepie.org/

Some just don't work, for whatever reason. File a bug with SimplePie and/or test their demo to see if feeds work or not.

m3avrck’s picture

m3avrck commented
Status: Active » Fixed

Fixed!

Turns out to be a bug in SimplePie, I put in a workaround to use foreach() instead of a for()

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)