PeekAndPoke\Component\PropertyAccess\PropertyAccessFactoryImpl::create()

Complexity

Conditions	5
Paths	4

Size

Total Lines	21
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	5.0342

Importance

Changes

0

<?php
    public function create(\ReflectionClass $class, \ReflectionProperty $property)
    {
        if ($property->isStatic()) {
            throw new \LogicException('Cannot create property access for static properties');
        }

        // is it a public property
        if ($property->isPublic()) {
            return PublicPropertyAccess::create($property->getName());
        }


        // can we you the reflection accessor
        if ($property->isProtected()
            || $property->getDeclaringClass()->getName() === $class->getName()) {

Consider using $property->class. There is an issue with getName() and APC-enabled PHP versions.

Consider using $class->name. There is an issue with getName() and APC-enabled PHP versions.

            return ReflectionPropertyAccess::create($class, $property->getName());
        }

        return ScopedPropertyAccess::create($property->getDeclaringClass()->getName(), $property->getName());

Consider using $property->class. There is an issue with getName() and APC-enabled PHP versions.

    }
}