Ray\Di\AopCollector

Complexity

Total Complexity

6

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	collect()	0	6	1
A	compile()	0	17	4
A	newInstance()	0	3	1

<?php

declare(strict_types=1);

namespace Ray\Di;

use LogicException;
use Ray\Aop\BindInterface;
use Ray\Aop\CompilerInterface;

use function method_exists;

/**
 * Collects AOP bindings for a dependency
 *
 * @psalm-import-type PointcutList from Types
 * @psalm-type AopBindings = array<string, list<string>>
 */
final class AopCollector implements CompilerInterface
{
    /** @var AopBindings */

The type Ray\Di\AopBindings was not found. Maybe you did not declare it correctly or list all dependencies?

    private array $bindings = [];

    /**
     * @param PointcutList $pointcuts

The type Ray\Di\PointcutList was not found. Maybe you did not declare it correctly or list all dependencies?

     *
     * @return AopBindings
     */
    public function collect(Dependency $dependency, array $pointcuts): array
    {
        $this->bindings = [];

It seems like array() of type array is incompatible with the declared type Ray\Di\AopBindings of property $bindings.

        $dependency->weaveAspects($this, $pointcuts);

        return $this->bindings;

The expression return $this->bindings returns the type array which is incompatible with the documented return type Ray\Di\AopBindings.

    }

    /**
     * {@inheritDoc}
     *
     * @return never
     */
    public function newInstance(string $class, array $args, BindInterface $bind)
    {
        throw new LogicException('AopCollector::newInstance() should never be called');
    }

    /**
     * {@inheritDoc}
     */
    public function compile(string $class, BindInterface $bind): string
    {
        $bindings = $bind->getBindings();
        if (! $bindings) {

$bindings is of type Ray\Aop\MethodBindings, thus it always evaluated to true.

            return $class;
        }

        foreach ($bindings as $method => $interceptors) {
            if (! method_exists($class, $method)) {
                continue;
            }

            /** @var list<string> $interceptors */
            $this->bindings[$method] = $interceptors;
        }

        return $class;
    }
}