Skip to content
New issue

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

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

Already on GitHub? # to your account

Exclude embedded documents from proxy generation #430

Merged

Conversation

alcaeus
Copy link
Member

@alcaeus alcaeus commented Sep 22, 2017

Related to doctrine/mongodb-odm#1628. The ProxyCacheWarmer still generates proxies for embedded documents, which causes errors if these are final and doctrine/common 2.8 is used.

@malarzm merging this down to master would force us to tag 3.4.0 since we've got at least one new feature (sharding command) in master. If you think we should fix this in 3.3, I'll create a support branch and backport the fix accordingly.

@alcaeus alcaeus requested a review from malarzm September 22, 2017 11:23
@alcaeus alcaeus added the bug label Sep 22, 2017
Copy link
Member

@malarzm malarzm left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@alcaeus alcaeus added this to the 3.4.0 milestone Sep 22, 2017
@alcaeus alcaeus merged commit 24620be into doctrine:master Sep 22, 2017
@alcaeus alcaeus deleted the fix-embedded-document-proxy-generation branch September 22, 2017 14:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants