\Doctrine\ORM\EntityManager::find() with optimistic lock for deleted entity raises an error #6705

New Issue

admin · 2026-01-22T15:37:20+01:00

admin commented

2026-01-22 15:37:20 +01:00

Originally created by @ilianiv on GitHub (Apr 29, 2021).

If the requested entity ID does not exist (e.g. because it has been deleted), the error is triggered.

Version: 2.8.4

Error:
Doctrine\ORM\UnitOfWork::lock(): Argument 1 ($entity) must be of type object, null given

Called in:
vendor/doctrine/orm/lib/Doctrine/ORM/EntityManager.php:474

public function find($className, $id, $lockMode = null, $lockVersion = null)
    {
       ...
       $entity = $unitOfWork->tryGetById($sortedId, $class->rootEntityName);
       ...
        switch (true) {
            case $lockMode === LockMode::OPTIMISTIC:
                $entity = $persister->load($sortedId);   // This may return NULL

>>>>>           $unitOfWork->lock($entity, $lockMode, $lockVersion);  

                return $entity;

            case $lockMode === LockMode::PESSIMISTIC_READ:
            case $lockMode === LockMode::PESSIMISTIC_WRITE:
                return $persister->load($sortedId, null, null, [], $lockMode);

            default:
                return $persister->loadById($sortedId);
        }
    }

Originally created by @ilianiv on GitHub (Apr 29, 2021). If the requested entity ID does not exist (e.g. because it has been deleted), the error is triggered. **Version:** 2.8.4 **Error:** Doctrine\ORM\UnitOfWork::lock(): Argument 1 ($entity) must be of type object, null given **Called in:** vendor/doctrine/orm/lib/Doctrine/ORM/EntityManager.php:474 ``` public function find($className, $id, $lockMode = null, $lockVersion = null) { ... $entity = $unitOfWork->tryGetById($sortedId, $class->rootEntityName); ... switch (true) { case $lockMode === LockMode::OPTIMISTIC: $entity = $persister->load($sortedId); // This may return NULL >>>>> $unitOfWork->lock($entity, $lockMode, $lockVersion); return $entity; case $lockMode === LockMode::PESSIMISTIC_READ: case $lockMode === LockMode::PESSIMISTIC_WRITE: return $persister->load($sortedId, null, null, [], $lockMode); default: return $persister->loadById($sortedId); } } ```

admin added the Bug label 2026-01-22 15:37:20 +01:00

admin closed this issue

2026-01-22 15:37:20 +01:00

admin referenced this issue

2026-01-22 16:06:26 +01:00

[PR #6705] Fix parameter name comparison in AbstractQuery regarding different types (fixes #6699) #10103

Sign in to join this conversation.