SimpleXML parser for aggregator [#302245]

This patch replaces the existing aggregator parser with a SimpleXML based solution.

It is part of the larger effort of improving Drupal's core aggregator. (The plan now is to create patches and open issues for every portion in the initial big patch here #236237 ).

RSS/Atom/RDF
Supports namespaces, enclosures
Based on SimpleXML
Doesn't require an external library
Only conditionally loaded

Comment	File	Size	Author
	aggregator_simplexml.patch	19.59 KB	aron novak

Comments

Comment #1

robin monks commented 31 August 2008 at 22:52

Status:

Needs review

» Needs work

When using http://robinmonks.com/feed as the feed, and then updating the feed:

* notice: Undefined variable: parser_out in /srv/w*a/htdocs/d1/modules/aggregator/aggregator.module on line 613.
* warning: Invalid argument supplied for foreach() in /srv/w*a/htdocs/d1/modules/aggregator/aggregator.module on line 613.
* notice: Undefined variable: parser_out in /srv/w*a/htdocs/d1/modules/aggregator/aggregator.module on line 627.
* notice: Undefined variable: parser_out in /srv/w*a/htdocs/d1/modules/aggregator/aggregator.module on line 628.
* warning: Invalid argument supplied for foreach() in /srv/w*a/htdocs/d1/modules/aggregator/aggregator.module on line 506.

Robin