OroCRM\Bundle\SalesBundle\Form\Handler\B2bCustomerEmailHandler

Complexity

Total Complexity

12

Size/Duplication

Total Lines	115
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	10

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	13	13	1
C	process()	34	34	7
A	handleDelete()	16	16	3
A	onSuccess()	6	6	1

<?php

namespace OroCRM\Bundle\SalesBundle\Form\Handler;

use Doctrine\ORM\EntityManagerInterface;

use OroCRM\Bundle\SalesBundle\Entity\B2bCustomer;
use OroCRM\Bundle\SalesBundle\Entity\B2bCustomerEmail;
use OroCRM\Bundle\SalesBundle\Validator\B2bCustomerEmailDeleteValidator;
use Symfony\Component\Form\FormInterface;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Security\Core\Exception\AccessDeniedException;

use Oro\Bundle\SoapBundle\Entity\Manager\ApiEntityManager;
use Oro\Bundle\SecurityBundle\SecurityFacade;

class B2bCustomerEmailHandler

This class seems to be duplicated in your project.

{
    /** @var FormInterface */
    protected $form;

    /** @var Request */
    protected $request;

    /** @var EntityManagerInterface */
    protected $manager;

    /** @var  B2bCustomerEmailDeleteValidator */
    protected $b2bCustomerEmailDeleteValidator;

    /** @var SecurityFacade */
    protected $securityFacade;

    /**
     * @param FormInterface $form
     * @param Request $request
     * @param EntityManagerInterface $manager
     * @param B2bCustomerEmailDeleteValidator $b2bCustomerEmailDeleteValidator
     * @param SecurityFacade $securityFacade
     */
    public function __construct(
        FormInterface $form,
        Request $request,

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

        EntityManagerInterface $manager,
        B2bCustomerEmailDeleteValidator $b2bCustomerEmailDeleteValidator,
        SecurityFacade $securityFacade
    ) {
        $this->form    = $form;
        $this->request = $request;
        $this->manager = $manager;
        $this->b2bCustomerEmailDeleteValidator = $b2bCustomerEmailDeleteValidator;
        $this->securityFacade = $securityFacade;
    }

    /**
     * Process form
     *
     * @param B2bCustomerEmail $entity
     *
     * @return bool True on successful processing, false otherwise
     *
     * @throws AccessDeniedException
     */
    public function process(B2bCustomerEmail $entity)
    {
        $this->form->setData($entity);

        $submitData = [
            'email' => $this->request->request->get('email'),
            'primary' => $this->request->request->get('primary')
        ];

        if (in_array($this->request->getMethod(), ['POST', 'PUT'])) {
            $this->form->submit($submitData);

            $b2bCustomerId = $this->request->request->get('entityId');
            if ($this->form->isValid() && $b2bCustomerId) {
                $customer = $this->manager->find(
                    'OroCRMSalesBundle:B2bCustomer',
                    $b2bCustomerId
                );
                if (!$this->securityFacade->isGranted('EDIT', $customer)) {
                    throw new AccessDeniedException();
                }

                if ($customer->getPrimaryEmail() && $this->request->request->get('primary') === true) {
                    return false;
                }

                $this->onSuccess($entity, $customer);

                return true;
            }
        }

        return false;
    }

    /**
     * @param $id
     * @param ApiEntityManager $manager
     * @throws \Exception
     */
    public function handleDelete($id, ApiEntityManager $manager)
    {
        /** @var B2bCustomerEmail $customerEmail */
        $customerEmail = $manager->find($id);
        if (!$this->securityFacade->isGranted('EDIT', $customerEmail->getOwner())) {
            throw new AccessDeniedException();
        }

        if ($this->b2bCustomerEmailDeleteValidator->validate($customerEmail)) {
            $em = $manager->getObjectManager();
            $em->remove($customerEmail);
            $em->flush();
        } else {
            throw new \Exception("orocrm.sales.email.error.delete.more_one", 500);
        }
    }

    /**
     * @param B2bCustomerEmail $entity
     * @param B2bCustomer $customer
     */
    protected function onSuccess(B2bCustomerEmail $entity, B2bCustomer $customer)
    {
        $entity->setOwner($customer);
        $this->manager->persist($entity);
        $this->manager->flush();
    }
}