Hydrate object with ID about doctrinemodule HOT 3 CLOSED

Weird... I think this may be a case that is not handled. What happens if you add a default @Id @GeneratedValue(strategy="AUTO") and make your email just a column?

from doctrinemodule.

I use MongoDB ODM (and doctrine-mongo-odm-module), I think that in that case it not important, but may be.

What happens if you add a default @id @GeneratedValue(strategy="AUTO") and make your email just a column?

All works nice, if I add some ID in my class and make email field string. Hydration not works if you try hydrate array which contain entity IDs and entity was not found in database by this IDs (at least as I understand).

I think problem in DoctrineModule\Stdlib\Hydrator\DoctrineObject::tryConvertArrayToObject:

protected function tryConvertArrayToObject($data, $object)
{
    $identifierNames  = $this->metadata->getIdentifierFieldNames($object);
    $identifierValues = array();

    if (empty($identifierNames)) {
        return $object;
    }

    foreach ($identifierNames as $identifierName) {
        if (!isset($data[$identifierName]) || empty($data[$identifierName])) {
            return $object;
        }

        $identifierValues[$identifierName] = $data[$identifierName];
    }

    return $this->find(get_class($object), $identifierValues);
}

Maybe last line need to be replaced by:

if ( $existingEntity = $this->find(get_class($object), $identifierValues) ) {
    return $existingEntity;
 }
return $object;

I am not completely sure. But it will fix the error =)

from doctrinemodule.

I will close this one since the entire hydrator has been rewritten. Please verify against latest version (0.7.0 as of current date)

from doctrinemodule.