minor #27249 [HttpKernel] do file_exists() check instead of silent notice (nicolas-grekas)

fabpot · fabpot · commit dd2c4c5a70cb · 2018-05-18T04:03:52.000+02:00
This PR was merged into the 3.4 branch. Discussion ---------- [HttpKernel] do file_exists() check instead of silent notice | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #27234 | License | MIT | Doc PR | - Commits ------- f8cde70 [HttpKernel] do file_exists() check instead of silent notice
diff --git a/src/Symfony/Component/HttpKernel/Kernel.php b/src/Symfony/Component/HttpKernel/Kernel.php
@@ -587,7 +587,7 @@ protected function initializeContainer()
             $errorLevel = error_reporting(\E_ALL ^ \E_WARNING);
             $fresh = $oldContainer = false;
             try {
-                if (\is_object($this->container = include $cache->getPath())) {
+                if (file_exists($cache->getPath()) && \is_object($this->container = include $cache->getPath())) {
                     $this->container->set('kernel', $this);
                     $oldContainer = $this->container;
                     $fresh = true;
@@ -650,7 +650,7 @@ protected function initializeContainer()
             }
         }
 
-        if (null === $oldContainer) {
+        if (null === $oldContainer && file_exists($cache->getPath())) {
             $errorLevel = error_reporting(\E_ALL ^ \E_WARNING);
             try {
                 $oldContainer = include $cache->getPath();
