Add a registration_host_type_id_registrations_list cache tag

Open on Drupal.org →

Created on 12 February 2025, about 2 months ago

Problem/Motivation

It would be simple to introduce a cache tag for the list of registrations for a host. We could then add that tag to HostEntity, the validators would pick it up automatically and not need to add the registration_list tag.

Then we would be able to use HostEntity::isAvailableForRegistration() in performance sensitive contexts instead of needing isOpenForRegistration().

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task

Status

Active

Version

3.3

Component

Registration Core

Created by

🇬🇧United Kingdom jonathanshaw Stroud, UK

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @jonathanshaw
Comment about 2 months ago →
🇺🇸United States john.oltman
You must have missed this which I recently took care of already:

https://www.drupal.org/project/registration/issues/3505538 📌 Implement a per host entity registration list cache tag Active

And no, we still need isOpenForRegistration. On a busy site, a host entity with lots of registration activity gives you the same performance problem if we use isAvailable - the problem is just limited to one host, but that could be the most important page on the site for a given day. We can't ever check for room from an access control handler, for any host.
Comment about 2 months ago →
🇬🇧United Kingdom jonathanshaw Stroud, UK
Nice!

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024