Skip to content

Using Model instead of Entity namespace #1134

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

Closed
wants to merge 1 commit into from
Closed

Using Model instead of Entity namespace #1134

wants to merge 1 commit into from

Conversation

msvrtan
Copy link

@msvrtan msvrtan commented May 25, 2013

If Model namespace is extended instead of Entity, DoctrineORM doesn't generate DB columns inherited from BaseUser class.

If Model namespace is extended instead of Entity, DoctrineORM doesn't generate DB columns inherited from BaseUser class.
@stof
Copy link
Member

stof commented May 25, 2013

The compiler pass is registering the mapping for the Model classes (there is no mapped column in the classes in Entity anymore)

@stof stof closed this May 25, 2013
@msvrtan
Copy link
Author

msvrtan commented May 25, 2013

user

If I use code with extending Model namespace in my DB I get only id column. When I extended User class from Entity I got all of defined properties from that class as columns when I run

php app/console doctrine:schema:update --force

I'am not sure what I'm doing wrong :(

@dbu
Copy link

dbu commented May 26, 2013

where you using the very latest version of FOSUserBundle, or just reading the latest documentation online but are using an older version of the bundle? you need a version after #1081 in your bundle or else the documentation is not correct.

@stof: should we add some sort of @since note to the doc to avoid this confusion?

@stof
Copy link
Member

stof commented May 28, 2013

@dbu IMO, no. the doc is versionned with the code. @since will make it harder to maintain.

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

Successfully merging this pull request may close these issues.

3 participants