Skip to content
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

Fix initializing lazy properties during hydration #1355

Merged
merged 2 commits into from
Feb 8, 2016

Conversation

malarzm
Copy link
Member

@malarzm malarzm commented Jan 31, 2016

Fixes #1347

Marking as WIP as for some reason PHP segfaults on my local machine, let's see what Travis will say - travis is fine, my machine will need an investigation of what's going on

@malarzm malarzm added the Bug label Jan 31, 2016
@malarzm malarzm added this to the 1.0.5 milestone Jan 31, 2016
@malarzm malarzm changed the title [WIP] Fix initializing lazy properties Fix initializing lazy properties Jan 31, 2016
@malarzm malarzm changed the title Fix initializing lazy properties Fix initializing lazy properties during hydration Jan 31, 2016
@malarzm
Copy link
Member Author

malarzm commented Jan 31, 2016

@zajca test you provided is commited here and passes, but if you could test this also in your app it'd be great :)

@alcaeus
Copy link
Member

alcaeus commented Feb 8, 2016

Changes look good 👍

malarzm added a commit that referenced this pull request Feb 8, 2016
Fix initializing lazy properties during hydration
@malarzm malarzm merged commit 01b334a into doctrine:1.0.x Feb 8, 2016
@malarzm malarzm deleted the proxy-public-prop branch February 8, 2016 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants