- πΊπΈUnited States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request β as a guide.
Was previously tagged for IS update and change record which still appear the case.
Did not review or test.
- First commit to issue fork.
- Merge request !5698Issue #2890844: Migration Lookup plugin does not return multiple values when matched β (Open) created by acbramley
- π¬π§United Kingdom barry_fisher
For what it's worth, I came across this issue and found the patch in #92 π Migration Lookup plugin does not return multiple values when matched Needs work worked perfectly for me.
Thanks @shashikant_chauhan!
For added context (see below), I'm processing a JSON file with the url source plugin from the migrate_plus module. As the paragraph mappings come from a `paragraphs` subkey in the JSON structure, I had to do the following in a sub_process plugin. The issue was that only migration lookups only worked for the first migration listed: `blog_paragraphs_copy`. I know this because I tried swapping the order and the `blog_paragraphs_copy_media` then worked but `blog_paragraphs_copy` didn't.
With the patch in #92 π Migration Lookup plugin does not return multiple values when matched Needs work both migration lookups are now working successfully. If an item in one migration is found, then it's used, otherwise the other is used.
field_structured_content: plugin: sub_process source: paragraphs process: target_id: - plugin: migration_lookup migration: - blog_paragraphs_copy - blog_paragraphs_copy_media source: id - plugin: extract index: - 0 target_revision_id: - plugin: migration_lookup migration: - blog_paragraphs_copy - blog_paragraphs_copy_media source: id - plugin: extract index: - 1
- π¬π§United Kingdom joekers UK
Thanks @ckng for the reroll - this was based on the 2890844-65.drupal.patch and now applies cleanly to 10.2.x