Tahoe\Bundle\MultiTenancyBundle\Handler\InvitationHandler

Complexity

Total Complexity

9

Size/Duplication

Total Lines	109
Duplicated Lines	23.85 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	2
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	1
A	acceptInvitationById()	13	13	2
A	acceptInvitation()	0	14	1
A	delete()	0	8	2
A	rejectInvitationById()	13	13	2
A	rejectInvitation()	0	6	1

<?php


namespace Tahoe\Bundle\MultiTenancyBundle\Handler;


use Doctrine\ORM\EntityManager;
use Doctrine\ORM\EntityRepository;
use Tahoe\Bundle\MultiTenancyBundle\Factory\InvitationFactory;
use Tahoe\Bundle\MultiTenancyBundle\Model\InvitationInterface;
use Tahoe\Bundle\MultiTenancyBundle\Repository\InvitationRepository;

/**
 * Class InvitationHandler
 *
 * @author Konrad Podgórski <konrad.podgorski@gmail.com>
 */
class InvitationHandler
{

    /**
     * @var EntityManager
     */
    protected $entityManager;
    /**
     * @var InvitationFactory
     */
    protected $invitationFactory;
    /**
     * @var InvitationRepository
     */
    protected $invitationRepository;
    /**
     * @var EntityRepository
     */
    protected $userRepository;
    /**
     * @var TenantUserHandler
     */
    protected $tenantUserHandler;

    /**
     * @param $entityManager
     * @param $invitationFactory
     * @param $invitationRepository
     * @param $tenantUserHandler
     * @param $userRepository
     */
    function __construct(

It is generally recommended to explicitly declare the visibility for methods.

        $entityManager,
        $invitationFactory,
        $invitationRepository,
        $tenantUserHandler,
        $userRepository
    ) {
        $this->entityManager = $entityManager;
        $this->invitationFactory = $invitationFactory;
        $this->invitationRepository = $invitationRepository;
        $this->tenantUserHandler = $tenantUserHandler;
        $this->userRepository = $userRepository;
    }

    public function acceptInvitationById($invitationId)

This method seems to be duplicated in your project.

    {
        $invitation = $this->invitationRepository->find($invitationId);

        if ($invitation === null) {
            throw new \Exception(sprintf(
                'Invitation with id %d doesn\'t exist. Maybe it\'s from different tenant?',
                $invitationId
            ));
        }

        return $this->acceptInvitation($invitation);
    }

    /**
     * @param InvitationInterface $invitation
     *
     * @return \Tahoe\Bundle\MultiTenancyBundle\Entity\TenantUser
     */
    public function acceptInvitation(InvitationInterface $invitation)
    {
        $user = $this->userRepository->findOneBy(array('emailCanonical' => $invitation->getEmail()));

        $tenant = $invitation->getTenant();

        $tenantUser = $this->tenantUserHandler->addUserToTenant($user, $tenant);

$user is of type object|null, but the function expects a object<Tahoe\Bundle\MultiTenancyBundle\Model\MultiTenantUserInterface>.

        $this->entityManager->flush();

        $this->delete($invitation, true);

        return $tenantUser;
    }

    public function delete($invitation, $withFlush = false)
    {
        $this->entityManager->remove($invitation);

        if ($withFlush) {
            $this->entityManager->flush();
        }
    }

    public function rejectInvitationById($invitationId)

This method seems to be duplicated in your project.

    {
        $invitation = $this->invitationRepository->find($invitationId);

        if ($invitation === null) {
            throw new \Exception(sprintf(
                'Invitation with id %d doesn\'t exist. Maybe it\'s from different tenant?',
                $invitationId
            ));
        }

        return $this->rejectInvitation($invitation);
    }

    public function rejectInvitation(InvitationInterface $invitation)
    {
        $this->delete($invitation, true);

        return true;
    }
}