App\Service\User\Find

Complexity

Total Complexity

7

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Test Coverage

Coverage

93.75%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	search()	0	3	1
A	getOne()	0	9	2
A	getUsersByPage()	0	18	3
A	getAll()	0	3	1

<?php

declare(strict_types=1);

namespace App\Service\User;

final class Find extends Base
{
    /**
     * @return array<string>
     */
    public function getAll(): array
    {
        return $this->userRepository->getAllUsers();
    }

    /**
     * @return array<string>
     */
    public function getUsersByPage(
        int $page,
        int $perPage,
        ?string $name,
        ?string $email
    ): array {
        if ($page < 1) {
            $page = 1;
        }
        if ($perPage < 1) {
            $perPage = self::DEFAULT_PER_PAGE_PAGINATION;
        }

        return $this->userRepository->getUsersByPage(

The expression return $this->userRepository->getUsersByPage($page, $perPage, $name, $email) returns the type array<string,array|array<string,double|integer>> which is incompatible with the documented return type string[].

            $page,
            $perPage,
            $name,
            $email
        );
    }

    public function getOne(int $userId): object
    {
        if (self::isRedisEnabled() === true) {
            $user = $this->getUserFromCache($userId);
        } else {
            $user = $this->getUserFromDb($userId)->toJson();
        }

        return $user;
    }

    public function search(string $usersName): array
    {
        return $this->userRepository->search($usersName);
    }
}