Prevent loss of layout changes when breaking lock in different workspace

Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to the Core change policies → .

I don't think we should take the route of providing a generic workspace-aware temp store implementation, we have #3026957: Provide a generic TempStoreIdentifierInterface that any object can use to better identify itself → for that, so let's keep this issue focused on the specific problem we have: LB's temp store needs to be workspace-aware.

Let's decorate the layout_builder.tempstore_repository service with a class that extends LayoutTempstoreRepository, and overrides its getKey() method to append the current workspace ID. Then test coverage needs to be added in \Drupal\Tests\workspaces\FunctionalJavascript\WorkspacesLayoutBuilderIntegrationTest.

Looks ready to me!

This is tricky but the solution makes sense. I wondered about whether we could add some kind of 'extra keys' container parameter that layout builder could respect, and workspaces could add to, a bit like required render cache contexts, but that feels like it should be its own issue if at all.

Committed/pushed to 11.x, thanks!

catch → closed merge request !10630

Automatically closed - issue fixed for 2 weeks with no activity.