samsonframework\container\annotation\Service

Complexity

Total Complexity

1

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	8
Bugs	2	Features	2

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1

<?php declare(strict_types = 1);
/**
 * Created by PhpStorm.
 * User: root
 * Date: 27.07.2016
 * Time: 1:55.
 */
namespace samsonframework\container\annotation;

use samsonframework\container\configurator\ServiceConfigurator;

/**
 * Service configurator annotation class.
 * @see    \samsonframework\container\configurator\ServiceConfigurator
 *
 * @author Vitaly Egorov <egorov@samsonos.com>
 *
 * @Annotation
 */
class Service extends ServiceConfigurator
{
    use AnnotationValueTrait;

    /**
     * Service annotation configurator constructor.
     *
     * @param string|array $valueOrValues Service unique name
     *
     * @throws \InvalidArgumentException
     */
    public function __construct($valueOrValues)
    {
        // Parse annotation value
        $serviceNameData = $this->parseAnnotationValue($valueOrValues);

It seems like $valueOrValues defined by parameter $valueOrValues on line 31 can also be of type string; however, samsonframework\container\annotation\AnnotationValueTrait::parseAnnotationValue() does only seem to accept array, maybe add an additional type check?

        // Pass to service configurator
        parent::__construct(array_shift($serviceNameData));
    }
}