Add documentation for auto primary key doctrine generators

[theofidry]

Closes #40

[umpirsky]

Is it possible to override ID generator for all entities at once (instead of doing it on per entity basis)?

[theofidry]

As it's done automatically, if you set the ID for all entities then the ID generator will be overridden for all

[umpirsky]

I don't want to set ID, but ID generator, like:

$dummyMetadata = $this->manager->getMetadataFactory()->getMetadataFor(Dummy::class);

// Disable the auto generation of the foreign key
$dummyMetadata->setIdGeneratorType(ClassMetadata::GENERATOR_TYPE_NONE);
$dummyMetadata->setIdGenerator(new MyIdGenerator());

But not only for Dummy::class, I want to do that for all entities.

[theofidry]

oh, yeah not out of the box. But you should be able to easily overwrite the existing persister loader to do that. Maybe it can be refactored a bit to make it easier

[umpirsky]

This PersisterLoader? How? It's @final BTW. :)

[theofidry]

Actually overriding https://github.com/theofidry/AliceDataFixtures/blob/master/src/Bridge/Doctrine/Persister/ObjectManagerPersister.php seems more correct. You can override it by creating your own class and registering a service with the same ID.

@final tags are here to say this was not intended to be extended. But it can be removed if the use case is valid, I just do this by default to ease the refactoring.

[umpirsky]

Thanks @theofidry!

Do you think that making ORMAssignedGenerator a service and injecting it into ObjectManagerPersister would make sense? It would ease overriding. I can submit PR if you are willing to merge such contrib?

[umpirsky]

@theofidry This makes it even harder to override.

[theofidry]

Do you think that making ORMAssignedGenerator a service and injecting it into ObjectManagerPersister would make sense? It would ease overriding. I can submit PR if you are willing to merge such contrib?

that or making the persister extendable and putting some logic in a method to make it easy to override. I'm fine either way.

This makes it even harder to override.

Indeed this should be in a factory instead :/

[umpirsky]

Fixed in #91. Thanks!