CredentialRegistrationChallenge::process() - Code Metrics - matthewslouismarie/auth-abstractor - Measure and Improve Code Quality continuously with Scrutinizer

CredentialRegistrationChallenge::process() B
last analyzed 2021-04-26 10:30 UTC

↳ Parent: CredentialRegistrationChallenge

Complexity

Conditions	7
Paths	12

Size

Total Lines	80
Code Lines	54

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	7
eloc	54
nc	12
nop	2
dl	0
loc	80
rs	8.0703
c	0
b	0
f	0

How to fix Long Method

<?php

declare(strict_types=1);

namespace LM\AuthAbstractor\Challenge;

use Psr\Http\Message\ServerRequestInterface;
use LM\AuthAbstractor\Configuration\IApplicationConfiguration;
use LM\AuthAbstractor\Implementation\Member;
use LM\AuthAbstractor\Model\AuthenticationProcess;
use LM\AuthAbstractor\Model\IAuthenticationProcess;
use Symfony\Bridge\PsrHttpMessage\Factory\HttpFoundationFactory;
use Symfony\Component\Form\Extension\Core\Type\PasswordType;
use LM\AuthAbstractor\Form\Constraint\ValidNewPassword;
use LM\AuthAbstractor\Implementation\ChallengeResponse;
use LM\AuthAbstractor\Model\IChallengeResponse;
use Symfony\Component\Form\Extension\Core\Type\RepeatedType;
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
use Symfony\Component\Form\FormError;
use Symfony\Component\Form\FormFactoryInterface;
use Symfony\Component\HttpFoundation\Response;
use Twig_Environment;
use Symfony\Component\HttpFoundation\Request;
use LM\AuthAbstractor\Enum\Persistence\Operation;
use LM\Common\Model\ArrayObject;
use LM\AuthAbstractor\Model\PersistOperation;

/**
 * A challenge for registering credentials (username + password) from the user.
 */
class CredentialRegistrationChallenge implements IChallenge
{
    /** @var IApplicationConfiguration */
    private $appConfig;

    /** @var ValidNewPassword */
    private $constraint;

    /** @var FormFactoryInterface */
    private $formFactory;

    /** @var HttpFoundationFactory */
    private $httpFoundationFactory;

    /** @var Twig_Environment */
    private $twig;

    /**
     * @internal
     */
    public function __construct(
        IApplicationConfiguration $appConfig,
        ValidNewPassword $constraint,
        FormFactoryInterface $formFactory,
        HttpFoundationFactory $httpFoundationFactory,
        Twig_Environment $twig
    ) {
        $this->appConfig = $appConfig;
        $this->constraint = $constraint;
        $this->formFactory = $formFactory;
        $this->httpFoundationFactory = $httpFoundationFactory;
        $this->twig = $twig;
    }

    /**
     * @internal
     */
    public function process(
        IAuthenticationProcess $process,
        ?ServerRequestInterface $httpRequest
    ): IChallengeResponse {
        $form = $this
            ->formFactory
            ->createBuilder()
            ->add('username')
            ->add('password', RepeatedType::class, [
                'constraints' => [
                    $this->constraint,
                ],
                'type' => PasswordType::class,
                'invalid_message' => 'The password fields must match.',
                'required' => true,
                'first_options'  => ['label' => 'Password'],
                'second_options' => ['label' => 'Repeat Password'],
            ])
            ->add('submit', SubmitType::class)
            ->getForm()
        ;

        if (null !== $httpRequest) {
            $form->handleRequest($this->httpFoundationFactory->createRequest($httpRequest));
        }

        if ($form->isSubmitted() && $this->appConfig->isExistingMember($form['username']->getData())) {
            $form
                ->get('username')
                ->addError(new FormError('The username is already taken.'))
            ;
        }

        if ($form->isSubmitted() && $form->isValid()) {
            $persistOperations = $process
            $persistOperations = /** @scrutinizer ignore-deprecated */ $process
                ->getTypedMap()
                ->has('persist_operations') ? $process
                ->has('persist_operations') ? /** @scrutinizer ignore-deprecated */ $process
                    ->getTypedMap()
                    ->get('persist_operations', ArrayObject::class)
            : new ArrayObject([], PersistOperation::class);
            : /** @scrutinizer ignore-deprecated */ new ArrayObject([], PersistOperation::class);
            // var_dump(Request::createFromGlobals());
            $member = new Member(
                password_hash($form->get('password')->getData(), PASSWORD_DEFAULT),
                $form->get('username')->getData()
            );
            $newDm = $process
            $newDm = /** @scrutinizer ignore-deprecated */ $process
                ->getTypedMap()
                ->add(
                    'member',
                    $member,
                    Member::class
                )
                ->set(
                    'persist_operations',
                    $persistOperations->add(
                        new PersistOperation($member, new Operation(Operation::CREATE)),
                        PersistOperation::class
                    ),
                    ArrayObject::class
                )
            ;

            return new ChallengeResponse(
                new AuthenticationProcess($newDm),
                null,
                false,
                true
            )
            ;
        }

        $response = new Response($this->twig->render('credential_registration.html.twig', [
            'form' => $form->createView(),
        ]));

        return new ChallengeResponse(
            $process,
            $response,
            false,
            false
        )
        ;
    }
}


matthewslouismarie / auth-abstractor

GitHub Access Token became invalid

CredentialRegistrationChallenge::process() B last analyzed 2021-04-26 10:30 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

CredentialRegistrationChallenge::process() B
last analyzed 2021-04-26 10:30 UTC