Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
#2724941: Need to be able to lookup destination IDs by partial source IDs deprecated \Drupal\migrate\Plugin\MigrateIdMapInterface::lookupDestinationId()
before deprecation testing was a thing and it never removed all the usages.
Proposed resolution
- Remove all usages
- Add @trigger_error
- Think about testing
Remaining tasks
User interface changes
None
API changes
None
Data model changes
None
Release notes snippet
N/a
Comment | File | Size | Author |
---|---|---|---|
#2 | 3068089-2.patch | 24.69 KB | alexpott |
Comments
Comment #2
alexpottComment #3
mikelutzI wondered if this should just be
if ($destination_ids = reset($lookup)) break;
but this is fine, and there is follow-up work around this block in #2890844: Migration Lookup plugin does not return multiple values when matched to actually allow access to the rest of the matches instead of just throwing them away.All straightforward, ready to go.
Comment #4
catchCommitted 5a27c27 and pushed to 8.8.x. Thanks!
Comment #6
catch