DefaultMagentoImportStrategy::getCustomerSearchContext() - Code Metrics - Inspection of "Merge remote-tracking branch 'remotes/dev/master'" - orocrm/crm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c2123c...66812a )

unknown

created 2016-09-30 06:12 UTC

getCustomerSearchContext() A

↳ Parent: DefaultMagentoImportStrategy

Complexity

Conditions	3
Paths	3

Size

Total Lines	21
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	21
rs	9.3142
c	0
b	0
f	0
cc	3
eloc	11
nc	3
nop	1

<?php

namespace OroCRM\Bundle\MagentoBundle\ImportExport\Strategy;

use Symfony\Component\PropertyAccess\PropertyAccess;
use Symfony\Component\PropertyAccess\PropertyAccessor;

use Oro\Bundle\ImportExportBundle\Strategy\Import\ConfigurableAddOrReplaceStrategy;
use Oro\Bundle\AddressBundle\Entity\Region;

use OroCRM\Bundle\MagentoBundle\Entity\Customer;
use OroCRM\Bundle\MagentoBundle\Entity\Region as MagentoRegion;

class DefaultMagentoImportStrategy extends ConfigurableAddOrReplaceStrategy
{
    /** @var PropertyAccessor */
    protected $propertyAccessor;

    /**
     * @param PropertyAccessor $propertyAccessor
     */
    public function setPropertyAccessor(PropertyAccessor $propertyAccessor)
    {
        $this->propertyAccessor = $propertyAccessor;
    }

    /**
     * {@inheritdoc}
     */
    protected function updateContextCounters($entity)
    {
        // increment context counter
        $identifier = $this->databaseHelper->getIdentifier($entity);
        if ($identifier) {
            $this->context->incrementUpdateCount();
        } else {
            $this->context->incrementAddCount();
        }

        return $entity;
    }

    /**
     * Specify channel as identity field
     *
     * For local entities created from not existing in Magento entities (as guest customer - without originId)
     * should be specified additional identities in appropriate strategy
     *
     * @param string $entityName
     * @param array $identityValues
     * @return null|object
     */
    protected function findEntityByIdentityValues($entityName, array $identityValues)

    {
        if (is_a($entityName, 'OroCRM\Bundle\MagentoBundle\Entity\IntegrationAwareInterface', true)) {
            $identityValues['channel'] = $this->context->getOption('channel');
        }

        return parent::findEntityByIdentityValues($entityName, $identityValues);
    }

    /**
     * Combine channel with identity values for entity search on local new entities storage
     *
     * For local entities created from not existing in Magento entities (as guest customer - without originId)
     * should be configured special identity fields or search context in appropriate strategy
     *
     * @param       $entity
     * @param       $entityClass
     * @param array $searchContext
     *
     * @return array|null
     */
    protected function combineIdentityValues($entity, $entityClass, array $searchContext)

    {
        if (is_a($entityClass, 'OroCRM\Bundle\MagentoBundle\Entity\IntegrationAwareInterface', true)) {
            $searchContext['channel'] = $this->context->getOption('channel');
        }

        return parent::combineIdentityValues($entity, $entityClass, $searchContext);
    }

    /**
     * @return PropertyAccessor
     */
    protected function getPropertyAccessor()
    {
        if (!$this->propertyAccessor) {
            $this->propertyAccessor = PropertyAccess::createPropertyAccessor();
        }
        return $this->propertyAccessor;
    }

    /**
     * Get existing registered customer or existing guest customer
     * Find existing customer using entity data for entities containing customer like Order and Cart
     *
     * @param object $entity
     *
     * @return null|Customer
     */
    protected function findExistingCustomerByContext($entity)
    {
        /** @var Customer|null $existingEntity */
        $searchContext = $this->getEntityCustomerSearchContext($entity);
        $existingEntity = $this->databaseHelper->findOneBy(
            'OroCRM\Bundle\MagentoBundle\Entity\Customer',
            $searchContext
        );

        return $existingEntity;
    }

