Skip to content
This repository has been archived by the owner on Feb 6, 2020. It is now read-only.

Allow proxymanager v2 installation #103

Conversation

Ocramius
Copy link
Member

This patch allows installation of ProxyManager v2.*

The changes are not breaking, but ProxyManager v2 has PHP 7 type-hints everywhere, which means that PHPUnit 5.* is required in order to mock them (specifically, PHPUnit 5.2.9+). This patch adds the required changes, as well as the polyfills required to have ProxyManager v2 behave like ProxyManager v1, configuration-wise

@Ocramius Ocramius added the bug label Mar 13, 2016
@@ -750,6 +752,10 @@ private function createLazyServiceDelegatorFactory()

if (! isset($this->lazyServices['write_proxy_files']) || ! $this->lazyServices['write_proxy_files']) {
$factoryConfig->setGeneratorStrategy(new EvaluatingGeneratorStrategy());
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This block becomes superfluous when ProxyManager v1 support is dropped (this is the default behavior now)

@Ocramius Ocramius force-pushed the feature/allow-proxymanager-v2-installation branch from 575e9c9 to ed4d4b1 Compare March 13, 2016 00:45
@sandrokeil
Copy link

@weierophinney Any reasons why this PR is not already merged. The lazy_services setting write_proxy_files has no effect with ProxyManager 2.0.

@weierophinney
Copy link
Member

@Ocramius Patch looks good to me; feel free to merge and tag a release.

@Ocramius Ocramius self-assigned this Jun 1, 2016
@Ocramius Ocramius modified the milestones: 3.1.0, 3.0.3 Jun 1, 2016
Ocramius added a commit that referenced this pull request Jun 1, 2016
@Ocramius
Copy link
Member Author

Ocramius commented Jun 1, 2016

Included into 3.1.0 via d27e109

@Ocramius Ocramius closed this Jun 1, 2016
@Ocramius Ocramius deleted the feature/allow-proxymanager-v2-installation branch June 1, 2016 13:13
Ocramius added a commit that referenced this pull request Jun 1, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants