Make SqlContentEntityStorage table mapping agnostic

Created on 25 May 2014, over 10 years ago
Updated 24 May 2023, over 1 year ago

Follow-up to #2183231: Make ContentEntityDatabaseStorage generate static database schemas for content entities :

Problem/Motivation

Right, now both the SqlContentEntityStorage and the schema handler have to work together based on a known table mapping, what makes the line between both object blurry.

Proposed resolution

- Make the SqlContentEntityStorage table mapping agnostic and get the table mapping from the schema handler
- As suggested by yched in #2183231: Make ContentEntityDatabaseStorage generate static database schemas for content entities we could create multiple table mapping classes for the different table mapping cases also.

Remaining tasks

- Agree on the exact way forward and do it.

User interface changes

-

API changes

None, only some base classes change a little

📌 Task
Status

Needs work

Version

11.0 🔥

Component
Entity 

Last updated 7 minutes ago

Created by

🇦🇹Austria fago Vienna

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

  • Needs reroll

    The patch will have to be re-rolled with new suggestions/changes described in the comments in the issue.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024