diff --git a/core/lib/Drupal/Core/Cache/Context/SessionCacheContext.php b/core/lib/Drupal/Core/Cache/Context/SessionCacheContext.php index c8b102717d53ebf42b9ba791032e75582778663b..bccd0f01a4690d6f2bc5136820c8612ac6d0e145 100644 --- a/core/lib/Drupal/Core/Cache/Context/SessionCacheContext.php +++ b/core/lib/Drupal/Core/Cache/Context/SessionCacheContext.php @@ -22,8 +22,11 @@ public static function getLabel() { * {@inheritdoc} */ public function getContext() { - $sid = $this->requestStack->getCurrentRequest()->getSession()->getId(); - return Crypt::hashBase64($sid); + $request = $this->requestStack->getCurrentRequest(); + if ($request->hasSession()) { + return Crypt::hashBase64($request->getSession()->getId()); + } + return 'none'; } } diff --git a/core/tests/Drupal/Tests/Core/Cache/Context/SessionCacheContextTest.php b/core/tests/Drupal/Tests/Core/Cache/Context/SessionCacheContextTest.php index 9538bd2511268d068d530dbc91589a10346c7532..a8cf6fb3dc4d7d63374cde935d5b22df62775dff 100644 --- a/core/tests/Drupal/Tests/Core/Cache/Context/SessionCacheContextTest.php +++ b/core/tests/Drupal/Tests/Core/Cache/Context/SessionCacheContextTest.php @@ -13,6 +13,13 @@ */ class SessionCacheContextTest extends UnitTestCase { + /** + * The request. + * + * @var \Symfony\Component\HttpFoundation\Request + */ + protected $request; + /** * The request stack. * @@ -27,36 +34,30 @@ class SessionCacheContextTest extends UnitTestCase { */ protected $session; - /** - * The session cache context. - * - * @var \Drupal\Core\Cache\Context\SessionCacheContext - */ - protected $cacheContext; - public function setUp() { - $request = new Request(); + $this->request = new Request(); $this->requestStack = new RequestStack(); - $this->requestStack->push($request); - - $this->session = $this->getMock('\Symfony\Component\HttpFoundation\Session\SessionInterface'); - $request->setSession($this->session); + $this->requestStack->push($this->request); - $this->cacheContext = new SessionCacheContext($this->requestStack); + $this->session = $this->getMockBuilder('\Symfony\Component\HttpFoundation\Session\SessionInterface') + ->getMock(); } /** * @covers ::getContext */ public function testSameContextForSameSession() { + $this->request->setSession($this->session); + $cache_context = new SessionCacheContext($this->requestStack); + $session_id = 'aSebeZ52bbM6SvADurQP89SFnEpxY6j8'; $this->session->expects($this->exactly(2)) ->method('getId') ->will($this->returnValue($session_id)); - $context1 = $this->cacheContext->getContext(); - $context2 = $this->cacheContext->getContext(); + $context1 = $cache_context->getContext(); + $context2 = $cache_context->getContext(); $this->assertSame($context1, $context2); $this->assertSame(FALSE, strpos($context1, $session_id), 'Session ID not contained in cache context'); } @@ -65,6 +66,9 @@ public function testSameContextForSameSession() { * @covers ::getContext */ public function testDifferentContextForDifferentSession() { + $this->request->setSession($this->session); + $cache_context = new SessionCacheContext($this->requestStack); + $session1_id = 'pjH_8aSoofyCDQiuVYXJcbfyr-CPtkUY'; $this->session->expects($this->at(0)) ->method('getId') @@ -75,12 +79,21 @@ public function testDifferentContextForDifferentSession() { ->method('getId') ->will($this->returnValue($session2_id)); - $context1 = $this->cacheContext->getContext(); - $context2 = $this->cacheContext->getContext(); + $context1 = $cache_context->getContext(); + $context2 = $cache_context->getContext(); $this->assertNotEquals($context1, $context2); $this->assertSame(FALSE, strpos($context1, $session1_id), 'Session ID not contained in cache context'); $this->assertSame(FALSE, strpos($context2, $session2_id), 'Session ID not contained in cache context'); } + /** + * @covers ::getContext + */ + public function testContextWithoutSessionInRequest() { + $cache_context = new SessionCacheContext($this->requestStack); + + $this->assertSame('none', $cache_context->getContext()); + } + }