Better form serialization: move #entity out

Created on 28 March 2013, about 12 years ago
Updated 21 March 2025, 12 days ago

Serialization of forms is a problem for #1863816: Allow plugins to have services injected β†’ so here's a patch that shifts entities (which are plugins) out of the recursive form array into the form_state arrays so that in a later step we can decide on how not to cache them (hardwire, move to form_state['no cache']['entity'], introduce form_state['no cache keys']). Small moves.

πŸ“Œ Task
Status

Postponed: needs info

Version

11.0 πŸ”₯

Component

forms system

Created by

πŸ‡¨πŸ‡¦Canada chx

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.

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.

  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    Just following up if still valid? If no follow up could close in 3 months

  • πŸ‡¨πŸ‡­Switzerland berdir Switzerland

    Unsure. form serialization has changed, we now basically only start to persist them on the first ajax request. So I think that basically makes this a non-issue unless those forms use ajax.

    Lets just close this unless someone has proof that this results in a measurable improvement?

Production build 0.71.5 2024