samsonframework\container\annotation\Inject

Complexity

Total Complexity

9

Size/Duplication

Total Lines	33
Duplicated Lines	18.18 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	5
Bugs	0	Features	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	toMethodMetadata()	0	4	1
C	toPropertyMetadata()	6	23	8

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

use samsonframework\container\metadata\MethodMetadata;
use samsonframework\container\metadata\PropertyMetadata;

/**
 * Injection annotation class.
 *
 * @Annotation
 */
class Inject extends CollectionValue implements MethodInterface, PropertyInterface
{
    /** {@inheritdoc} */
    public function toMethodMetadata(MethodMetadata $metadata)
    {
        $metadata->dependencies = $this->collection;

The property dependencies does not seem to exist in samsonframework\container\metadata\MethodMetadata.

    }

    /** {@inheritdoc} */
    public function toPropertyMetadata(PropertyMetadata $propertyMetadata)
    {
        // Get @Inject("value")
        $propertyMetadata->injectable = array_shift($this->collection);

        // Check if we need to append namespace to injectable
        if ($propertyMetadata->injectable !== null && strpos($propertyMetadata->injectable, '\\') === false) {

This code seems to be duplicated across your project.

            $propertyMetadata->injectable = $propertyMetadata->classMetadata->nameSpace . '\\' . $propertyMetadata->injectable;
        }

        // Check if we need to append namespace to type hint
        if ($propertyMetadata->typeHint !== null && strpos($propertyMetadata->typeHint, '\\') === false) {

This code seems to be duplicated across your project.

            $propertyMetadata->typeHint = $propertyMetadata->classMetadata->nameSpace . '\\' . $propertyMetadata->typeHint;
        }

        // Check for inheritance violation
        if ($propertyMetadata->injectable !== null && $propertyMetadata->typeHint !== null) {
            $inheritance = array_merge([$propertyMetadata->injectable], class_parents($propertyMetadata->injectable));
            if (!in_array($propertyMetadata->typeHint, $inheritance, true)) {
                throw new \InvalidArgumentException('@Inject dependency violates ' . $propertyMetadata->typeHint . ' inheritance');
            }
        }
    }
}