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

#2 849834_feeds.patch6.99 KBrbayliss
Members fund testing for the Drupal project. Drupal Association Learn more


alex_b’s picture

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

rbayliss’s picture

Status: Active » Needs review
6.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

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

This is committed, thank you!


Status: Fixed » Closed (fixed)

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