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
\Drupal\embed\Form\EmbedButtonForm
gets the entity manager injected. But shortly before 8.0.0, that service was deprecated and split up into many more smaller services (the EntityManager
service was enormous/unusable). See https://www.drupal.org/node/2549139.
Proposed resolution
Inject only the necessary services split off from EntityManager
, rather than this enormous service.
Remaining tasks
Do it.
User interface changes
None.
API changes
None.
Data model changes
None.
Comment | File | Size | Author |
---|---|---|---|
#8 | embed-replace_entity_manager-2625764-8.patch | 3.1 KB | oenie |
#3 | embed-replace_entity_manager-2625764-3.patch | 2.71 KB | oenie |
Comments
Comment #2
slashrsm CreditAttribution: slashrsm as a volunteer commentedComment #3
oenie CreditAttribution: oenie as a volunteer commentedReplaced the EntityManager by the EntityTypeManager (only usage seemed to be getStorage())
Comment #8
oenie CreditAttribution: oenie as a volunteer commentedThis should fare a bit better ...
Comment #9
oenie CreditAttribution: oenie as a volunteer commentedComment #10
Wim LeersNot yet updated.
Can be fixed on commit.
Comment #11
slashrsm CreditAttribution: slashrsm at Examiner.com commentedCommitted. Thanks!