NovaThinKit\Nova\ResourceTemplates\TemplateFinder

Complexity

Total Complexity

11

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

91.3%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	templatesMap()	0	16	6
A	templatesNames()	0	6	1
A	find()	0	10	4

<?php

namespace NovaThinKit\Nova\ResourceTemplates;

use Laravel\Nova\Resource;

class TemplateFinder
{
    public static array $templatesMap = [];

    public static function templatesMap(string $modelClass, array $map = null, $merge = true)
    {
        if (is_array($map)) {
            collect($map)->each(function ($templateClass, $templateName) {

$map of type array is incompatible with the type Illuminate\Contracts\Support\Arrayable expected by parameter $value of collect().

                if (!is_string($templateName)) {
                    throw new \Exception('Template name should be string');
                }
                if (!is_a($templateClass, ResourceTemplate::class, true)) {
                    throw new \Exception('Template class should extends ResourceTemplate');
                }
            });
            static::$templatesMap[$modelClass] = $merge && (static::$templatesMap[$modelClass] ?? false)
                ? $map + static::$templatesMap[$modelClass] : $map;
        }

        return static::$templatesMap[$modelClass] ?? [];
    }

    public static function templatesNames(string $modelClass): array
    {
        return collect(static::templatesMap($modelClass))

It seems like static::templatesMap($modelClass) can also be of type array; however, parameter $value of collect() does only seem to accept Illuminate\Contracts\Support\Arrayable, maybe add an additional type check?

            ->mapWithKeys(function ($templateClass, $templateName) {
                return [$templateName => $templateClass::name()];
            })->all();
    }


    public static function find(string $template, Resource $resource): ?ResourceTemplate
    {
        $class = static::templatesMap($resource::$model)[$template] ?? null;
        if ($class
            && is_string($class)
            && is_a($class, ResourceTemplate::class, true)) {
            return new $class($resource);
        }

        return null;
    }
}