Spatie\ValueObject\ValueObjectDefinition

Complexity

Total Complexity

9

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	resolveUseStatements()	0	26	4
A	getValueObjectProperties()	0	14	2
A	hasAlias()	0	4	1
A	resolveAlias()	0	4	1

<?php

namespace Spatie\ValueObject;

use ReflectionClass as BaseReflectionClass;
use ReflectionProperty;

class ValueObjectDefinition extends BaseReflectionClass
{
    /** @var array */
    protected $uses = [];

    /** @var \Spatie\ValueObject\ValueObject */
    protected $valueObject;

    public function __construct(ValueObject $valueObject)
    {
        parent::__construct($valueObject);

        $this->valueObject = $valueObject;

        $this->resolveUseStatements();
    }

    protected function resolveUseStatements()
    {
        $handle = fopen($this->getFileName(), 'r');

        while ($line = fgets($handle)) {
            $line = trim($line);

            if (strpos($line, 'use ') !== 0) {
                continue;
            }

            $fqcn = str_replace(['use ', ';'], '', $line);

            $classParts = explode('\\', $fqcn);

            $alias = end($classParts);

            $this->uses[$alias] = $fqcn;

            if (strpos($line, 'class') !== false) {
                break;
            }
        }

        fclose($handle);
    }

    public function hasAlias(string $alias): bool
    {
        return isset($this->uses[$alias]);
    }

    public function resolveAlias(string $alias): string
    {
        return $this->uses[$alias];
    }

    /**
     * @return \Spatie\ValueObject\ValueObjectProperty[]
     */
    public function getValueObjectProperties(): array
    {
        $properties = [];

        foreach (parent::getProperties(ReflectionProperty::IS_PUBLIC) as $reflectionProperty) {

It seems like you call parent on a different method (getProperties() instead of getValueObjectProperties()). Are you sure this is correct? If so, you might want to change this to $this->getProperties().

            $properties[$reflectionProperty->getName()] = ValueObjectProperty::fromReflection(
                $this->valueObject,
                $this,
                $reflectionProperty
            );
        }

        return $properties;
    }
}