Imanghafoori\Middlewarize\Pipeline

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

93.33%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	carry()	0	34	4

<?php

namespace Imanghafoori\Middlewarize;

use Illuminate\Contracts\Support\Responsable;
use Illuminate\Http\Request;
use Illuminate\Pipeline\Pipeline as CorePipe;

class Pipeline extends CorePipe
{
    /**
     * Get a Closure that represents a slice of the application onion.
     *
     * @return \Closure
     */
    protected function carry()
    {
        return function ($stack, $pipe) {
            return function ($passable) use ($stack, $pipe) {
                if (is_callable($pipe)) {
                    // If the pipe is an instance of a Closure, we will just call it directly but
                    // otherwise we'll resolve the pipes out of the container and call it with
                    // the appropriate method and arguments, returning the results back out.
                    return $pipe($passable, $stack);
                } elseif (! is_object($pipe)) {
                    [$name, $parameters] = $this->parsePipeString($pipe);

The variable $name seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

The variable $parameters seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

                    // If the pipe is a string we will parse the string and resolve the class out
                    // of the dependency injection container. We can then build a callable and
                    // execute the pipe function giving in the parameters that are required.
                    $name = explode('@', $name);

The variable $name seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

                    $pipe = $this->getContainer()->make($name[0]);

Consider using a different name than the imported variable $pipe, or did you forget to import by reference?

                    $parameters = array_merge([$passable, $stack], $parameters);

The variable $parameters seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

                } else {
                    // If the pipe is already an object we'll just make a callable and pass it to
                    // the pipe as-is. There is no need to do any extra parsing and formatting
                    // since the object we're given was already a fully instantiated object.
                    $parameters = [$passable, $stack];
                }

                $method = $name[1] ?? $this->method;

                return method_exists($pipe, $method)
                    ? $pipe->{$method}(...$parameters)
                    : $pipe(...$parameters);
            };
        };
    }

}