I tested Aggregation against some Aggregation feeds (the results are here: http://aggregation.novaak.net/?q=node/177 )
I found the following errors. There are two Atom feeds for example:
http://thewee3.blogspot.com/atom.xml and http://www.google.com/reader/public/atom/user/15260472571963679390/state...
After I create a feed and the module create the feed items, I see that the body of the node is empty in the case of all feed items.
And at the first feed, the feed items don't have a link to the original article.

CommentFileSizeAuthor
#2 ATOM10_0.patch2.21 KBAshraf Amayreh
#1 ATOM10.patch1.65 KBAshraf Amayreh

Comments

Ashraf Amayreh’s picture

Ashraf Amayreh commented
Assigned: Unassigned » Ashraf Amayreh
Status: Active » Reviewed & tested by the community
StatusFileSize
new ATOM10.patch1.65 KB

Thanks Aron,

I have a security update that the security team needs to commit before I can commit anything else and create a new release. So I'm going to attach the ATOM handler for you to test. Just drop it in place of the existing one. I really appreciate it if you could pound on my handlers as much as possible. Remember to rename it from .patch to .inc

Let me know what you think.

Ashraf Amayreh’s picture

Ashraf Amayreh commented
StatusFileSize
new ATOM10_0.patch2.21 KB

Yet even a better ATOM handler.

Ashraf Amayreh’s picture

Ashraf Amayreh commented
Status: Reviewed & tested by the community » Fixed

Ok, I've committed it now.

Simply download the new release to test it.

Ashraf Amayreh’s picture

Ashraf Amayreh commented
Status: Fixed » Closed (fixed)

I'm closing the issues that I think are finito because my issue list is growing and I'm starting to forget some important ones.

This is a pruning effort by me to keep things manageable.

Do contact me from my user page if you think there's something that requires urgent attention :-) Thanks Aaron.