Codefocus\ManagedCache\Traits\IdentifiesEloquentModels

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	isModel()	0	3	2
A	getModelClassNameAndId()	0	12	2

<?php

namespace Codefocus\ManagedCache\Traits;

use Illuminate\Database\Eloquent\Model;

trait IdentifiesEloquentModels
{
    /**
     * Return whether the specified class name is an Eloquent Model.
     *
     * @param mixed $value
     *
     * @return bool
     */
    protected function isModel($value): bool
    {
        return is_object($value) && is_subclass_of($value, Model::class);
    }

    /**
     * Returns an array containing the Model class name, and the model id.
     * The id is null if $model is a string.
     *
     * @param Model|string $model
     *
     * @return array
     */
    protected function getModelClassNameAndId($model): array
    {
        if ($this->isModel($model)) {
            return [
                get_class($model),

It seems like $model can also be of type string; however, parameter $object of get_class() does only seem to accept object, maybe add an additional type check?

                $model->getKey(),
            ];
        }

        return [
            $model,
            null,
        ];
    }
}