Doctrine 2 - ORM

Single Table Inheritance - Discriminator Column is not populated

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: Git Master
  • Fix Version/s: 2.1
  • Component/s: ORM
  • Security Level: All
  • Labels:
    None

Description

I have the following class inheritance; if I have the mapped super class in there, the discriminator column is not populated; if I remove it, it is correctly set.

/**
 * @ORM\Entity
 * @ORM\Table(name = "a")
 * @ORM\ChangeTrackingPolicy("DEFERRED_EXPLICIT")
 * @ORM\InheritanceType("SINGLE_TABLE")
 * @ORM\DiscriminatorColumn(name="type", type="string", length=20)
 * @ORM\DiscriminatorMap({
 *     "c"   = "C",
 *     "d"   = "D"
 * })
 */
abstract class A { }

/**
 * @ORM\MappedSuperClass
 */
abstract class B extends A { }

/**
 * @ORM\Entity
 */
class C extends B { }

/**
 * @ORM\Entity
 */
class D extends B { }

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Benjamin Eberlei added a comment -

This was fixed or something.

Show
Benjamin Eberlei added a comment - This was fixed or something.
Hide
Permalink
Nikolai Spassoff added a comment -

The following hierarchy doesn't work after commit 5ff44b5ec75f6d716ec3.

The thrown exception is:
'Entity\AbstractMaschine' has to be part of the descriminator map of 'Entity\Inventory'

Inventory is just a plain abstract class which contains the @Id field and a few methods used by all subclasses.

Show
Nikolai Spassoff added a comment - The following hierarchy doesn't work after commit 5ff44b5ec75f6d716ec3. The thrown exception is: 'Entity\AbstractMaschine' has to be part of the descriminator map of 'Entity\Inventory' Inventory is just a plain abstract class which contains the @Id field and a few methods used by all subclasses.
Hide
Permalink
Benjamin Eberlei added a comment -

Yes, please add it to the discriminator map, it is necessary for internal algorithms that this class is in the map.

Show
Benjamin Eberlei added a comment - Yes, please add it to the discriminator map, it is necessary for internal algorithms that this class is in the map.
Hide
Permalink
Benjamin Eberlei added a comment -

Ok this has been changed, its not necessary for abstract classes anymore.

Show
Benjamin Eberlei added a comment - Ok this has been changed, its not necessary for abstract classes anymore.

People

  • Assignee:
    Benjamin Eberlei
    Reporter:
    Johannes Schmitt
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: