8000 Changed ORM default naming strategy to underscore by hason · Pull Request #813 · symfony/symfony-standard · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

Changed ORM default naming strategy to underscore #813

Merged
merged 1 commit into from
May 22, 2015
Merged

Conversation

hason
Copy link
Contributor
@hason hason commented May 22, 2015

This PR partially fixes the problem with case-sensitive identifiers http://dev.mysql.com/doc/refman/5.0/en/identifier-case-sensitivity.html

Replaces doctrine/DoctrineBundle#406

@Pierstoval
Copy link
Contributor

I think that this can be very useful 👍

@hacfi
Copy link
hacfi commented May 22, 2015

👍

1 similar comment
@TomasVotruba
Copy link

👍

@stof
Copy link
Member
stof commented May 22, 2015

👍 The default value cannot be changed in DoctrineBundle because it would be a BC break for existing projects, but the underscore is a much saner choice than the default naming strategy keeping camel-cased names. It has much less compatibility issues across platforms.

@fabpot
Copy link
Member
fabpot commented May 22, 2015

Thank you @hason.

@fabpot fabpot merged commit b712c39 into symfony:2.7 May 22, 2015
fabpot added a commit that referenced this pull request May 22, 2015
This PR was merged into the 2.7 branch.

Discussion
----------

Changed ORM default naming strategy to underscore

This PR partially fixes the problem with case-sensitive identifiers http://dev.mysql.com/doc/refman/5.0/en/identifier-case-sensitivity.html

Replaces doctrine/DoctrineBundle#406

Commits
-------

b712c39 Changed ORM default naming strategy to underscore
@weaverryan
Copy link
Member

👍 Wish we had done this awhile ago :)

@stof
Copy link
Member
stof commented May 22, 2015

@weaverryan better late than never

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants
0