[DDC-2708][DCOM-219] Fix PHP 5.4.8 not returning true is_callable on abstract methods anymore.

Author: beberlei

lib/Doctrine/Common/Persistence/Mapping/RuntimeReflectionService.php

@@ -21,6 +21,9 @@
 
 use ReflectionClass;
 use ReflectionProperty;
+use ReflectionMethod;
+use ReflectionException;
+
 use Doctrine\Common\Reflection\RuntimePublicReflectionProperty;
 use Doctrine\Common\Persistence\Mapping\MappingException;
 
@@ -92,6 +95,12 @@ public function getAccessibleProperty($class, $property)
      */
     public function hasPublicMethod($class, $method)
     {
-        return method_exists($class, $method) && is_callable(array($class, $method));
+        try {
+            $reflectionMethod = new ReflectionMethod($class, $method);
+        } catch (ReflectionException $e) {
+            return false;
+        }
+
+        return $reflectionMethod->isPublic();
     }
 }

lib/Doctrine/Common/Persistence/Mapping/StaticReflectionService.php

@@ -78,6 +78,6 @@ public function getAccessibleProperty($class, $property)
      */
     public function hasPublicMethod($class, $method)
     {
-        return method_exists($class, $method) && is_callable(array($class, $method));
+        return true;
     }
 }

tests/Doctrine/Tests/Common/Persistence/Mapping/StaticReflectionServiceTest.php

@@ -58,7 +58,7 @@ public function testGetReflectionClass()
     public function testGetMethods()
     {
         $this->assertTrue($this->reflectionService->hasPublicMethod(__CLASS__, "testGetMethods"));
-        $this->assertFalse($this->reflectionService->hasPublicMethod(__CLASS__, "testGetMethods2"));
+        $this->assertTrue($this->reflectionService->hasPublicMethod(__CLASS__, "testGetMethods2"));
     }
 
     public function testGetAccessibleProperty()