Problem/Motivation
CacheCollector doesn't detect changing cid after cid has been set. If the cid changes for some reason later on the processing the old cache key will be still used.
Proposed resolution
Re-factor CacheCollector to use a $this->storage[$cid] prefix for all internal storage like HookElementInfo() or implement a kind of SubscriberInterface so that a changing getCid() can be detected and handled.
Remaining tasks
-
User interface changes
-
API changes
-
Comments
Comment #1
fabianx commentedTagging
Comment #2
joelpittetThis doesn't seem to be Twig related. Removing tag. Also, since RC this may not be possible?
Should we move this to 8.1.x?
Comment #3
fabianx commentedYeah, no priority for 8.0.x
Comment #4
catchUnless there's a functional bug at the moment (in which case we need an issue referenced that this would fix/blocks), this looks like a task to me.
Comment #20
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
Comment #21
smustgrave commentedWanted to bump 1 more time if still valid? Still a major?