PasswordResetRequestHandler::getPasswordResetMessage() - Code Metrics - Inspection of "54 add forgot password" - sumocoders/FrameworkMultiUserBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#12)

unknown

created 2016-06-15 08:53 UTC

getPasswordResetMessage() A

↳ Parent: PasswordResetRequestHandler

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	1	Features	1

Metric	Value
c	5
b	1
f	1
dl	0
loc	7
rs	9.4285
cc	1
eloc	4
nc	1
nop	1

<?php

namespace SumoCoders\FrameworkMultiUserBundle\Command;

use SumoCoders\FrameworkMultiUserBundle\User\PasswordReset as UserPasswordReset;
use SumoCoders\FrameworkMultiUserBundle\User\UserRepositoryCollection;
use Swift_Mailer;
use Swift_Message;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
use Symfony\Component\Translation\TranslatorInterface;

class PasswordResetRequestHandler
{
    /**
     * @var UserRepositoryCollection
     */
    private $userRepositoryCollection;

    /**
     * @var Swift_Mailer
     */
    private $mailer;

    /**
     * @var TranslatorInterface
     */
    private $translator;

    /**
     * @var UrlGeneratorInterface
     */
    private $router;

    /**
     * PasswordResetRequestHandler constructor.
     *
     * @param UserRepositoryCollection $userRepositoryCollection
     * @param Swift_Mailer $mailer
     * @param TranslatorInterface $translator
     * @param UrlGeneratorInterface $router
     */
    public function __construct(
        UserRepositoryCollection $userRepositoryCollection,
        Swift_Mailer $mailer,
        TranslatorInterface $translator,
        UrlGeneratorInterface $router
    ) {
        $this->userRepositoryCollection = $userRepositoryCollection;
        $this->mailer = $mailer;
        $this->translator = $translator;
        $this->router = $router;
    }

    /**
     * Creates a password reset token and sends an email to the user.
     *
     * @param PasswordResetRequest $command
     *
     * @return int
     */
    public function handle(PasswordResetRequest $command)
    {
        $user = $command->getUser();
        $user->generatePasswordResetToken();
        $repository = $this->userRepositoryCollection->findRepositoryByClassName(get_class($user));
        $repository->save($user);

        return $this->sendPasswordResetToken($user);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * Sends the password reset token to the user.
     *
     * @param UserPasswordReset $user
     *
     * @return int
     */
    private function sendPasswordResetToken(UserPasswordReset $user)
    {
        $messageBody = $this->getPasswordResetMessage($user);

        $message = Swift_Message::newInstance()
            ->setSubject('Password reset requested')
            ->setFrom('[email protected]')
            ->setTo($user->getEmail())
            ->setBody($messageBody, 'text/plain');

        return $this->mailer->send($message);
    }

    /**
     * Creates the password reset message.
     *
     * @param UserPasswordReset $user
     *
     * @return string
     */
    private function getPasswordResetMessage(UserPasswordReset $user)
    {
        $token = $user->getPasswordResetToken();
        $url = $this->router->generate('multi_user_reset_password', ['token' => $token], true);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        return $this->translator->trans('sumocoders.multiuserbundle.mail.request_password', ['%link%' => $url]);
    }
}


1			<?php
2
3			namespace SumoCoders\FrameworkMultiUserBundle\Command;
4
5			use SumoCoders\FrameworkMultiUserBundle\User\PasswordReset as UserPasswordReset;
6			use SumoCoders\FrameworkMultiUserBundle\User\UserRepositoryCollection;
7			use Swift_Mailer;
8			use Swift_Message;
9			use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
10			use Symfony\Component\Translation\TranslatorInterface;
11
12			class PasswordResetRequestHandler
13			{
14			/**
15			* @var UserRepositoryCollection
16			*/
17			private $userRepositoryCollection;
18
19			/**
20			* @var Swift_Mailer
21			*/
22			private $mailer;
23
24			/**
25			* @var TranslatorInterface
26			*/
27			private $translator;
28
29			/**
30			* @var UrlGeneratorInterface
31			*/
32			private $router;
33
34			/**
35			* PasswordResetRequestHandler constructor.
36			*
37			* @param UserRepositoryCollection $userRepositoryCollection
38			* @param Swift_Mailer $mailer
39			* @param TranslatorInterface $translator
40			* @param UrlGeneratorInterface $router
41			*/
42			public function __construct(
43			UserRepositoryCollection $userRepositoryCollection,
44			Swift_Mailer $mailer,
45			TranslatorInterface $translator,
46			UrlGeneratorInterface $router
47			) {
48			$this->userRepositoryCollection = $userRepositoryCollection;
49			$this->mailer = $mailer;
50			$this->translator = $translator;
51			$this->router = $router;
52			}
53
54			/**
55			* Creates a password reset token and sends an email to the user.
56			*
57			* @param PasswordResetRequest $command
58			*
59			* @return int
60			*/
61			public function handle(PasswordResetRequest $command)
62			{
63			$user = $command->getUser();
64			$user->generatePasswordResetToken();
65			$repository = $this->userRepositoryCollection->findRepositoryByClassName(get_class($user));
66			$repository->save($user);
67
68			return $this->sendPasswordResetToken($user);
			0 ignored issues – show Documentation introduced 2016-06-15 07:06 UTC by Report Bug Copy Issue Report `$user` is of type `object<SumoCoders\Framew...dle\User\UserInterface>`, but the function expects a `object<SumoCoders\Framew...dle\User\PasswordReset>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
69			}
70
71			/**
72			* Sends the password reset token to the user.
73			*
74			* @param UserPasswordReset $user
75			*
76			* @return int
77			*/
78			private function sendPasswordResetToken(UserPasswordReset $user)
79			{
80			$messageBody = $this->getPasswordResetMessage($user);
81
82			$message = Swift_Message::newInstance()
83			->setSubject('Password reset requested')
84			->setFrom('[email protected]')
85			->setTo($user->getEmail())
86			->setBody($messageBody, 'text/plain');
87
88			return $this->mailer->send($message);
89			}
90
91			/**
92			* Creates the password reset message.
93			*
94			* @param UserPasswordReset $user
95			*
96			* @return string
97			*/
98			private function getPasswordResetMessage(UserPasswordReset $user)
99			{
100			$token = $user->getPasswordResetToken();
101			$url = $this->router->generate('multi_user_reset_password', ['token' => $token], true);
			0 ignored issues – show Documentation introduced 2016-06-14 15:12 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `integer`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
102
103			return $this->translator->trans('sumocoders.multiuserbundle.mail.request_password', ['%link%' => $url]);
104			}
105			}
106

sumocoders / FrameworkMultiUserBundle

Pull Request — master (#12)

getPasswordResetMessage() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like