[DDC-1379] Entity Generator Bug with extended classes Created: 16/Sep/11 Updated: 18/Aug/14 Resolved: 09/Jan/12
|Project:||Doctrine 2 - ORM|
|Reporter:||Mark Badolato||Assignee:||Benjamin Eberlei|
I'm presently using Doctrine 2 in conjunction with Symfony 2 and have come across a bug with the Entity Generator when I'm extending an abstract class. Here's a minor example to demonstrate (please note, any annotations problems or typos are a result of my typing in this ticket. This isn't an actual example, but does accurately show how to reproduce the issue)
Using a setup similar to the above, where Bar extends Foo, there is an issue with the Generator.
1) The generator is redeclaring Foo's properties, as private variables, in Bar
So, opening Bar after running the generator, shows the getters and setters, but shows:
This causes issues because the properties are being recreated as local versions, and their scope is set to private, conflicted with the more open protected version in the abstract class. To me, they shouldn't be placed in Bar because they're declared as protected in Foo and thus Bar has access to them. Also, when they are placed in Bar, the file formatting is messed up around them, suggesting that they were mis-placed in the file.
2) This may or may not be a bug (it may be intended behavior), but... I would think that when running the generator, the getters and setters for the protected properties declared in Foo would be placed in Foo, and not placed in the class(es) that extend Foo (in this case, Bar). If I also have a class Baz that extends Foo, then Bar and Baz will now have copies of the same getFirstName(), getLastName(), and getIsActive() methods, which is unneeded. They can be placed in the parent class, and overridden in the subclasses if more/different functionality/behavior is needed by the subclass.
Hopefully this is a clear example. Please feel free to hit me up for more info!
|Comment by Christoph Schaefer [ 30/Oct/11 ]|
I can confirm this behavior.
after generation of FooBar:
It looks like the generator doesn't check if an attribute is inherited and therefore is already defined.
This bug is really annoying, especially when working with bigger objects.
Tested within Symfony2 v2.0.4
|Comment by Benjamin Eberlei [ 09/Jan/12 ]|
As per the help message of the entity generation inheritance generation is not yet supported and will not work.
Its planned for 2.3.
|Comment by Doctrine Bot [ 05/Aug/14 ]|
A related Github Pull-Request [GH-1098] was assigned:
|Comment by Doctrine Bot [ 18/Aug/14 ]|
A related Github Pull-Request [GH-1098] was closed: