[RFC] Migrate doctrine:generate:entity logic to make:entity

[gabiudrescu]

For me, one of the first downsides I have encountered while testing out Flex with Symfony4 is the missing command of doctrine:generate:entity with all the helper questions + the not working doctrine:generate:entities (see doctrine/DoctrineBundle#729 fore more info).

My previous workflow whenever I was prototyping something was:

• generate the entities with doctrine:generate:entity by answering to a couple of questions;
• I always opted for YAML mapping, so the next step was to edit the files to create the relations between entities
• updating the entities with doctrine:generate:entities to get the new properties and appropriate methods (e.g. __construct with initialized ArrayCollection objects, addElement, removeElement etc.)
• update schema

With the current make:entity command, it took me more than 4 times the time in order to get an up & running 3 entities related between them (Job, Category, Affiliate - remember good ol' Jobeet tutorial).

I would like to bring under your attention the need of migrating the functionality from SensioGeneratorBundle to the MakerBundle for doctrine:generate:entity command (very useful when first creating an entity).

Also, I would like to propose discussing the idea of migrating doctrine:generate:entities from DoctrineBundle, as the core maintainers from that project are not in favor of maintaining this command for the purpose of generating setters and getters based on mapping information.

I don't know if the 2nd proposal should be part of make:entity command or another one command.

[javiereguiluz]

I agree completely. Creating an interactive make:entity was on our TODO list since day one. For the first version of the bundle we didn't want to include interactive commands, but we're working on this.

[gabiudrescu]

@javiereguiluz that's an awesome news! I find those interactive commands very helpful and allow me to be very productive. hope I'm not the only one using / asking for them :)

how about the other idea: taking the responsibility of doctrine:generate:entities from DoctrineBundle to MakerBundle for generating entities based on existing mapping (yml, xml or even annotations)?

[javiereguiluz]

I don't know what to say about your second question. We'll need to think about it.

[weaverryan]

The trick will be to try to not create too many questions. The old command suffered from usability problems: people would make a typo 10 questions in and not be able to recover. That’s the challenge.

I would also like to support relations ships. And I would like to generate getters + setters: that’s especially useful because we can generate really awesome add/remover methods for relationships that synchronize the inverse side.

[Gregwar]

I am currently using Symfony flex and not having entities available to generate accessors is really troublesome in the workflow.

[fredpo]

@gabiudrescu
I use the same chain for prototyping and bootstrapping a project. This whole Flex move is completely pointless without an efficient way to generate Entities, which is time consuming and is a stumbling block when you just want an API up and running quickly.

I'm glad to see this issue getting attention.

[Reflex14]

I was searching for this, because I could not believe it was gone. Happy to see there is an issue, confirming this.

We are on the start of a new project and putting ideas directly into your project, this fast, is the most fun part of any project. And I don't care if I made a typo in the process, most of the time I'll continue and fix it later on.

Right now I'm stubborn and will continue using SF4 but I have a SF3.3 project alongside to generate my orm and entities 🤣

[LinkingYou]

The official symfony documentation (3.4 / 4.0) describes that you can use other formats instead of annotations ( https://symfony.com/doc/current/doctrine.html ).

It also works as described - Symfony gets the needed database / table info from the configured YML file.

But it does not make sense if I should write the YML files in the future and then create the appropriate entity class by hand.

I would also be very happy if the new Maker Bundle can also create and update the appropriate Entity classes from Yml files. I also love annotations elsewhere - but with entities, working with yml files is much faster and easier.

[stof]

The other thing to remember is that the SensioGeneratorBundle command was relying on the Doctrine ORM EntityGenerator, which is not actively maintained and is actively recommended against using it by the Doctrine ORM team.

[zoop1984]

Glad to see that it isn't just me. I've been searching for it everywhere. Might want to take a look at the docs page at https://symfony.com/doc/current/doctrine.html
It first talks about how to generate an entity, then about how to expand fields (including xml/yml examples) and then "The Product class is fully-configured and ready to save to a product table.", thats plainly not true, the steps above it don't update the product class at all and thus the migrations stuff afterwards fails. The example only works if you use annotations, since you'll be defining the field variables at the same time (and afterwards, no other xml/yml examples are given anymore, its all about those horrible annotations).

The main selling point for Symfony always was for us how it can take work away from you, not having to worry to perfectly create object classes, just define your fields and Symfony does the rest. Never believed in annotations but luckily there were always alternate options. But now, we need to do everything manually. Work that previously took seconds, now takes minutes. Time to switch back to Symfony 3.x, in our opinion, Symfony 4 simply isn't production ready yet.

[pascalwacker]

Is there an ETA on an improved make:entity command? And will there be a replacement to generate getter/setter? Sure IDE like PHPStrom can do it too, but I'd still prefer a console command for it.

[weaverryan]

In working on the make:entity improvements right now. I could have a PR as early as this weekend :)

[gabiudrescu]

@weaverryan that's awesome news! Santa is coming early this year :D

[gabiudrescu]

Can't hardly wait to beta test this