    /**
     * Get search context for entity customer
     *
     * @param object $entity
     *
     * @return array
     */
    protected function getEntityCustomerSearchContext($entity)
    {
        $customer = $entity->getCustomer();
        if ($customer instanceof Customer) {
            $searchContext = $this->getCustomerSearchContext($customer);
        } else {
            $searchContext = [
                'channel' => $entity->getChannel()
            ];
        }

        return $searchContext;
    }

    /**
     * Get customer search context by channel and website if exists
     *
     * @param Customer $customer
     *
     * @return array
     */
    protected function getCustomerSearchContext(Customer $customer)
    {
        $searchContext = [
            'channel' => $customer->getChannel()
        ];

        if ($customer->getWebsite()) {
            $website = $this->databaseHelper->findOneBy(
                'OroCRM\Bundle\MagentoBundle\Entity\Website',
                [
                    'originId' => $customer->getWebsite()->getOriginId(),
                    'channel' => $customer->getChannel()
                ]
            );
            if ($website) {
                $searchContext['website'] = $website;
            }
        }

        return $searchContext;
    }

    /**
     * Find existing Magento Region entity
     * Find by Code if parameter $regionId not passed
     *
     * @param Region      $entity
     * @param string|null $regionId
     *
     * @return null|MagentoRegion
     */
    protected function findRegionEntity(Region $entity, $regionId = null)
    {
        $existingEntity = null;

        if (!$regionId) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $regionId = $entity->getCode();
        }

        /** @var Region $magentoRegion */
        $magentoRegion = $this->databaseHelper->findOneBy(
            'OroCRM\Bundle\MagentoBundle\Entity\Region',
            [
                'regionId' => $regionId
            ]
        );
        if ($magentoRegion) {
            $existingEntity = $this->databaseHelper->findOneBy(
                'Oro\Bundle\AddressBundle\Entity\Region',
                [
                    'combinedCode' => $magentoRegion->getCombinedCode()
                ]
            );
        }

        return $existingEntity;
    }
}


