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.
Another child of #2886622: Deprecate all EntityManager methods with E_USER_DEPRECATED, now that all the non entity manager methods are done, this addressed the last remaining injected instances of entity manager -> entity type manager.
This is on top of #3035953: Add @trigger_error() to deprecated EntityManager->EntityFieldManager methods, so not sure if it will apply.
Comment | File | Size | Author |
---|---|---|---|
#2 | entity-manager-constructor-3042545-2.patch | 89.01 KB | Berdir |
entity-manager-constructor.patch | 84.35 KB | Berdir | |
Comments
Comment #2
Berdir... not sure what happend with that patch.
Comment #3
martin107 CreditAttribution: martin107 as a volunteer commentedWow almost a 90k patch
After a slow visual scan ...
a) DeprecatedServicePropertyTrait everywhere I expected
b) all @param definitions updated -- consistently.
c) patch contains only changes aimed at solving the problem
d) no extra coding standard changes lurking in the test results.
FWIW I agree with the idea behind this patch... this is a good cleanup.
Comment #4
andypost+1 to get this in while beta
Comment #5
andypostComment #6
larowlanCommitted 12d7ed5 and pushed to 8.8.x. Thanks!