Translation\Bundle\Command\StorageTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getStorage()	0	9	2

<?php

/*
 * This file is part of the PHP Translation package.
 *
 * (c) PHP Translation team <tobias.nyholm@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Translation\Bundle\Command;

use Translation\Bundle\Service\StorageManager;
use Translation\Bundle\Service\StorageService;

trait StorageTrait
{
    /**
     * @var StorageManager
     */
    private $storageManager;

    /**
     * @param string|string[]|null $configName
     *
     * @throws \InvalidArgumentException
     */
    private function getStorage($configName): StorageService
    {
        if (null === $storage = $this->storageManager->getStorage($configName)) {
            $availableStorages = $this->storageManager->getNames();

            throw new \InvalidArgumentException(\sprintf('Unknown storage "%s". Available storages are "%s".', $configName, \implode('", "', $availableStorages)));

It seems like $configName can also be of type string[]; however, parameter $values of sprintf() does only seem to accept double|integer|string, maybe add an additional type check?

        }

        return $storage;
    }
}