Fix hidden bug when entity properties are updated via ConfigEntityBase::set()

Created on 30 September 2025, 4 days ago

As described in #3542808-5: Add variants of plugin getters to our entity classes that don’t throw exceptions , there is a hidden bug where updating properties of our index/server entities via their set() methods (inherited from ConfigEntityBase) could leave them in an inconsistent state, as that method won’t properly reset the corresponding cached plugins or server entity (e.g., $this->processorInstances when setting $this->processor_settings).

Unfortunately, the set() method is used by EntityForm so it is more a coincidence that this hasn’t led to any actual bugs in the module so far. (There could be such bugs in related modules, though.)

We should make sure that changing properties via set() never leaves the entity in an inconsistent state.

🐛 Bug report
Status

Needs review

Version

1.0

Component

General code

Created by

🇦🇹Austria drunken monkey Vienna, Austria

Live updates comments and jobs are added and updated live.
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.

No activities found.

Production build 0.71.5 2024