Doyo\UserBundle\Util\PasswordUpdater::hashPassword()

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	3

Importance

Changes

0

<?php
    public function hashPassword(UserInterface $user)

Missing doc comment for function hashPassword()

    {

Opening brace should be on the same line as the declaration

        $plainPassword = $user->getPlainPassword();
        if (0 === \strlen($plainPassword)) {
            return;
        }

No blank line found after control structure

        $encoder = $this->encoderFactory->getEncoder($user);
        if ($encoder instanceof BCryptPasswordEncoder) {
            $user->setSalt(null);

TRUE, FALSE and NULL should be uppercase as per the configured coding-style; instead of null please use NULL.

        } else {
            $salt = rtrim(str_replace('+', '.', base64_encode(random_bytes(32))), '=');
            $user->setSalt($salt);
        }

No blank line found after control structure

        $hashedPassword = $encoder->encodePassword($plainPassword, $user->getSalt());
        $user->setPassword($hashedPassword);
        $user->eraseCredentials();
    }