hydrate composite PK about doctrine-laminas-hydrator HOT 6 CLOSED

Why isn't $metadata->getReflectionClass()? Can we solve that to get the same value as a new \ReflectionClass?

from doctrine-laminas-hydrator.

Ok, I think the code works ok without line #L485 and the problem is just testHydrateOneToManyAssociationByReferenceUsingIdentifiersArrayForRelations missing a expectation to return the correct mock for $this->objectManager->getClassMetadata(get_class($object)) in https://github.com/doctrine/doctrine-laminas-hydrator/blob/master/src/DoctrineObject.php#L136

from doctrine-laminas-hydrator.

Can you put that into a PR?

from doctrine-laminas-hydrator.

Yes, I'm trying to figure out how to fix it without changing too much tests

The problem with testHydrateOneToManyAssociationByReferenceUsingIdentifiersArrayForRelations is that if you keep the PK, it calls an aditional hydrate() in https://github.com/doctrine/doctrine-laminas-hydrator/blob/master/src/DoctrineObject.php#L496 (that's why it's failing only in ByReferenceUsingIdentifiersArray)

In this specific test case (by reference + array identifier) we need to mock two metadata in configureObjectManagerForOneToManyEntity (currently it's only mocking $this->metadata for OneToManyEntity but we also need a mock for ByValueDifferentiatorEntity), but I think that requires to do quite a big refactor in the test case :/

from doctrine-laminas-hydrator.

@rodsouto this repo has migrated to Doctrine 3.0. Are you still going to submit a PR?

from doctrine-laminas-hydrator.

Not in the short term, you can close this if you like

from doctrine-laminas-hydrator.