make sure PHPStan cache directory is configurable and created

rectorphp · Aug 17, 2023 · 8b3c539 · 8b3c539
1 parent 4637080
commit 8b3c539
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/packages/Config/RectorConfig.php b/packages/Config/RectorConfig.php
@@ -280,7 +280,7 @@ public function cacheDirectory(string $directoryPath): void
     }
 
     /**
-     * @info Rector no longer user compiled container, but Laravel lazy one.
+     * @info Rector no longer uses compiled container, but Laravel lazy one.
      * This option is for PHPStan container in case the sys_get_temp_dir() is not available for the current user.
      */
     public function containerCacheDirectory(string $directoryPath): void

diff --git a/packages/NodeTypeResolver/DependencyInjection/PHPStanServicesFactory.php b/packages/NodeTypeResolver/DependencyInjection/PHPStanServicesFactory.php
@@ -47,12 +47,14 @@ public function __construct(
             $purifiedConfigFiles[] = $purifiedConfigFile;
         }
 
+        $cacheDirectory = SimpleParameterProvider::provideStringParameter(Option::CACHE_DIR);
+        $phpstanCacheDirectory = $cacheDirectory . '/phpstan-container-for-rector';
+
+        // make sure directory exists as PHPStan Nette container will not create it and crash
+        \Nette\Utils\FileSystem::createDir($phpstanCacheDirectory);
+
         $containerFactory = new ContainerFactory(getcwd());
-        $this->container = $containerFactory->create(
-            sys_get_temp_dir() . '/phpstan-container-for-rector',
-            $additionalConfigFiles,
-            []
-        );
+        $this->container = $containerFactory->create($phpstanCacheDirectory, $additionalConfigFiles, []);
 
         // clear temporary files, after container is created
         $filesystem = new Filesystem();