Application\Api\Field\Mutation\ConfirmRegistration::build()

Complexity

Conditions	4
Paths	1

Size

Total Lines	49
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	11.5287

Importance

Changes

0

<?php
    public static function build(): array
    {
        return [
            'name' => 'confirmRegistration',
            'type' => Type::nonNull(Type::boolean()),
            'description' => 'First step to register as a new user.',
            'args' => [
                'token' => Type::nonNull(_types()->get('Token')),
                'input' => Type::nonNull(_types()->getPartialInput(User::class)),
            ],
            'resolve' => function ($root, array $args, SessionInterface $session): bool {

The parameter $session is not used and could be removed.

                /** @var UserRepository $repository */
                $repository = _em()->getRepository(User::class);

                /** @var User $user */
                $repository->getAclFilter()->setEnabled(false);
                $user = $repository->findOneByToken($args['token']);

The method findOneByToken() does not exist on Application\Repository\UserRepository. Since you implemented __call, consider adding a @method annotation.

                $repository->getAclFilter()->setEnabled(true);

                if (!$user || !$user->isTokenValid()) {
                    throw new Exception('Cannot confirm registration with an invalid token');
                }

                // Do it
                $input = $args['input'];
                Helper::hydrate($user, $input);

                // Active the member
                $user->setRole(User::ROLE_MEMBER);
                $user->setStatus(User::STATUS_ACTIVE);

                // Create mandatory booking for him
                User::setCurrent($user);
                $booking = new Booking();
                _em()->persist($booking);

                $booking->setOwner($user);
                $booking->setStatus(BookingStatusType::BOOKED);
                $booking->setStartDate(new Chronos());

                $mandatoryBookables = _em()->getRepository(Bookable::class)->findByBookingType(BookingTypeType::MANDATORY);

The method findByBookingType() does not exist on Application\Repository\BookableRepository. Since you implemented __call, consider adding a @method annotation.

                foreach ($mandatoryBookables as $bookable) {
                    $booking->addBookable($bookable);
                }

                _em()->flush();

                return true;
            },