- Issue created by @kristiaanvandeneynde
- Merge request !11462Don't load individual placeholders from cache → (Open) created by kristiaanvandeneynde
- 🇧🇪Belgium kristiaanvandeneynde Antwerp, Belgium
Will fail tests, but that's the idea. If it's only performance tests that fail, we have no collateral damage.
- 🇬🇧United Kingdom catch
One performance test to update, but otherwise looks like it should be green. Not entirely sure why there are more cache gets in this case.
- 🇨🇭Switzerland berdir Switzerland
Asserted the render bin operations, but not sure I fully understand this.
--- Expected +++ Actual @@ @@ 16 => 'entity_view:block:stark_site_...-admin' 17 => 'entity_view:block:stark_syndi...-admin' 18 => 'navigation:navigation:[langua...-admin' - 19 => 'navigation_user_block:[languages:language_interface]=en:[theme]=stark:[user.permissions]=is-admin, shortcut_set_navigation_links:[languages:language_interface]=en:[theme]=stark:[user]=2' + 19 => 'navigation_user_block:[languages:language_interface]=en:[theme]=stark:[user.permissions]=is-admin' 20 => 'navigation_user_block:[langua...ser]=2' - 21 => 'view:frontpage:display:page_1:[languages:language_interface]=en:[theme]=stark:[url.query_args]=:[url.site]=http://core.ddev.site:[user.node_grants:view]=all:[user.permissions]=is-admin' - 22 => 'view:frontpage:display:page_1:[languages:language_interface]=en:[theme]=stark:[user.permissions]=is-admin' + 21 => 'shortcut_set_navigation_links:[languages:language_interface]=en:[theme]=stark:[user]=2' + 22 => 'view:frontpage:display:page_1:[languages:language_interface]=en:[theme]=stark:[url.query_args]=:[url.site]=http://core.ddev.site:[user.node_grants:view]=all:[user.permissions]=is-admin' + 23 => 'view:frontpage:display:page_1...-admin' )
something about the shortcut placeholder in the navigation, I think it's being fetched separately now, but I'm not sure why the CachedStrategy isn't picking it up with the other one? maybe due to the user cache context somehow?