Problem/Motivation

Standard RSS feed can include multiple tags, for example, PubMed Feed

Currently only one is imported.

Steps to reproduce

Create new RSS feed, add text field to a content type and map import from Author Name.
screenshot

You can see that only first value is imported. It might be useful to import all authors in a predefined format. Proposed format would be comma separated, for example,
Markus Eyting
Min Xie
Felix Michalik

will generate Markus Eyting, Min Xie, Felix Michalik

Proposed resolution

Add option to import "All Authors" to default configuration for RSS feed

CommentFileSizeAuthor
rss-author-name.png41.14 KBirinaz

Issue fork feeds-3519989

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

irinaz created an issue. See original summary.

megachriz made their first commit to this issue’s fork.

megachriz opened merge request !211

megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Active » Needs review

I've added a source called 'authors'. It returns an array. If you want to format this as a comma-seperated string, use the Feeds Tamper plugin "Implode".

irinaz’s picture

irinaz commented

I tested code using DrupalPod extension, added new field text field "all authors" with unlimited values and another field with one value.
First field is imported correctly, second generates correct warning
" All authors one line (field_all_authors_one_line): All authors one line: this field cannot hold more than 1 values.
Please check your mappings. "
Marking RTBC

irinaz’s picture

irinaz commented
Status: Needs review » Reviewed & tested by the community

Tested on Pubmed Import, works as expected

  • megachriz committed 962a6043 on 8.x-3.x 
    Issue #3519989 by megachriz, irinaz: Added mapping source 'authors' to...
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Reviewed & tested by the community » Fixed

Thanks for testing! I merged the code.

Status: Fixed » Closed (fixed)

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