GuzzleHttp\Command\Guzzle\Handler\ValidatedDescriptionHandler::__invoke()

Complexity

Conditions	10
Paths	1

Size

Total Lines	39
Code Lines	21

Duplication

Lines	25
Ratio	64.1 %

Code Coverage

Tests	23
CRAP Score	10.3248

Importance

Changes

0

<?php namespace GuzzleHttp\Command\Guzzle\Handler;
    public function __invoke(callable $handler)
    {
        return function (CommandInterface $command) use ($handler) {
            $errors = [];
            $operation = $this->description->getOperation($command->getName());

            foreach ($operation->getParams() as $name => $schema) {

This code seems to be duplicated across your project.

                $value = $command[$name];
                if (! $this->validator->validate($schema, $value)) {
                    $errors = array_merge($errors, $this->validator->getErrors());
                } elseif ($value !== $command[$name]) {
                    // Update the config value if it changed and no validation
                    // errors were encountered
                    $command[$name] = $value;
                }
            }

            if ($params = $operation->getAdditionalParameters()) {
                foreach ($command->toArray() as $name => $value) {

This code seems to be duplicated across your project.

                    // It's only additional if it isn't defined in the schema
                    if (! $operation->hasParam($name)) {
                        // Always set the name so that error messages are useful
                        $params->setName($name);
                        if (! $this->validator->validate($params, $value)) {
                            $errors = array_merge($errors, $this->validator->getErrors());
                        } elseif ($value !== $command[$name]) {
                            $command[$name] = $value;
                        }
                    }
                }
            }

            if ($errors) {

The expression $errors of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                throw new CommandException('Validation errors: ' . implode("\n", $errors), $command);
            }

            return $handler($command);
        };
    }
}