Turns out we missed removing the storable-StorageControllerInterface in #1761040: Rename Storable, Entity, and Configurable to Entity, ContentEntity, and ConfigEntity. Attached patch does so.

CommentFileSizeAuthor
#3 entity-remove-storagecontrollerinterface.patch3.22 KBeffulgentsia
d8_entity_storage_interface.patch10.45 KBfago
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Entity system, +Configurables

d'oh, yeah, that makes perfect sense.

IIRC, I left it out, because the shorter class name without "Entity" prefix looked more sensible and "appealing" ;)

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

effulgentsia’s picture

effulgentsia CreditAttribution: effulgentsia commented
Status: Fixed » Needs review
FileSize
entity-remove-storagecontrollerinterface.patch3.22 KB

Original patch missed the "remove" part. Here's that.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.