CustomerHandler::saveForm() - Code Metrics - Inspection of "Merge remote-tracking branch 'remotes/dev/1.9' int..." - orocrm/crm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 1.9 ( 482137...30c3e5 )

unknown

created 2016-05-04 11:19 UTC

CustomerHandler::saveForm() D

↳ Parent: CustomerHandler

Complexity

Conditions	10
Paths	15

Size

Total Lines	43
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	43
rs	4.8196
cc	10
eloc	23
nc	15
nop	2

How to fix Complexity

<?php

namespace OroCRM\Bundle\MagentoBundle\Form\Handler;

use Symfony\Component\Form\FormInterface;

use Oro\Bundle\FormBundle\Model\UpdateHandler;
use OroCRM\Bundle\MagentoBundle\Entity\Address;
use OroCRM\Bundle\MagentoBundle\Entity\Customer;
use OroCRM\Bundle\MagentoBundle\Service\CustomerStateHandler;

class CustomerHandler extends UpdateHandler
{
    /**
     * @var CustomerStateHandler
     */
    protected $stateHandler;

    /**
     * @param CustomerStateHandler $stateHandler
     * @return CustomerHandler
     */
    public function setStateHandler($stateHandler)
    {
        $this->stateHandler = $stateHandler;

        return $this;
    }

    /**
     * @param Customer $entity
     * @return bool
     */
    public function handleRegister(Customer $entity)
    {
        if ($this->request->getMethod() === 'POST') {
            $manager = $this->doctrineHelper->getEntityManager($entity);
            $entity->setGuest(false);
            $entity->setIsActive(true);
            $this->stateHandler->markCustomerForSync($entity);
            $this->stateHandler->markAddressesForSync($entity);

            $manager->persist($entity);
            $manager->flush();

            return true;
        }

        return false;
    }

    /**
     * {@inheritdoc}
     */
    protected function saveForm(FormInterface $form, $entity)
    {
        if (!$entity instanceof Customer) {
            throw new \InvalidArgumentException('Customer expected');
        }

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

            if ($form->isValid()) {
                $addressesToSync = [];
                if ($entity->getId()) {
                    $this->stateHandler->markCustomerForSync($entity);

                    if (!$entity->getAddresses()->isEmpty()) {
                        foreach ($entity->getAddresses() as $address) {
                            if (!$address->getOriginId()) {
                                $addressesToSync[] = $address;
                            } else {
                                $this->stateHandler->markAddressForSync($address);
                            }
                        }
                    }
                }
                $this->saveEntity($entity);

                // Process trigger listen for update, because create will trigger export during import
                // This will schedule new entity for export
                if (!$entity->getOriginId()) {
                    $this->scheduleCustomerSyncToMagento($entity);
                }

                foreach ($addressesToSync as $address) {
                    $this->scheduleAddressSyncToMagento($address);
                }

                return true;
            }
        }

        return false;
    }

    /**
     * @param object $entity
     */
    protected function saveEntity($entity)
    {
        $manager = $this->doctrineHelper->getEntityManager($entity);
        $manager->persist($entity);

        // flush entity with related entities
        $manager->flush();
    }

    /**
     * @param Customer $entity
     */
    protected function scheduleCustomerSyncToMagento(Customer $entity)
    {
        $this->stateHandler->markCustomerForSync($entity);
        $this->saveEntity($entity);
    }

    /**
     * @param Address $entity
     */
    protected function scheduleAddressSyncToMagento(Address $entity)
    {
        $this->stateHandler->markAddressForSync($entity);
        $this->saveEntity($entity);
    }
}


1			<?php
2
3			namespace OroCRM\Bundle\MagentoBundle\Form\Handler;
4
5			use Symfony\Component\Form\FormInterface;
6
7			use Oro\Bundle\FormBundle\Model\UpdateHandler;
8			use OroCRM\Bundle\MagentoBundle\Entity\Address;
9			use OroCRM\Bundle\MagentoBundle\Entity\Customer;
10			use OroCRM\Bundle\MagentoBundle\Service\CustomerStateHandler;
11
12			class CustomerHandler extends UpdateHandler
13			{
14			/**
15			* @var CustomerStateHandler
16			*/
17			protected $stateHandler;
18
19			/**
20			* @param CustomerStateHandler $stateHandler
21			* @return CustomerHandler
22			*/
23			public function setStateHandler($stateHandler)
24			{
25			$this->stateHandler = $stateHandler;
26
27			return $this;
28			}
29
30			/**
31			* @param Customer $entity
32			* @return bool
33			*/
34			public function handleRegister(Customer $entity)
35			{
36			if ($this->request->getMethod() === 'POST') {
37			$manager = $this->doctrineHelper->getEntityManager($entity);
38			$entity->setGuest(false);
39			$entity->setIsActive(true);
40			$this->stateHandler->markCustomerForSync($entity);
41			$this->stateHandler->markAddressesForSync($entity);
42
43			$manager->persist($entity);
44			$manager->flush();
45
46			return true;
47			}
48
49			return false;
50			}
51
52			/**
53			* {@inheritdoc}
54			*/
55			protected function saveForm(FormInterface $form, $entity)
56			{
57			if (!$entity instanceof Customer) {
58			throw new \InvalidArgumentException('Customer expected');
59			}
60
61			$form->setData($entity);
62			if (in_array($this->request->getMethod(), ['POST', 'PUT'], true)) {
63			$form->submit($this->request);
64
65			if ($form->isValid()) {
66			$addressesToSync = [];
67			if ($entity->getId()) {
68			$this->stateHandler->markCustomerForSync($entity);
69
70			if (!$entity->getAddresses()->isEmpty()) {
71			foreach ($entity->getAddresses() as $address) {
72			if (!$address->getOriginId()) {
73			$addressesToSync[] = $address;
74			} else {
75			$this->stateHandler->markAddressForSync($address);
76			}
77			}
78			}
79			}
80			$this->saveEntity($entity);
81
82			// Process trigger listen for update, because create will trigger export during import
83			// This will schedule new entity for export
84			if (!$entity->getOriginId()) {
85			$this->scheduleCustomerSyncToMagento($entity);
86			}
87
88			foreach ($addressesToSync as $address) {
89			$this->scheduleAddressSyncToMagento($address);
90			}
91
92			return true;
93			}
94			}
95
96			return false;
97			}
98
99			/**
100			* @param object $entity
101			*/
102			protected function saveEntity($entity)
103			{
104			$manager = $this->doctrineHelper->getEntityManager($entity);
105			$manager->persist($entity);
106
107			// flush entity with related entities
108			$manager->flush();
109			}
110
111			/**
112			* @param Customer $entity
113			*/
114			protected function scheduleCustomerSyncToMagento(Customer $entity)
115			{
116			$this->stateHandler->markCustomerForSync($entity);
117			$this->saveEntity($entity);
118			}
119
120			/**
121			* @param Address $entity
122			*/
123			protected function scheduleAddressSyncToMagento(Address $entity)
124			{
125			$this->stateHandler->markAddressForSync($entity);
126			$this->saveEntity($entity);
127			}
128			}
129

orocrm / crm

Push — 1.9 ( 482137...30c3e5 )

CustomerHandler::saveForm() D

Complexity

Size

Duplication

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like