Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-388

Private properties in @MappedSupperclass don't work

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0-BETA1
    • Component/s: Mapping Drivers
    • Security Level: All
    • Labels:
      None
    • Environment:
      PHP 5.3.1

      Description

      All of my entites extend an abstract class Model to inherit the 'id' property and some other functionality. It has the @MappedSuperclass annotation.

      If the id property is declared private to the Model class, I'll get the following exception when using entites extending it:

      [03-Mar-2010 11:36:29] exception 'ReflectionException' with message 'Property id does not exist' in library/Doctrine/ORM/Mapping/ClassMetadata.php:370
      Stack trace:
      #0 library/Doctrine/ORM/Mapping/ClassMetadata.php(370): ReflectionClass->getProperty('id')
      #1 [internal function]: Doctrine\ORM\Mapping\ClassMetadata->__wakeup()
      ...
      

      I haven't looked at Doctrine code, but perhaps you should be looking for the property on the class which actually has the @Column annotation?

        Issue Links

          Activity

          Hide
          Roman S. Borschel added a comment -

          Fixed in http://github.com/doctrine/doctrine2/commit/d4232d906e433b1fe4dd8aa85aa7a4aca3a2cf4c .
          Make sure to clear the metadata cache if necessary.

          Show
          Roman S. Borschel added a comment - Fixed in http://github.com/doctrine/doctrine2/commit/d4232d906e433b1fe4dd8aa85aa7a4aca3a2cf4c . Make sure to clear the metadata cache if necessary.
          Hide
          Roman S. Borschel added a comment -

          Yes, I think the 'inherited' key is simply not set for fields that are inherited from mapped superclasses. This needs to be fixed.

          Show
          Roman S. Borschel added a comment - Yes, I think the 'inherited' key is simply not set for fields that are inherited from mapped superclasses. This needs to be fixed.

            People

            • Assignee:
              Roman S. Borschel
              Reporter:
              Jaka Jancar
            • Votes:
              2 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: