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.
The docs for \Drupal\Core\Field\Plugin\Field\FieldFormatter\EntityReferenceEntityFormatter references that an entity manager should be provided, but it is incorrect. Instead \Drupal::entityManager() is used in the code instead of it being injected. We also use the deprecated entity_view() function.
Comment | File | Size | Author |
---|---|---|---|
#5 | 2562099-5.patch | 4.6 KB | amateescu |
#3 | 2562099-fix-entityreferenceentityformatter.patch | 4.06 KB | Dave Reid |
#2 | 2562099-fix-entityreferenceentityformatter.patch | 4.35 KB | Dave Reid |
Comments
Comment #2
Dave ReidComment #3
Dave ReidComment #4
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedWe should use the 'entity_display.repository' service now.
And the 'entity_type.manager' service here.
Comment #5
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedFixed the two minor review points from above.. the patch didn't apply anymore so there's no interdiff.
The changes are very small so I'm setting this straight to RTBC.
Comment #6
catchDue to the constructor changes this needs to go into a minor release only. Committed/pushed to 8.1.x, thanks!