SignpostMarv\DaftObject\SchemaOrg\DaftObjectTraits\HasDirector

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Test Coverage

Coverage

75%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	SetDirector()	0	3	1
A	GetDirector()	0	12	1

<?php
/**
* @author SignpostMarv
*/
declare(strict_types=1);

namespace SignpostMarv\DaftObject\SchemaOrg\DaftObjectTraits;

use SignpostMarv\DaftObject\SchemaOrg\TypeUtilities;

trait HasDirector
{
    use DaftObjectTrait;

    /**
    * @return array<int, \SignpostMarv\DaftObject\SchemaOrg\Person>
    */
    public function GetDirector() : array
    {
        /**
        * @var array<int, \SignpostMarv\DaftObject\SchemaOrg\Person>
        */
        $out = TypeUtilities::ExpectRetrievedValueIsArray(
            'director',
            $this->RetrievePropertyValueFromData('director'),

It seems like RetrievePropertyValueFromData() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            static::class
        );

        return $out;
    }

    /**
    * @param array<int, \SignpostMarv\DaftObject\SchemaOrg\Person> $value
    */
    public function SetDirector(array $value) : void
    {
        $this->NudgePropertyWithUniquePersons('director', __METHOD__, $value);

The method NudgePropertyWithUniquePersons() does not exist on SignpostMarv\DaftObject\SchemaOrg\DaftObjectTraits\HasDirector. Did you maybe mean NudgePropertyWithUniqueValuesOfThings()?

    }
}