Sarala\Query\ApiRequestInspector

Complexity

Total Complexity

10

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	inspect()	0	4	1
A	setRequestReceivedAtApiEndpoint()	0	8	3
A	sanitizeIncludes()	0	8	2
A	isIncludeAllowed()	0	10	3

<?php

declare(strict_types=1);

namespace Sarala\Query;

use Illuminate\Http\Request;
use Illuminate\Support\Str;
use Sarala\Exceptions\UnauthorizedIncludeException;
use Sarala\Http\Requests\ApiRequestAbstract;

class ApiRequestInspector
{
    /** @var ApiRequestAbstract */
    private $request = null;

    public function __construct(array $parameters)
    {
        $this->setRequestReceivedAtApiEndpoint($parameters);
    }

    public function inspect()
    {
        $this->sanitizeIncludes();
    }

    private function setRequestReceivedAtApiEndpoint(array $parameters)
    {
        foreach ($parameters as $parameter) {
            if ($parameter instanceof Request) {
                $this->request = $parameter;

$parameter is of type object<Illuminate\Http\Request>, but the property $request was declared to be of type object<Sarala\Http\Requests\ApiRequestAbstract>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

            }
        }
    }

    private function sanitizeIncludes()
    {
        $this->request->includes()->each(function ($params, $field) {

The method includes() does not exist on Sarala\Http\Requests\ApiRequestAbstract. Did you maybe mean allowedIncludes()?

            if (! $this->isIncludeAllowed($field)) {
                throw new UnauthorizedIncludeException($field, $this->request->allowedIncludes());
            }
        });
    }

    private function isIncludeAllowed($field): bool
    {
        foreach ($this->request->allowedIncludes() as $allowedInclude) {
            if (Str::startsWith($allowedInclude, $field)) {
                return true;
            }
        }

        return false;
    }
}