Silverback\ApiComponentsBundle\ApiPlatform\Metadata\Resource\RoutableResourceMetadataFactory::create()

Complexity

Conditions	4
Paths	4

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function create(string $resourceClass): ResourceMetadata
    {
        $resourceMetadata = $this->decorated->create($resourceClass);

        $refl = new \ReflectionClass($resourceClass);
        if (!$refl->implementsInterface(RoutableInterface::class)) {
            return $resourceMetadata;
        }

        $itemOps = $resourceMetadata->getItemOperations();
        foreach ($itemOps as $name => $config) {
            if ($config['security'] ?? null) {
                continue;
            }
            $itemOps[$name]['security'] = sprintf("is_granted('%s', object)", AbstractRoutableVoter::READ_ROUTABLE);
        }

        return $resourceMetadata->withItemOperations($itemOps);

It seems like $itemOps can also be of type null; however, parameter $itemOperations of ApiPlatform\Core\Metadata\Resource\ResourceMetadata::withItemOperations() does only seem to accept array, maybe add an additional type check?

    }