codefocus / managedcache

src/Traits/IdentifiesEloquentModels.php

<?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
     * @param int|null $modelId (default: null)
     *
     * @return array
     */
    protected function getModelClassNameAndId($model, ?int $modelId = null): 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?

                $modelId ?? $model->getKey(),
            ];
        }

        return [
            $model,
            $modelId,
        ];
    }
}