Explicitly set FileWriterGeneratorStrategy (#412)

* Explicitly set FileWriterGeneratorStrategy It defaults to the EvaluatingGeneratorStrategy * Explicitly use EvaluatingGeneratorStrategy when not writing proxies * Allow ProxyManager ~2.0 to be used
PHP-DI · May 29, 2016 · dd22e25 · dd22e25
1 parent ad0c4e0
commit dd22e25
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/doc/lazy-injection.md b/doc/lazy-injection.md
@@ -86,7 +86,7 @@ Lazy injection requires the [Ocramius/ProxyManager](https://github.com/Ocramius/
 {
     "require": {
         "php-di/php-di": "*",
-        "ocramius/proxy-manager": "~1.0"
+        "ocramius/proxy-manager": "~1.0 | ~2.0"
     }
 }
 ```

diff --git a/src/DI/Proxy/ProxyFactory.php b/src/DI/Proxy/ProxyFactory.php
@@ -4,7 +4,9 @@
 
 use ProxyManager\Configuration;
 use ProxyManager\Factory\LazyLoadingValueHolderFactory;
+use ProxyManager\FileLocator\FileLocator;
 use ProxyManager\GeneratorStrategy\EvaluatingGeneratorStrategy;
+use ProxyManager\GeneratorStrategy\FileWriterGeneratorStrategy;
 
 /**
  * Creates proxy classes.
@@ -71,6 +73,7 @@ private function createProxyManager()
 
         if ($this->writeProxiesToFile) {
             $config->setProxiesTargetDir($this->proxyDirectory);
+            $config->setGeneratorStrategy(new FileWriterGeneratorStrategy(new FileLocator($this->proxyDirectory)));
             spl_autoload_register($config->getProxyAutoloader());
         } else {
             $config->setGeneratorStrategy(new EvaluatingGeneratorStrategy());