Merge pull request #434 from mnocon/fix-mkdir-warnings

tobias-93 · web-flow · commit 8c39c9e5b374 · 2022-06-30T21:51:11.000+02:00
Changed ExposedRoutesExtractor to handle mkdir warnings
diff --git a/Extractor/ExposedRoutesExtractor.php b/Extractor/ExposedRoutesExtractor.php
@@ -147,7 +147,9 @@ public function getCachePath(string $locale = null): string
     {
         $cachePath = $this->cacheDir.DIRECTORY_SEPARATOR.'fosJsRouting';
         if (!file_exists($cachePath)) {
-            mkdir($cachePath);
+            if (false === @mkdir($cachePath)) {
+                throw new \RuntimeException('Unable to create Cache directory ' . $cachePath);
+            }
         }
 
         if (isset($this->bundles['JMSI18nRoutingBundle'])) {

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,9 @@ public function getCachePath(string $locale = null): string`
`147`	`147`	`{`
`148`	`148`	`$cachePath = $this->cacheDir.DIRECTORY_SEPARATOR.'fosJsRouting';`
`149`	`149`	`if (!file_exists($cachePath)) {`
`150`		`- mkdir($cachePath);`
	`150`	`+ if (false === @mkdir($cachePath)) {`
	`151`	`+ throw new \RuntimeException('Unable to create Cache directory ' . $cachePath);`
	`152`	`+ }`
`151`	`153`	`}`
`152`	`154`
`153`	`155`	`if (isset($this->bundles['JMSI18nRoutingBundle'])) {`