ProxyManager bugfix

[blanchonvincent]

No description provided.

[blanchonvincent]

@Ocramius have a look please

[Ocramius]

@blanchonvincent this still dumps all the files to disk (see the EvaluatingGeneratorStrategy bit) - any way of allowing that too? :)

[blanchonvincent]

@Ocramius Hmm I think I don't understand. If you use "write_proxy_files => false", you'll use the EvaluatingGeneratorStrategy ?

[Ocramius]

Yes basically :-)

[blanchonvincent]

@Ocramius and I made a mistake ? Sorry Marco, I think I don't where I wrong in this PR :/

[weierophinney]

@blanchonvincent Can you add a description to the PR indicating the problem you encountered, and how you're attempting to solve it, please? When I look at the changes, I have doubts about what you're trying to accomplish, and whether the approach is sound... but mostly due to the fact that I'm unclear what the purpose is.

[Ocramius]

@blanchonvincent may I also suggest a dev dependency bump? :)

[blanchonvincent]

@weierophinney I added a fail test case : https://github.com/blanchonvincent/zf2/blob/d08395942f275d32cf2af308d1b9b2044ca4c2c3/tests/ZendTest/ServiceManager/Proxy/LazyServiceFactoryFactoryTest.php#L82

This has been fixed in ProxyManager project : https://github.com/Ocramius/ProxyManager/pull/87/files

The problem is when auto_generate_proxies is equal to true, because if the proxy was never generated, it will be never generate.

[blanchonvincent]

@Ocramius what is a dev dependency bump ? a dependence from the ProxyManager project ?

[Ocramius]

Yep, basically using 0.5.x for running the tests

[blanchonvincent]

@Ocramius Yep, agree if we can

[weierophinney]

@blanchonvincent and @Ocramius -- any news on completing this? or should I close?

[blanchonvincent]

This PR fix a bug (see the fail test case), so maybe can we merge ? cc @Ocramius

[Maks3w]

@Ocramius ping

[Ocramius]

This can be closed - I will provide another PR in some minutes (to develop)

[Maks3w]

@Ocramius Thanks. Alert us when PR become public before :)

[Ocramius]

@Maks3w provided #5858 (2.3 only, but can be fitted for 2.2 as well)