HMLB\UserBundle\Handler\RequestPasswordResetHandler

Complexity

Total Complexity

2

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	6	1

<?php

declare (strict_types = 1);

namespace HMLB\UserBundle\Handler;

use HMLB\DDD\Message\Command\Command;
use HMLB\DDD\Message\Command\Handler;
use HMLB\UserBundle\Command\RequestPasswordReset;
use HMLB\UserBundle\User\User;
use HMLB\UserBundle\User\UserRepository;

class RequestPasswordResetHandler implements Handler
{
    /**
     * @var UserRepository
     */
    private $userRepository;

    /**
     * @param UserRepository $userRepository
     */
    public function __construct(UserRepository $userRepository)
    {
        $this->userRepository = $userRepository;
    }

    /**
     * @param Command|RequestPasswordReset $command
     */
    public function handle(Command $command)
    {
        /** @var User $user */
        $user = $this->userRepository->get($command->getUserId());

It seems like you code against a specific sub-type and not the parent class HMLB\DDD\Message\Command\Command as the method getUserId() does only exist in the following sub-classes of HMLB\DDD\Message\Command\Command: HMLB\UserBundle\Command\ChangeEmail, HMLB\UserBundle\Command\ChangePassword, HMLB\UserBundle\Command\ConfirmEmail, HMLB\UserBundle\Command\RequestEmailValidation, HMLB\UserBundle\Command\RequestPasswordReset, HMLB\UserBundle\Command\ResetPassword. Maybe you want to instanceof check for one of these explicitly?

        $user->requestPasswordReset();
    }
}