DDC-351: JoinedSubclassPersister uses id field name in query 'DELETE FROM...' #437

New Issue

admin · 2026-01-22T12:38:10+01:00

admin commented

2026-01-22 12:38:10 +01:00

Originally created by @doctrinebot on GitHub (Feb 19, 2010).

Originally assigned to: @beberlei on GitHub.

Jira issue originally created by user cloun:

My entity has identity field with name which is not equal with identity column in database:

class File
{
  /****
   * @Column(name="file_id")
   * @Id
   */
  protected $id;
  ...

When this entity is deleted by JoinedSubclassPersister, field name is used instead of column name, so query looks like 'DELETE FROM file WHERE id = ?'.
I think that problem is in Doctrine\ORM\Persisters\JoinedSubclassPersister.php line 275:

    public function delete($entity)
    {
        $id = array_combine(
            $this->_class->identifier,
            $this->_em->getUnitOfWork()->getEntityIdentifier($entity)
        );
       ...

but should like in update method

    public function delete($entity)
    {
        $id = array_combine(
            $this->_class->getIdentifierColumnNames(),
            $this->_em->getUnitOfWork()->getEntityIdentifier($entity)
        );
       ...

Originally created by @doctrinebot on GitHub (Feb 19, 2010). Originally assigned to: @beberlei on GitHub. Jira issue originally created by user cloun: My entity has identity field with name which is not equal with identity column in database: ``` class File { /**** * @Column(name="file_id") * @Id */ protected $id; ... ``` When this entity is deleted by JoinedSubclassPersister, field name is used instead of column name, so query looks like 'DELETE FROM file WHERE id = ?'. I think that problem is in Doctrine\ORM\Persisters\JoinedSubclassPersister.php line 275: ``` public function delete($entity) { $id = array_combine( $this->_class->identifier, $this->_em->getUnitOfWork()->getEntityIdentifier($entity) ); ... ``` but should like in update method ``` public function delete($entity) { $id = array_combine( $this->_class->getIdentifierColumnNames(), $this->_em->getUnitOfWork()->getEntityIdentifier($entity) ); ... ```

admin added the Bug label 2026-01-22 12:38:10 +01:00

admin closed this issue

2026-01-22 12:38:10 +01:00

admin commented

2026-01-22 12:38:11 +01:00

@doctrinebot commented on GitHub (Feb 19, 2010):

Comment created by romanb:

Thanks! Good catch.

@Benjamin: Can you apply this? I'm in the middle of finishing up my larger refactorings :)

@doctrinebot commented on GitHub (Feb 19, 2010): Comment created by romanb: Thanks! Good catch. @Benjamin: Can you apply this? I'm in the middle of finishing up my larger refactorings :)

admin commented

2026-01-22 12:38:12 +01:00

@doctrinebot commented on GitHub (Feb 19, 2010):

Comment created by @beberlei:

Fixed.

@doctrinebot commented on GitHub (Feb 19, 2010): Comment created by @beberlei: Fixed.

admin commented

2026-01-22 12:38:13 +01:00

@doctrinebot commented on GitHub (Feb 19, 2010):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Feb 19, 2010): Issue was closed with resolution "Fixed"

admin commented