PeekAndPoke\Component\PropertyAccess\PropertyAccessFactoryImpl

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

88.89%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	create()	0	21	5

<?php
/**
 * Created by gerk on 13.11.17 08:06
 */

namespace PeekAndPoke\Component\PropertyAccess;

/**
 * @author Karsten J. Gerber <kontakt@karsten-gerber.de>
 */
class PropertyAccessFactoryImpl implements PropertyAccessFactory
{
    /**
     * @param \ReflectionClass    $class
     * @param \ReflectionProperty $property
     *
     * @return PropertyAccess
     */
    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.

    }
}