Spiechu\SymfonyCommonsBundle\Service\XmlSchemaValidatorFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getValidator()	0	10	2

<?php

declare(strict_types=1);

namespace Spiechu\SymfonyCommonsBundle\Service;

use Symfony\Component\Config\Exception\FileLocatorFileNotFoundException;

class XmlSchemaValidatorFactory extends AbstractSchemaValidatorFactory
{
    /**
     * @param string $id
     *
     * @throws FileLocatorFileNotFoundException
     * @throws \InvalidArgumentException
     * @throws \RuntimeException
     *
     * @return XmlSchemaValidator
     */
    public function getValidator(string $id): XmlSchemaValidator
    {
        $this->assertHasSchema($id);

        if (!$this->registeredSchemas[$id] instanceof XmlSchemaValidator) {
            $this->registeredSchemas[$id] = new XmlSchemaValidator($this->fileLocator->locate($this->registeredSchemas[$id]));

It seems like $this->fileLocator->locate($this->registeredSchemas[$id]) targeting Symfony\Component\Config\FileLocatorInterface::locate() can also be of type array; however, Spiechu\SymfonyCommonsBundle\Service\XmlSchemaValidator::__construct() does only seem to accept string, maybe add an additional type check?

        }

        return $this->registeredSchemas[$id];
    }
}