Don't serialize computed fields

Created on 28 October 2023, over 1 year ago
Updated 2 November 2023, over 1 year ago

Problem/Motivation

Found in πŸ“Œ Make POST requests render cacheable Needs work see long explanation over there.

ContentEntityBase serializes all field values. We don't need to serialize computed field values, and some of them are expensive or risky to populate, like path aliases.

Steps to reproduce

Proposed resolution

Add ComputedFieldItemListInterface and check for it in ContentEntityBase::__sleep()

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Needs work

Version

11.0 πŸ”₯

Component
EntityΒ  β†’

Last updated 1 day ago

Created by

πŸ‡¬πŸ‡§United Kingdom catch

Live updates comments and jobs are added and updated live.
  • Performance

    It affects performance. It is often combined with the Needs profiling tag.

Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024