Problem/Motivation

  1. Certain cache bins are using the ChainedFastBackend, which isn't fully working with the current override of only "simple" cache backends. This concerns mainly the discovery cache bin, which we need to override to deal with asset_injector config for example.
  2. The config.storage service needs to redirect all calls to the already implemented WseCacheBackend because there's notices like the one below otherwise if the core service gets called, as that service isn't aware of workspace spefific cache ids.

Steps to reproduce

  • Enable wse_config and asset_injector
  • Create a new workspace
  • Switch to the new workspace
  • Add a new CSS asset_injector config and click on "Save And Continue Editing"
  • Make CSS changes inside the above created asset_injector config and click on "Save And Continue Editing"
  • After repeating the step above a few times (amount of times differs), the changed asset injector's CSS won't get applied to the page anymore

Proposed resolution

  1. Override config.storage to redirect all calls to this class to WseCacheBackend.
  2. Override cache.backend.chainedfast and extend the ChainedFastBackend to disable caching in the fast backend inside workspaces.
CommentFileSizeAuthor
image (5).png1.07 MBs_leu

Issue fork wse-3275422

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

s_leu created an issue. See original summary.

amateescu’s picture

Status: Active » Fixed

Merged into 1.0.x, thanks!

Status: Fixed » Closed (fixed)

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

amateescu’s picture

Component: Code » Workspace Config