Skip to content
This repository has been archived by the owner. It is now read-only.

Rewrite RSS Import tool to Qt #175

Open
zxiiro opened this issue May 30, 2012 · 3 comments
Open

Rewrite RSS Import tool to Qt #175

zxiiro opened this issue May 30, 2012 · 3 comments

Comments

@zxiiro
Copy link
Member

@zxiiro zxiiro commented May 30, 2012

Today we use python-feedparser for this functionality however we could reduce Freeseer 3rdparty dependencies if we used QtXml instead since we already use Qt quite heavily.

Qt provides the ability to read xml files in the form of QtXml.

@ghost ghost assigned riolowry Sep 20, 2013
@riolowry
Copy link
Contributor

@riolowry riolowry commented Sep 20, 2013

per http://qt-project.org/doc/qt-5.0/qtxml/qtxml-module.html the QtXml module is not actively maintained anymore. The QXmlStreamReader and QXmlStreamWriter classes in Qt Core are recommended instead.

@riolowry
Copy link
Contributor

@riolowry riolowry commented Sep 22, 2013

per discussion with @dideler, python-feedparser provides functionality for url lookup and fetching of the feed which includes error handling and is well tested and well-maintained, if we replace this we need to implement and test this ourselves so it may be worth looking at including python-feedparser source code in the lib folder (possibly as a git submodule or some other approach. It is MIT licensed so is compatible with our project). This would reduce the end user's need to download an additional dependency to run the software.

Alternatively after some research I believe we should be able to implement the feed lookup and fetching using QUrl class which is also in QtCore.

Which approach would you recommend @zxiiro?

@zxiiro
Copy link
Member Author

@zxiiro zxiiro commented Sep 23, 2013

If you can do it all in Qt I'd say go that route. QUrl should cover what you need here but you'll need to do more research to make sure.

The key is to ensure that your new tool works exactly like the old tool that it's replacing. One thing of note is the old tool despite being called the "RSS Import tool" it's not actually very generic. It was designed specifically for fosslc.org's RSS feed. Example feeds we've used in the past:

http://fosslc.org/drupal/presentations_rss/sc2011
http://fosslc.org/drupal/presentations_rss/summercamp2010

I think every site my be different. I actually envision a rewrite of the RSS tool would include turning it into a full fledged Freeseer plugin type related to bug #151. This way other conferences and sites could make a RSS import plugin that is specific to their conference RSS data structure.

@zxiiro zxiiro removed this from the Open Academy / UCOSP milestone Feb 12, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants