Skip to content

Fix - #17676 (backport #17919 to 2.3) #17947

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 28, 2016

Conversation

Ocramius
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #17676
License MIT
Doc PR

This is a backport of #17919

"ocramius/proxy-manager": "~0.3.1|~1.0|~2.0",
"phpdocumentor/reflection": "^1.0.7"
},
"conflict": {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Too much stuff slipped in here. Need to get rid of a few of those.

@@ -11,6 +11,7 @@

namespace Symfony\Bridge\ProxyManager\Tests\LazyProxy\Dumper;

use ProxyManager\ProxyGenerator\LazyLoading\MethodGenerator\StaticProxyConstructor;
Copy link
Member

Choose a reason for hiding this comment

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

This is not needed.

@Ocramius Ocramius force-pushed the fix/#17676-backport-#17919-to-2.3 branch from 160a03e to 0c6400a Compare February 28, 2016 15:53
@fabpot
Copy link
Member

fabpot commented Feb 28, 2016

Thank you @Ocramius.

@fabpot fabpot merged commit 0c6400a into symfony:2.3 Feb 28, 2016
fabpot added a commit that referenced this pull request Feb 28, 2016
This PR was merged into the 2.3 branch.

Discussion
----------

Fix - #17676 (backport #17919 to 2.3)

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #17676
| License       | MIT
| Doc PR        |

This is a backport of #17919

Commits
-------

0c6400a  #17676 - making the proxy instantiation compatible with ProxyManager 2.x by detecting proxy features
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants