Swis\JsonApi\Client\DocumentFactory

Complexity

Total Complexity

10

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Test Coverage

Coverage

96.43%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	make()	0	11	3
A	getIncluded()	0	6	1
A	itemCanBeIncluded()	0	5	4
A	getIncludedFromItem()	0	17	2

<?php

declare(strict_types=1);

namespace Swis\JsonApi\Client;

use Swis\JsonApi\Client\Exceptions\UnsupportedDataException;
use Swis\JsonApi\Client\Interfaces\DataInterface;
use Swis\JsonApi\Client\Interfaces\DocumentInterface;
use Swis\JsonApi\Client\Interfaces\ItemInterface;

class DocumentFactory
{
    /**
     * @param \Swis\JsonApi\Client\Interfaces\DataInterface $data
     *
     * @return \Swis\JsonApi\Client\ItemDocument|\Swis\JsonApi\Client\CollectionDocument
     */
    public function make(DataInterface $data): DocumentInterface
    {
        if ($data instanceof ItemInterface) {
            $document = new ItemDocument();
        } elseif ($data instanceof Collection) {
            $document = new CollectionDocument();
        } else {
            throw new UnsupportedDataException(sprintf('%s is not supported as input', get_class($data)));
        }

        return $document->setData($data)->setIncluded($this->getIncluded($data));
    }

    /**
     * @param \Swis\JsonApi\Client\Interfaces\DataInterface $data
     *
     * @return \Swis\JsonApi\Client\Collection
     */
    private function getIncluded(DataInterface $data): Collection
    {
        return Collection::wrap($data)

$data of type Swis\JsonApi\Client\Interfaces\DataInterface is incompatible with the type iterable expected by parameter $value of Illuminate\Support\Collection::wrap().

            ->flatMap(fn (ItemInterface $item) => $this->getIncludedFromItem($item))
            ->unique(static fn (ItemInterface $item) => sprintf('%s:%s', $item->getType(), $item->getId()))
            ->values();
    }

    /**
     * @param \Swis\JsonApi\Client\Interfaces\ItemInterface $item
     *
     * @return \Swis\JsonApi\Client\Collection
     */
    private function getIncludedFromItem(ItemInterface $item): Collection
    {
        return Collection::make($item->getRelations())

$item->getRelations() of type array is incompatible with the type Illuminate\Contracts\Support\Arrayable expected by parameter $items of Illuminate\Support\Collection::make().

            ->reject(
                static function ($relationship) {
                    /* @var \Swis\JsonApi\Client\Interfaces\OneRelationInterface|\Swis\JsonApi\Client\Interfaces\ManyRelationInterface $relationship */
                    return $relationship->shouldOmitIncluded() || !$relationship->hasIncluded();
                }
            )
            ->flatMap(
                static function ($relationship) {
                    /* @var \Swis\JsonApi\Client\Interfaces\OneRelationInterface|\Swis\JsonApi\Client\Interfaces\ManyRelationInterface $relationship */
                    return Collection::wrap($relationship->getIncluded());

It seems like $relationship->getIncluded() can also be of type Swis\JsonApi\Client\Interfaces\ItemInterface; however, parameter $value of Illuminate\Support\Collection::wrap() does only seem to accept iterable, maybe add an additional type check?

                }
            )
            ->flatMap(fn (ItemInterface $item) => Collection::wrap($item)->merge($this->getIncludedFromItem($item)))

$item of type Swis\JsonApi\Client\Interfaces\ItemInterface is incompatible with the type iterable expected by parameter $value of Illuminate\Support\Collection::wrap().

            ->filter(fn (ItemInterface $item) => $this->itemCanBeIncluded($item));
    }

    /**
     * @param \Swis\JsonApi\Client\Interfaces\ItemInterface $item
     *
     * @return bool
     */
    private function itemCanBeIncluded(ItemInterface $item): bool
    {
        return $item->hasType()
            && $item->hasId()
            && ($item->hasAttributes() || $item->hasRelationships());
    }
}