-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Doctrine ORM documentation example Product entity wrong annotation definition #7686
Comments
Hi @wajdijurry. The examples in the docs are accurate. Are you actually receiving an error for Also, where did you put the |
The Product entity example is using the annotation classes in the 2.6 docs without importing them with @wajdijurry Do you like to create a PR for the documentation of 2.6. |
Yes sure. |
Hi @SenseException , It seems that the online docs is for doctrine ORM v3.0, but it says to install doctrine ORM v2.6.2, so for which version should I open a new PR ? since it seems that I did not got your last comment :\ |
For the 2.6 branch because of the imported annotations. If there is 2.6 mentioned in the 3.0 docs, this should fixed too in a separate PR. |
But the bug is invalid for v2.6, since the annotation class is not imported and it is used without an alias |
Because in the examples of 2.6, the annotation classes, like |
Handled by #7753 |
Bug Report
Summary
Documentation should be updated, I think it's outdated.
Current behavior
When annotation parser tries to parse the entity annotations, it does not, since the classes (@Orm\Entity), (@Orm\Table), (@Orm\Id), (@Orm\Column) and (@Orm\GeneratedValue) are not defined.
How to reproduce
Follow the steps described in the documentation:
https://www.doctrine-project.org/projects/doctrine-orm/en/latest/tutorials/getting-started.html#what-is-doctrine
vendor/bin/doctrine orm:schema-tool:update --force --dump-sql
[OK] No Metadata Classes to process.
Expected behavior
Fix
@Entity
instead of@ORM\Entity
The text was updated successfully, but these errors were encountered: