SignpostMarv\DaftObject\SchemaOrg\DaftObjectTraits\HasSku

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

54.55%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	SetSku()	0	6	1
A	GetSku()	0	12	1

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

namespace SignpostMarv\DaftObject\SchemaOrg\DaftObjectTraits;

use SignpostMarv\DaftObject\SchemaOrg\TypeUtilities;

trait HasSku
{
    /**
    * @return array<int, string>
    */
    public function GetSku() : array
    {
        /**
        * @var array<int, string>
        */
        $out = TypeUtilities::ExpectRetrievedValueIsArray(
            'sku',
            $this->RetrievePropertyValueFromData('sku'),

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, string> $value
    */
    public function SetSku(array $value) : void
    {
        $this->NudgePropertyWithUniqueTrimmedStringsMightNotBeString(

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

            'sku',
            __METHOD__,
            $value
        );
    }
}