We should provide a method of DI for entities in order to avoid \Drupal calls in classes. This is especially pertinent now that we have bundle classes in core, where the only alternative to entity-specific services is \Drupal calls on the bundle class.
Entities have never been injected with services, because it would not have kept them serializable, a feature required by the REST module. However, if we make all entities extend \Drupal\Core\DependencyInjection\DependencySerialization that was introduced by
#2004282: Injected dependencies and serialization hell →
, services will no longer be a problem for serialization as only their IDs will be serialized.
Doing this will let us write cleaner production code, as we no longer need the wrappers that
#2134857: PHPUnit test the entity base classes →
introduces for testing, and our tests will no longer have to mock those wrapper methods.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Is this still actively being pursued or is there a workaround?
@drupals.user There haven't been any updates in 2+ years, so no, it isn't actively being updated.
As a workaround, you can use \Drupal calls without DI.