Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EasyTask] RSS 1.0 (RDF) support #22

Closed
Phyks opened this issue Aug 7, 2014 · 4 comments
Closed

[EasyTask] RSS 1.0 (RDF) support #22

Phyks opened this issue Aug 7, 2014 · 4 comments

Comments

@Phyks
Copy link
Member

Phyks commented Aug 7, 2014

This is an easytask, something beginners can fix quickly, to have a first look in the code. All the necessary information to solve this issue should be given and it should be pretty much straightforward.

RDF feeds are not parsed, such as the debian security feed.

These are old feeds and Freeder only handles RSS 2.0 and Atom for now. However, this should not be a big deal to add support for these feeds, as most of the difference is just a namespace extension. I need to use fully Freeder instead of Leed, so I can work on it (probably around 20/08). Feel free to work on it in the meantime if you want.

Description:
AFAIK, the problem comes from feed2array which returns false for such a feed. I think the problem is due to the first switch which distinguishes rss, ATOM and default cases for the first tag in the feed. However, with a RDF feed, the first tag is rdf, falling into default case and thus returning false. First, rdf case should be added to this switch. Then, RDF feeds should be parsed almost the same way as RSS feeds, so just a few corrections below to ensure everything is working nicely, and some tests.

Some links, for myself:

@Phyks
Copy link
Member Author

Phyks commented Aug 23, 2014

Started to implement it, but still some work to do.

@Phyks
Copy link
Member Author

Phyks commented Sep 2, 2014

Backend feed2array handles them now. To be tested further.

@Phyks
Copy link
Member Author

Phyks commented Sep 3, 2014

TODO: guid + date are not working

@Phyks
Copy link
Member Author

Phyks commented Sep 3, 2014

Closed by abe41de.

@Phyks Phyks closed this as completed Sep 3, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant