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

Remove note on findBy not supporting entity #5771

Merged
merged 1 commit into from Apr 12, 2016
Merged

Conversation

@teohhanhui
Copy link
Contributor

teohhanhui commented Apr 12, 2016

No description provided.

@@ -699,8 +699,6 @@ You can also load by owning side associations through the repository:
$number = $em->find('MyProject\Domain\Phonenumber', 1234);
$user = $em->getRepository('MyProject\Domain\User')->findOneBy(array('phone' => $number->getId()));
Be careful that this only works by passing the ID of the associated entity, not yet by passing the associated entity itself.

This comment has been minimized.

Copy link
@teohhanhui

teohhanhui Apr 12, 2016

Author Contributor

Do we need to note since which version?

This comment has been minimized.

Copy link
@Ocramius

Ocramius Apr 12, 2016

Member

Not relevant: docs are versioned too, although only latest are deployed.

@Ocramius Ocramius added this to the 2.6.0 milestone Apr 12, 2016
@Ocramius Ocramius self-assigned this Apr 12, 2016
@Ocramius Ocramius merged commit 76a5229 into doctrine:master Apr 12, 2016
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Scrutinizer 2 new issues
Details
@Ocramius

This comment has been minimized.

Copy link
Member

Ocramius commented Apr 12, 2016

👍

@teohhanhui teohhanhui deleted the teohhanhui:patch-1 branch Apr 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.