- Issue created by @larowlan
- π¨π¦Canada Charlie ChX Negyesi πCanada
larowlan β credited Ghost of Drupal Past β .
- @ghost-of-drupal-past opened merge request.
- π«π·France andypost
Looks like duplicate of π SessionCacheContext class should implement CacheContextInterface Needs work
- π¨π¦Canada Charlie ChX Negyesi πCanada
The two should be merged into one, I am not partial which stays alive. That one has a test (yay!) but this has a bugfix of some importance:
if (!$this->requestStack->getSession()->getId()) { $metadata->setCacheMaxAge(0); }
- @ghost-of-drupal-past opened merge request.
- @ghost-of-drupal-past opened merge request.
- π¨π¦Canada Charlie ChX Negyesi πCanada
Wrong issue, please disregard. This newfangled process completely confuses me. Patches were way easier.
- π«π·France prudloff Lille
prudloff β changed the visibility of the branch drupal-3442009 to hidden.
- π«π·France prudloff Lille
prudloff β changed the visibility of the branch 3442009-oop-hooks-using to hidden.
- π«π·France prudloff Lille
I suppose we should merge π SessionCacheContext class should implement CacheContextInterface Needs work first then this issue can focus on adding the 0 max age fix when there is no session.