Doctrine Common
  1. Doctrine Common
  2. DCOM-231

Lifecycle-Callback MappingExceptions for abstract functions in PHP >=5.4.8

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4.2
    • Component/s: None
    • Labels:
      None
    • Environment:
      Windows 7 x64, PHP 5.5.3

      Description

      Hi,

      i noticed that in PHP starting with Version 5.4.8 the ReflectionService implementations returns a different result for the hasPublicMethod function than in PHP <= 5.4.7 when called on an abstract function. This prevents me from defining a lifecycle callback on an abstract function for a mapped superclass.

      I think that at least for mapped supperclasses it should be checked if the class and the function is abstract so it can be assumed that every extending class has this function and it is in fact callable.

      The bug was introduced by a bugfix in PHP 5.4.8.

      cheers

      konrad

        Activity

        Hide
        Doctrine Bot added a comment -

        A related Github Pull-Request [GH-301] was closed:
        https://github.com/doctrine/common/pull/301

        Show
        Doctrine Bot added a comment - A related Github Pull-Request [GH-301] was closed: https://github.com/doctrine/common/pull/301
        Hide
        Konrad Mohrfeldt added a comment -

        oh… and… this bug should probably be moved to common as it affects orm but is caused in common

        Show
        Konrad Mohrfeldt added a comment - oh… and… this bug should probably be moved to common as it affects orm but is caused in common
        Hide
        Konrad Mohrfeldt added a comment -

        i’ve created a pull request to fix this issue. see https://github.com/doctrine/common/pull/301

        Show
        Konrad Mohrfeldt added a comment - i’ve created a pull request to fix this issue. see https://github.com/doctrine/common/pull/301

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Konrad Mohrfeldt
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: