VGirol\JsonApiFaker\Laravel\Factory\IsResource

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	3
A	setValues()	0	5	1

<?php

namespace VGirol\JsonApiFaker\Laravel\Factory;

use Illuminate\Database\Eloquent\Model;
use VGirol\JsonApiFaker\Exception\JsonApiFakerException;

/**
 * An abstract factory for resources (resource object or resource identifer).
 */
trait IsResource
{
    use HasModel;

    /**
     * Class constructor.
     *
     * @param Model|null  $model
     * @param string|null $resourceType
     *
     * @return void
     * @throws JsonApiFakerException
     */
    public function __construct($model = null, ?string $resourceType = null)
    {
        if (($model !== null) && ($resourceType !== null)) {
            $this->setValues($model, $resourceType);
        }
    }

    /**
     * Set the model and the resource type.
     *
     * @param Model  $model
     * @param string $resourceType
     *
     * @return static
     * @throws JsonApiFakerException
     */
    public function setValues($model, string $resourceType)
    {
        return $this->setModel($model)
            ->setId($model->getKey())

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

            ->setResourceType($resourceType);
    }
}