App\Http\Controllers\User\UserController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	20	1
A	profile()	0	12	2
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace App\Http\Controllers\User;

use App\Modules\User\UserRepository;
use Psr\Http\Message\ResponseFactoryInterface;
use Psr\Http\Message\ResponseInterface as Response;
use Psr\Http\Message\ServerRequestInterface;
use Yiisoft\Data\Paginator\OffsetPaginator;
use Yiisoft\Data\Reader\Sort;
use Yiisoft\Router\CurrentRoute;
use Yiisoft\Yii\View\ViewRenderer;

final class UserController
{
    private const PAGINATION_INDEX = 5;

    public function __construct(private ViewRenderer $viewRenderer)
    {
        $this->viewRenderer = $viewRenderer->withControllerName('user');
    }

    public function index(
        CurrentRoute $currentRoute,
        ServerRequestInterface $request,
        UserRepository $userRepository
    ): Response {
        $page = (int)$currentRoute->getArgument('page', '1');
        $sortOrderString = $request->getQueryParams();

        $dataReader = $userRepository
            ->findAll()
            ->withSort(Sort::only(['id', 'login'])->withOrderString($sortOrderString['sort'] ?? ''));

        $paginator = (new OffsetPaginator($dataReader))->withPageSize(self::PAGINATION_INDEX);

        return $this->viewRenderer->render(
            'index',
            [
                'page' => $page,
                'paginator' => $paginator,
                'sortOrder' => $sortOrderString['sort'] ?? '',
            ]
        );
    }

    public function profile(
        CurrentRoute $currentRoute,
        ResponseFactoryInterface $responseFactory,
        UserRepository $userRepository
    ): Response {
        $login = $currentRoute->getArgument('login');
        $item = $userRepository->findByLogin($login);

It seems like $login can also be of type null; however, parameter $login of App\Modules\User\UserRepository::findByLogin() does only seem to accept string, maybe add an additional type check?

        if ($item === null) {
            return $responseFactory->createResponse(404);
        }

        return $this->viewRenderer->render('profile', ['item' => $item]);
    }
}