[drupal-devel] [feature] OPML Import for Aggregator

Dries drupal-devel at drupal.org
Thu Jan 27 21:22:25 UTC 2005

 Project:      Drupal
 Version:      cvs
 Component:    aggregator.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  arnabdotorg
 Reported by:  arnabdotorg
 Updated by:   Dries
 Status:       patch

1. This patch does not conform Drupal's coding style at all.

2. The error handling of aggregator_import() is bare bones (not

3. Use drupal_xml_parser_create() instead of xml_parser_create().

4. Drop the "sorry"'s.

5.  form_set_error() is used incorrectly.

Please refine and resubmit.


Previous comments:

January 27, 2005 - 08:13 : arnabdotorg

Attachment: http://drupal.org/files/issues/import_opml.patch (3.96 KB)

The aggregator supports OPML export for all feeds, but not import. This
patch enables import of OPML lists, allowing people to import lists of
RSS feeds they've already assembled in another RSS app.

Demo: http://arnab.drupaldevs.org/aggregator

Patch: http://arnab.drupaldevs.org/tmp/import_opml.patch

View: http://drupal.org/node/16282
Edit: http://drupal.org/project/comments/add/16282

More information about the drupal-devel mailing list