Add posibility to show all process plugins not only first one.

  field_ap_technologies:
   -
    plugin: skip_on_empty
    method: process
    source: Technologies
   -
    plugin: explode
    delimiter: ','
   -
    plugin: entity_generate
    ignore_case: true
CommentFileSizeAuthor
#4 after.png11.72 KBalexdoma
#4 before.png11.2 KBalexdoma
#2 2932571-display_all_process_info.patch771 bytesalexdoma

Comments

alexdoma created an issue. See original summary.

alexdoma’s picture

alexdoma commented
StatusFileSize
new 2932571-display_all_process_info.patch771 bytes

Patch for this

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Active » Needs review

Can we get a screenshot of what this looks like? And any tests you can add would be super nice.

alexdoma’s picture

alexdoma commented
StatusFileSize
new before.png11.2 KB
new after.png11.72 KB

Hi, screenshots were added

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Needs review » Reviewed & tested by the community

That looks much prettier.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Oops, now we need a re-roll.

alexdoma’s picture

alexdoma commented

Hi, @heddn !
Can you explain what i should change in my patch? Thank you

heddn’s picture

heddn
Primary language English
Location Illinois
commented

The changes don't apply on HEAD because of other changes. The instructions for a re-roll: https://www.drupal.org/patch/reroll

alexdoma’s picture

alexdoma commented

@heddn
I could apply the patch, so no reroll is needed.
i have used last code from here https://git.drupalcode.org/project/migrate_tools

alexdoma’s picture

alexdoma commented
Status: Needs work » Needs review
heddn’s picture

heddn
Primary language English
Location Illinois
commented
Category: Feature request » Task
Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs reroll

Sure enough, it does.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Reviewed & tested by the community » Fixed

  • heddn committed c3de8be on 8.x-4.x authored by alexdoma 
    Issue #2932571 by alexdoma, heddn: Display process information of a...

Status: Fixed » Closed (fixed)

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