Koded\Http\ValidatableTrait::validate()

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes	8
Bugs	1	Features	2

<?php
    public function validate(HttpInputValidator $validator, ?Data &$input = null): ?Response
    {
        $input ??= new Immutable($this->getParsedBody() ?? []);

It seems like $this->getParsedBody() ?? array() can also be of type Koded\Http\Interfaces\Response; however, parameter $data of Koded\Stdlib\Immutable::__construct() does only seem to accept array, maybe add an additional type check?

        if (0 === $input->count()) {

The method count() does not exist on Koded\Stdlib\Data. It seems like you code against a sub-type of said class. However, the method does not exist in Koded\Stdlib\Argument or Koded\Stdlib\Configuration. Are you sure you never get one of those?

            $errors = ['validate' => 'Nothing to validate', 'code' => HttpStatus::BAD_REQUEST];
            return new JsonResponse($errors, HttpStatus::BAD_REQUEST, [
                'Content-Type' => 'application/problem+json'
            ]);
        }
        if (empty($errors = $validator->validate($input))) {
            return null;
        }
        $errors['status'] = (int)($errors['status'] ?? HttpStatus::BAD_REQUEST);
        return new JsonResponse($errors, $errors['status'], [
            'Content-Type' => 'application/problem+json'
        ]);