ORM\Id\AssignedGenerator try to current() on non-array

Benjamin Eberlei added a comment - 05/Jun/11 10:54 AM Optimized error handling

Slam added a comment - 08/Jun/11 4:37 PM - edited Error optimizing is welcome but this does not resolve the issue. Again, if $value is new object, for example a new Entity\User with an empty protected $id : if (is_object($value)) { if (!$em->getUnitOfWork()->isInIdentityMap($value)) { throw ORMException::entityMissingForeignAssignedId($entity, $value); } // NOTE: Single Columns as associated identifiers only allowed - this constraint it is enforced. $identifier[$idField] = current($em->getUnitOfWork()->getEntityIdentifier($value)); } else { $identifier[$idField] = $value; } will ALWAYS throw an exception in the case the Entity is new. In my projects, i resolved with if (is_object($value)) { if (!$em->getUnitOfWork()->isInIdentityMap($value)) { $identifier[$idField] = null ; } else { // NOTE: Single Columns as associated identifiers only allowed - this constraint it is enforced. $identifier[$idField] = current($em->getUnitOfWork()->getEntityIdentifier($value)); } } else { $identifier[$idField] = $value; } I have not unit-tested the solution (i'm sorry) and I think a more accurate investigation is needed.

Benjamin Eberlei added a comment - 11/Jun/11 7:39 AM Please read: http://www.doctrine-project.org/docs/orm/2.0/en/tutorials/composite-primary-keys.html#general-considerations This is a necessary requirement, that means if you want to create two entities with a composite pk key of the one depending on the other you have to use two flush operations when using MySQL Auto Increment Keys. This will work with PostgreSQL or Oracle Sequences, but not with Auto increment keys.

Slam added a comment - 11/Jun/11 8:46 AM Ok, roger. Just one note: you must know that both two entities are STATE_NEW, because if you try to getEntityState($dependentEntity) , UnitOfWork.php:2155 will throw a E_RECOVERABLE_ERROR because the $independentEntity could not be converted to a string (because it has no id).