Ticket #421 (assigned enhancement)

Opened 4 years ago

Last modified 3 years ago

Patch to replace MagpieRSS with SimplePie

Reported by: gregarius.net@… Owned by: sdcosta
Priority: normal Milestone: Gregarius 0.5.5
Component: BUGS Version:
Severity: normal Keywords:
Cc:

Description

I've taken a first swipe at replacing MagpieRSS with SimplePie?. The attached diff against Gregarius trunk @1498 works fairly well for me. Although Magpie isn't doing any parsing, I haven't entirely excised it from the project, mainly due to the patches added by Gregarius which I'm either not comfortable changing or haven't gotten around to fixing yet.

SP isn't quite ready for prime time, but they're quickly nearing a 1.0 release.

Attachments

gregarius-simplepie.patch (118.1 kB) - added by gregarius.net@… 4 years ago.

Change History

Changed 4 years ago by gregarius.net@…

Changed 4 years ago by sdcosta

  • owner changed from mbonetti to sdcosta
  • status changed from new to assigned

Thanks for the patch. It is quite useful. However, I think we will wait till SimplePie RC1 is released. The developers in #simplepie said that they would separate out the parsing functions into a new class and we will just use that when it is available.

Since SimplePie does not support digest authentication, etags other things that Snoopy does, we will continue to use Snoopy to fetch the feed and then use SimplePie for the parsing.

Changed 3 years ago by reporter

RC1 landed recently, but you probably already knew that :)

http://simplepie.org/blog/2007/06/23/simplepie-10-release-candidate-is-here/

Not too sure if the work on separating the parser ever got done, but might be worth another look

Note: See TracTickets for help on using tickets.