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

Update association-mapping.rst to ease understanding of what associations mean #5948

Merged
merged 2 commits into from
Nov 26, 2016
Merged

Conversation

michsk
Copy link

@michsk michsk commented Jul 21, 2016

Added pointers on how to interpret the method names. This helped me to quicker grasp the concept of the Relations and there methods. Added a @var annotation to the Collection interface to make it clear that the $groups is a child of the Collection interface.

Added pointers on how to interpret the method names. This helped me to quicker grasp the concept of the Relations and there methods. Added a @var annotation to the `Collection` interface to make it clear that the `$groups` is a child of the `Collection` interface.
@michsk
Copy link
Author

michsk commented Jul 21, 2016

related: #5946 (comment)

@Ocramius Ocramius changed the title Update association-mapping.rst Update association-mapping.rst to ease understanding of what associations mean Jul 21, 2016
@@ -16,6 +16,8 @@ This chapter is split into three different sections.
- :ref:`association_mapping_defaults` are explained that simplify the use-case examples.
- :ref:`collections` are introduced that contain entities in associations.

One tip for working with relations is to read the relation from left to right, where the left word referece to the current class.
Copy link
Member

Choose a reason for hiding this comment

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

Can you add some wording examples here? Such as "OneToMany" means "One" (this side), to "Many" (referenced side).

Added spoken word examples of the relationship methods
@Ocramius
Copy link
Member

Nice improvement over here, thanks a lot!

@Ocramius Ocramius self-assigned this Nov 26, 2016
@Ocramius Ocramius added this to the 2.6.0 milestone Nov 26, 2016
@Ocramius Ocramius merged commit f116973 into doctrine:master Nov 26, 2016
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.

2 participants