feeds_config_form() is tailored to FeedsConfigurable::configForm() for no special reason. It could just as well be used for any form on an object. It can easily be abstracted out.

This is a prerequisite for #651478: Mapping on import

CommentFileSizeAuthor
#2 849834_feeds.patch6.99 KBrbayliss
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alex_b’s picture

alex_b CreditAttribution: alex_b commented

This work is basically done in #651478-34: Mapping on import

rbayliss’s picture

rbayliss CreditAttribution: rbayliss commented
Status: Active » Needs review
FileSize
849834_feeds.patch6.99 KB

I think this covers the switch. Tests seem to run fine. Most of this is taken straight from the mapping on import patch.

alex_b’s picture

alex_b CreditAttribution: alex_b commented
Title: Abstract feeds_config_form() » Generalize feeds_config_form() to feeds_form()
Status: Needs review » Fixed

This is committed, thank you!

http://drupal.org/cvs?commit=419172

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.