PageRegion::loadForTheme can statically regions cache per-theme

Created on 6 August 2025, 2 days ago

Overview

During preview rendering, \Drupal\experience_builder\Entity\PageRegion::loadForTheme can be called multiple times via \Drupal\experience_builder\Entity\PageRegion::loadForActiveTheme

This causes multiple loadByProperties calls for the same page regions in a request.

Proposed resolution

Statically cache regions by theme in the method

User interface changes

✨ Feature request

Status

Active

Version

1.0

Component

… to be triaged

Created by

🇺🇸United States mglaman WI, USA

Live updates comments and jobs are added and updated live.

Merge Requests

!1416PageRegion::loadForTheme can statically regions cache per-theme
Open
🇺🇸United States mglaman
updated 1 day ago

Comments & Activities

Issue created by @mglaman
Merge request !1416Statically cache page regions per-theme when loading → (Open) created by mglaman
Comment 2 days ago →
🇺🇸United States mglaman WI, USA
Comment 2 days ago →
🇺🇸United States mglaman WI, USA
I wonder if opting into static_cache would have the same effect as done in ✨ Opt into static_cache for component config entities Active instead of this approach.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024