Issue #2387419 by MegaChriz: Auto detect dependencies when putting Feeds importer in a feature

feeds.module

@@ -720,6 +720,18 @@ function feeds_field_extra_fields() {
   return $extras;
 }
 
+/**
+ * Implements hook_features_pipe_COMPONENT_alter() for component "feeds_importer".
+ *
+ * Automatically adds dependencies when a Feed importer is selected in Features.
+ */
+function feeds_features_pipe_feeds_importer_alter(&$pipe, $data, &$export) {
+  foreach ($data as $importer_id) {
+    if ($importer = feeds_importer_load($importer_id)) {
+      $export['dependencies'] = array_merge($export['dependencies'], $importer->dependencies());
+    }
+  }
+}
 
 /**
  * Implements hook_system_info_alter().

includes/FeedsConfigurable.inc

@@ -214,6 +214,16 @@ abstract class FeedsConfigurable {
     drupal_set_message(t('Your changes have been saved.'));
     feeds_cache_clear(FALSE);
   }
+
+  /**
+   * Returns an array of required modules.
+   *
+   * @return array
+   *   The modules that this configurable requires.
+   */
+  public function dependencies() {
+    return array();
+  }
 }
 
 /**

includes/FeedsImporter.inc

@@ -271,6 +271,17 @@ class FeedsImporter extends FeedsConfigurable {
     }
     parent::configFormSubmit($values);
   }
+
+  /**
+   * Implements FeedsConfigurable::dependencies().
+   */
+  public function dependencies() {
+    $dependencies = parent::dependencies();
+    foreach ($this->plugin_types as $plugin_type) {
+      $dependencies = array_merge($dependencies, $this->$plugin_type->dependencies());
+    }
+    return $dependencies;
+  }
 }
 
 /**

includes/FeedsSource.inc

@@ -799,4 +799,12 @@ class FeedsSource extends FeedsConfigurable {
     lock_release("feeds_source_{$this->id}_{$this->feed_nid}");
   }
 
+  /**
+   * Implements FeedsConfigurable::dependencies().
+   */
+  public function dependencies() {
+    $dependencies = parent::dependencies();
+    return array_merge($dependencies, $this->importer()->dependencies());
+  }
+
 }

plugins/FeedsPlugin.inc

@@ -209,6 +209,21 @@ abstract class FeedsPlugin extends FeedsConfigurable implements FeedsSourceInter
     }
     return $result;
   }
+
+  /**
+   * Implements FeedsConfigurable::dependencies().
+   */
+  public function dependencies() {
+    $dependencies = parent::dependencies();
+
+    // Find out which module provides this plugin.
+    $plugin_info = $this->pluginDefinition();
+    if (isset($plugin_info['module'])) {
+      $dependencies[$plugin_info['module']] = $plugin_info['module'];
+    }
+
+    return $dependencies;
+  }
 }
 
 /**

plugins/FeedsProcessor.inc

@@ -1012,6 +1012,21 @@ abstract class FeedsProcessor extends FeedsPlugin {
     return $message;
   }
 
+  /**
+   * Overrides FeedsPlugin::dependencies().
+   */
+  public function dependencies() {
+    $dependencies = parent::dependencies();
+
+    // Find out which module defined the entity type.
+    $info = $this->entityInfo();
+    if (isset($info['module'])) {
+      $dependencies[$info['module']] = $info['module'];
+    }
+
+    return $dependencies;
+  }
+
 }
 
 class FeedsProcessorBundleNotDefined extends Exception {}

plugins/FeedsTermProcessor.inc

@@ -224,4 +224,14 @@ class FeedsTermProcessor extends FeedsProcessor {
     }
     throw new Exception(t('No vocabulary defined for Taxonomy Term processor.'));
   }
+
+  /**
+   * Overrides FeedsProcessor::dependencies().
+   */
+  public function dependencies() {
+    $dependencies = parent::dependencies();
+    $dependencies['taxonomy'] = 'taxonomy';
+    return $dependencies;
+  }
+
 }