- Issue created by @berdir
- Merge request !10798improve isPendingRevisionSupportEnabled implementation → (Open) created by berdir
- 🇨🇭Switzerland berdir Switzerland
Unsure what to do about the cache tags. We can depend on entity entity_bundles cache tag, because they get invalidated on any workflow change. But that implies that we know about the implementation of the content moderation API that we're using.
- 🇨🇭Switzerland berdir Switzerland
Ok, this has a problem because it already needs this information in hook_entity_bundle_info_alter(). And we can't run after content_moderation, because the alter hook in content_moderation in turn depends on the translatable flag to be set.
Added the direct check using workflow config entities for this case.
- 🇧🇪Belgium kristiaanvandeneynde Antwerp, Belgium
Replace with a workflow_list cache tag. That said, this alone won't help, since the implementation then again loads all workflow entities.
Why would adding a list cache tag load all entities?
In case you're worried there are too many workflows that are not related to content moderation, we could introduce a few custom list cache tags in workflows so you can have something like: workflow_list:type:content_moderation
- 🇨🇭Switzerland berdir Switzerland
It used to, but the new implementation no longer needs to load.