App\User\UserService

Complexity

Total Complexity

7

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	logout()	0	4	2
A	login()	0	18	4
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace App\User;

use App\Exception\BadRequestException;
use Throwable;
use Yiisoft\Auth\IdentityInterface;
use Yiisoft\Auth\IdentityRepositoryInterface;
use Yiisoft\User\User;

This use statement conflicts with another class in this namespace, App\User\User. Consider defining an alias.

final class UserService
{
    private IdentityRepositoryInterface $identityRepository;
    private User $user;

    public function __construct(User $user, IdentityRepositoryInterface $identityRepository)
    {
        $this->user = $user;
        $this->identityRepository = $identityRepository;
    }

    /**
     * @param string $login
     * @param string $password
     *
     * @throws BadRequestException
     *
     * @return IdentityInterface
     */
    public function login(string $login, string $password): IdentityInterface
    {
        $identity = $this->identityRepository->findByLogin($login);

The method findByLogin() does not exist on Yiisoft\Auth\IdentityRepositoryInterface. It seems like you code against a sub-type of Yiisoft\Auth\IdentityRepositoryInterface such as App\User\UserRepository.

        if ($identity === null) {
            throw new BadRequestException('No such user.');
        }

        if (!$identity->validatePassword($password)) {
            throw new BadRequestException('Invalid password.');
        }

        if (!$this->user->login($identity)) {
            throw new BadRequestException();
        }

        $identity->resetToken();
        $this->identityRepository->save($identity);

The method save() does not exist on Yiisoft\Auth\IdentityRepositoryInterface. It seems like you code against a sub-type of Yiisoft\Auth\IdentityRepositoryInterface such as App\User\UserRepository.

        return $identity;
    }

    /**
     * @throws BadRequestException
     * @throws Throwable
     */
    public function logout(): void
    {
        if (!$this->user->logout()) {
            throw new BadRequestException();
        }
    }
}