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

[Dries]

 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
helpful).  

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.

Dries



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