Map feed Title and feed URL

Great job!
Tested on 7.x-2.0-alpha7. Works as expected.

This also worked for me on 7.x-2.0-alpha7.

Being bold and setting to reviewed and tested.

There are a few coding standard violations.

I can understand parent:title and feed_source:url, but importer name and description??

Description looks fine to me. Reapplied #2 patch at latest 7.x-2.x-dev. Fixed some code style issues.

Sure, it looks fine. But at some point we have to ask ourselves what features are useful and necessary, and what's just cluttering things up. I've never even set a description for an importer.

How is this supposed to work?

I tried patches #2 and #6 separately and then I go to:

mysite.com/admin/structure/feeds/my_feed/mapping

And then I click on Mapping in the node processor and I would expect to be able to map the feed title but nothing has changed, there are no new options.

Am I missing something here?

Am I missing something here?

Patch #6 attach new mapping sources only when it attached to contet_type.
I've fix this, also I change option name:
from Feed source: URL to Feed source: Source.

I have test this patch only with HTTP Fetcher, so please test it with files.

Please use patch from #11.

I tested the patch in action and I found no issues with it. I tested with an importer attached to a content type and with the fetchers HTTP fetcher and File fetcher.

Code review:

1. +++ b/plugins/FeedsParser.inc
   @@ -141,16 +158,40 @@ abstract class FeedsParser extends FeedsPlugin {
   +    // Node elements.
   +    elseif (strpos($element_key, 'parent:') === 0 &&
   +        $source->feed_nid &&
   +        ($node = node_load($source->feed_nid))) {
   

   These lines could be improved a bit. According to the code standards, if control structure condition tend to get long it's better to prepare the conditions separately.
   https://www.drupal.org/coding-standards#linelength

2. Since this patch adds new sources, these sources need test coverage. Create a test method for each source. I think for the source "feed_source:source" there should be a test that covers the case for HTTP fetcher and another test that covers the case for the File fetcher.