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
CommentFileSizeAuthor
aggregator_simplexml.patch19.59 KBaron novak

Comments

robin monks’s picture

robin monks commented
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

alex_b’s picture

alex_b commented

Aron, is this issue duplicate of #72915: Mofe feed parser to includes/feed.inc now? http://drupal.org/node/72915#comment-992724

Please mark as duplicate if that's the case.

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Needs work » Closed (duplicate)

http://drupal.org/node/72915#comment-992724

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Closed (duplicate) » Closed (fixed)