Skip to content

Conversation

@mhsdesign
Copy link
Member

@mhsdesign mhsdesign commented Nov 2, 2025

in Neos 9 (ESCR) the workspace is no longer a fallback but a full copy of the base workspace content stream from a certain point. Any changes like base workspace change or publish or discard will lead to a full workspace cache flush (which is a new cache tag) There is no shine through from the base workspace and to explicitly request the new contents synchronise is uses (which is a RebaseWorkspaces and flushes the full workspace)

cleanup to #5221 which introduces Workspace content cache tags (\Neos\Neos\Fusion\Cache\CacheTag::forWorkspaceName())

Upgrade instructions

Review instructions

Checklist

  • Code follows the PSR-2 coding style
  • Tests have been created, run and adjusted as needed
  • The PR is created against the lowest maintained branch
  • Reviewer - PR Title is brief but complete and starts with FEATURE|TASK|BUGFIX
  • Reviewer - The first section explains the change briefly for change-logs
  • Reviewer - Breaking Changes are marked with !!! and have upgrade-instructions

…paceChain`

in Neos 9 (ESCR) the workspace is no longer a fallback but a full copy of the base workspace content stream from a certain point.
Any changes like base workspace change or publish or discard will lead to a full workspace cache flush (which is a new cache tag)
There is no shine through from the base workspace and to explicitly request the new contents synchronise is uses (which is a RebaseWorkspaces and flushes the full workspace)
* @param Node|null $node
* @return array<string,Workspace>
*/
public function getWorkspaceChain(?Node $node): array
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this helper method is removed for the minor release 9.1 i think its okay as a: the method is not documented and thus not as much api as say nodeTypeTag and b: its absolutely useless for caching in Neos 9 thats why we remove it ourselves as well. So if anyone upgrading to 9.1 finds a reference to that in his cache config it can just be removed.
If you disagree ill have that removed for 10.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant