App\Http\Controllers\Api\User\Index

Complexity

Total Complexity

2

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	21	1
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace App\Http\Controllers\Api\User;

use App\Contracts\Http\Responses\ResponseFactory;
use App\Http\Requests\Api\User\IndexRequest;
use App\Models\User;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Http\JsonResponse;
use KoenHoeijmakers\LaravelFilterable\Contracts\Filtering;

final class Index
{
    private ResponseFactory $responseFactory;

    private Filtering $filtering;

    public function __construct(ResponseFactory $responseFactory, Filtering $filtering)
    {
        $this->responseFactory = $responseFactory;
        $this->filtering = $filtering;
    }

    public function __invoke(IndexRequest $request): JsonResponse
    {
        $builder = User::query()->select(['id', 'name', 'email']);

        $this->filtering->builder($builder)

It seems like $builder can also be of type Illuminate\Database\Query\Builder; however, parameter $builder of KoenHoeijmakers\LaravelFilterable\Contracts\Filtering::builder() does only seem to accept Illuminate\Database\Eloquent\Builder, maybe add an additional type check?

            ->filterFor('name', static function (Builder $builder, string $value) {
                $builder->where('name', 'like', '%' . $value . '%');
            })
            ->filterFor('email', static function (Builder $builder, string $value) {
                $builder->where('email', 'like', '%' . $value . '%');
            })
            ->sortFor('name')
            ->sortFor('email')
            ->defaultSorting('name')
            ->filter();

        $paginator = $builder->paginate(
            $request->input('per_page')
        );

        return $this->responseFactory->paginator($paginator);
    }
}