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
- "Storables" and "entities" do not differ.
EntityFormController
expects anEntityInterface
, so aConfigurableInterface
is incompatible.- A "
StorableFormController
" is nonsense.
Goal
- Restore entity system sanity.
Proposed solution
- Revert
StorableInterface
toEntityInterface
. - Optional: Introduce a
ContentInterface
and use it forNode
,Comment
,Term
(but notUser
).
Alternate solution
- Typehint with
StorableInterface
Comments
Comment #1
tim.plunkettAdded an alternate solution.
Comment #2
sunWell, the bottom line is:
You may
But
I refuse to hack the entity system even more for a concept that doesn't exist in this universe.
If we need a ContentInterface, then we need to do that.
Comment #3
tim.plunkett#1761040: Rename Storable, Entity, and Configurable to Entity, ContentEntity, and ConfigEntity is a more more measured discussion of this, and it was posted a mere 7 minutes before this, so I'm closing this as a duplicate.
Comment #3.0
tim.plunkettadded alternate solution