[HttpFoundation] fixed the check of 'proxy-revalidate' in Response::mustRevalidate()

axiac · stof · commit 6c22f0af24de · 2015-08-03T19:22:14.000+02:00
diff --git a/src/Symfony/Component/HttpFoundation/Response.php b/src/Symfony/Component/HttpFoundation/Response.php
@@ -587,7 +587,7 @@ public function setPublic()
      */
     public function mustRevalidate()
     {
-        return $this->headers->hasCacheControlDirective('must-revalidate') || $this->headers->has('proxy-revalidate');
+        return $this->headers->hasCacheControlDirective('must-revalidate') || $this->headers->hasCacheControlDirective('proxy-revalidate');
     }
 
     /**
diff --git a/src/Symfony/Component/HttpFoundation/Tests/ResponseTest.php b/src/Symfony/Component/HttpFoundation/Tests/ResponseTest.php
@@ -92,6 +92,22 @@ public function testMustRevalidate()
         $this->assertFalse($response->mustRevalidate());
     }
 
+    public function testMustRevalidateWithMustRevalidateCacheControlHeader()
+    {
+        $response = new Response();
+        $response->headers->set('cache-control', 'must-revalidate');
+
+        $this->assertTrue($response->mustRevalidate());
+    }
+
+    public function testMustRevalidateWithProxyRevalidateCacheControlHeader()
+    {
+        $response = new Response();
+        $response->headers->set('cache-control', 'proxy-revalidate');
+
+        $this->assertTrue($response->mustRevalidate());
+    }
+
     public function testSetNotModified()
     {
         $response = new Response();

Original file line number	Diff line number	Diff line change
`@@ -587,7 +587,7 @@ public function setPublic()`
`587`	`587`	`*/`
`588`	`588`	`public function mustRevalidate()`
`589`	`589`	`{`
`590`		`- return $this->headers->hasCacheControlDirective('must-revalidate') \|\| $this->headers->has('proxy-revalidate');`
	`590`	`+ return $this->headers->hasCacheControlDirective('must-revalidate') \|\| $this->headers->hasCacheControlDirective('proxy-revalidate');`
`591`	`591`	`}`
`592`	`592`
`593`	`593`	`/**`