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

[DDC-3467] Embedded on private fields in an inheritance don't work correctly #1234

Open
wants to merge 1 commit into
base: old-prototype-3.x
Choose a base branch
from

Conversation

Nemo64
Copy link

@Nemo64 Nemo64 commented Dec 30, 2014

The tried creating a test case for DDC-3467. I hope I put everything in the right place as my knowledge of this project is currently limited.

@doctrinebot
Copy link

Hello,

thank you for creating this pull request. I have automatically opened an issue
on our Jira Bug Tracker for you. See the issue link:

http://www.doctrine-project.org/jira/browse/DDC-3469

We use Jira to track the state of pull requests and the versions they got
included in.

@@ -0,0 +1,23 @@
<?php
/**
Copy link

Choose a reason for hiding this comment

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

You could probably clean it up a little.

@till
Copy link

till commented Jan 9, 2015

@Nemo64 is the idea not to see an exception?

@till
Copy link

till commented Jan 9, 2015

Maybe add a couple assertions on the actual metadata?

@Ocramius Ocramius changed the title added basic test for private embedded field in inheritence [DDC-3467] [Test] added basic test for private embedded field in inheritence Jan 17, 2015
@Ocramius Ocramius changed the title [DDC-3467] [Test] added basic test for private embedded field in inheritence [DDC-3467] Added basic test for private embedded field in inheritence Jan 24, 2015
@Ocramius Ocramius changed the title [DDC-3467] Added basic test for private embedded field in inheritence [DDC-3467] Embedded on private fields in an inheritance don't work correctly Jan 24, 2015
@linaori
Copy link

linaori commented Sep 25, 2015

I ran into this issue today, @till the bug is that the variable is not accessible while it isn't in the child class. The embeddable should be in the parent and not the child.

I can temporarily work around this by making it protected, but I'd rather avoid that.

@lcobucci
Copy link
Member

@Ocramius I think this was handled on #5867

Base automatically changed from master to old-prototype-3.x February 23, 2021 08:18
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.

None yet

6 participants