10000 Update doctrine.rst by sr972 · Pull Request #4053 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

Update doctrine.rst #4053

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Update doctrine.rst #4053

wants to merge 1 commit into from

Conversation

sr972
Copy link
@sr972 sr972 commented Jul 25, 2014
Q A
Doc fix? [yes]
New docs? [no]
Applies to [2.3+]

In 'Deleting an Object' you are not going to delete the entity itself but the given object.

Update doctrine.rst
b269ff0 
In 'Deleting an Object' you are not going to delete the entity itself but the given object.
@mtrojanowski
8000 Copy link
Contributor

The term entity in fact describes the unique object but maybe it's better to use the term object to avoid confusion that you will remove the whole table from the DB.

@sr972
Copy link
Author
sr972 commented Jul 25, 2014

@mtrojanowski This seems to be a good compromise in the definition and usage of the word entity in this section 👍

@wouterj
Copy link
Member
wouterj commented Jul 25, 2014

But since we said that the class is called an Entity at the start of this article, I'm +! for using object instead of entity here.

@weaverryan
Copy link
Member

I like it! Thanks @sr972!

@weaverryan weaverryan 65F2 closed this in 44091b1 Jul 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sr972 @mtrojanowski @wouterj @weaverryan
0