1		<?php
2
3		namespace OroCRM\Bundle\MagentoBundle\ImportExport\Strategy;
4
5		use Symfony\Component\PropertyAccess\PropertyAccess;
6		use Symfony\Component\PropertyAccess\PropertyAccessor;
7
8		use Oro\Bundle\ImportExportBundle\Strategy\Import\ConfigurableAddOrReplaceStrategy;
9		use Oro\Bundle\AddressBundle\Entity\Region;
10
11		use OroCRM\Bundle\MagentoBundle\Entity\Customer;
12		use OroCRM\Bundle\MagentoBundle\Entity\Region as MagentoRegion;
13
14		class DefaultMagentoImportStrategy extends ConfigurableAddOrReplaceStrategy
15		{
16		/** @var PropertyAccessor */
17		protected $propertyAccessor;
18
19		/**
20		* @param PropertyAccessor $propertyAccessor
21		*/
22		public function setPropertyAccessor(PropertyAccessor $propertyAccessor)
23		{
24		$this->propertyAccessor = $propertyAccessor;
25		}
26
27		/**
28		* {@inheritdoc}
29		*/
30		protected function updateContextCounters($entity)
31		{
32		// increment context counter
33		$identifier = $this->databaseHelper->getIdentifier($entity);
34		if ($identifier) {
35		$this->context->incrementUpdateCount();
36		} else {
37		$this->context->incrementAddCount();
38		}
39
40		return $entity;
41		}
42
43		/**
44		* Specify channel as identity field
45		*
46		* For local entities created from not existing in Magento entities (as guest customer - without originId)
47		* should be specified additional identities in appropriate strategy
48		*
49		* @param string $entityName
50		* @param array $identityValues
51		* @return null\|object
52		*/
53	View Code Duplication	protected function findEntityByIdentityValues($entityName, array $identityValues)
		0 ignored issues – show Duplication introduced 2016-07-08 06:07 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
54		{
55		if (is_a($entityName, 'OroCRM\Bundle\MagentoBundle\Entity\IntegrationAwareInterface', true)) {
56		$identityValues['channel'] = $this->context->getOption('channel');
57		}
58
59		return parent::findEntityByIdentityValues($entityName, $identityValues);
60		}
61
62		/**
63		* Combine channel with identity values for entity search on local new entities storage
64		*
65		* For local entities created from not existing in Magento entities (as guest customer - without originId)
66		* should be configured special identity fields or search context in appropriate strategy
67		*
68		* @param $entity
69		* @param $entityClass
70		* @param array $searchContext
71		*
72		* @return array\|null
73		*/
74	View Code Duplication	protected function combineIdentityValues($entity, $entityClass, array $searchContext)
		0 ignored issues – show Duplication introduced 2016-07-08 06:07 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
75		{
76		if (is_a($entityClass, 'OroCRM\Bundle\MagentoBundle\Entity\IntegrationAwareInterface', true)) {
77		$searchContext['channel'] = $this->context->getOption('channel');
78		}
79
80		return parent::combineIdentityValues($entity, $entityClass, $searchContext);
81		}
82
83		/**
84		* @return PropertyAccessor
85		*/
86		protected function getPropertyAccessor()
87		{
88		if (!$this->propertyAccessor) {
89		$this->propertyAccessor = PropertyAccess::createPropertyAccessor();
90		}
91		return $this->propertyAccessor;
92		}
93
94		/**
95		* Get existing registered customer or existing guest customer
96		* Find existing customer using entity data for entities containing customer like Order and Cart
97		*
98		* @param object $entity
99		*
100		* @return null\|Customer
101		*/
102		protected function findExistingCustomerByContext($entity)
103		{
104		/** @var Customer\|null $existingEntity */
105		$searchContext = $this->getEntityCustomerSearchContext($entity);
106		$existingEntity = $this->databaseHelper->findOneBy(
107		'OroCRM\Bundle\MagentoBundle\Entity\Customer',
108		$searchContext
109		);
110
111		return $existingEntity;
112		}
113
114		/**
115		* Get search context for entity customer
116		*
117		* @param object $entity
118		*
119		* @return array
120		*/
121		protected function getEntityCustomerSearchContext($entity)
122		{
123		$customer = $entity->getCustomer();
124		if ($customer instanceof Customer) {
125		$searchContext = $this->getCustomerSearchContext($customer);
126		} else {
127		$searchContext = [
128		'channel' => $entity->getChannel()
129		];
130		}
131
132		return $searchContext;
133		}
134
135		/**
136		* Get customer search context by channel and website if exists
137		*
138		* @param Customer $customer
139		*
140		* @return array
141		*/
142		protected function getCustomerSearchContext(Customer $customer)
143		{
144		$searchContext = [
145		'channel' => $customer->getChannel()
146		];
147
148		if ($customer->getWebsite()) {
149		$website = $this->databaseHelper->findOneBy(
150		'OroCRM\Bundle\MagentoBundle\Entity\Website',
151		[
152		'originId' => $customer->getWebsite()->getOriginId(),
153		'channel' => $customer->getChannel()
154		]
155		);
156		if ($website) {
157		$searchContext['website'] = $website;
158		}
159		}
160
161		return $searchContext;
162		}
163
164		/**
165		* Find existing Magento Region entity
166		* Find by Code if parameter $regionId not passed
167		*
168		* @param Region $entity
169		* @param string\|null $regionId
170		*
171		* @return null\|MagentoRegion
172		*/
173		protected function findRegionEntity(Region $entity, $regionId = null)
174		{
175		$existingEntity = null;
176
177		if (!$regionId) {
		0 ignored issues – show Bug Best Practice introduced 2016-09-30 06:24 UTC by Report Bug Copy Issue Report The expression `$regionId` of type `string\|null` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
178		$regionId = $entity->getCode();
179		}
180
181		/** @var Region $magentoRegion */
182		$magentoRegion = $this->databaseHelper->findOneBy(
183		'OroCRM\Bundle\MagentoBundle\Entity\Region',
184		[
185		'regionId' => $regionId
186		]
187		);
188		if ($magentoRegion) {
189		$existingEntity = $this->databaseHelper->findOneBy(
190		'Oro\Bundle\AddressBundle\Entity\Region',
191		[
192		'combinedCode' => $magentoRegion->getCombinedCode()
193		]
194		);
195		}
196
197		return $existingEntity;
198		}
199		}
200

orocrm / crm

Push — master ( c2123c...66812a )

getCustomerSearchContext() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like