BunchSubject::getMediaRoles() - Code Metrics - Inspection of "* Add #PAC-47" - techdivision/import-product - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 22.x ( 69d64a )

by Tim

created 2020-05-19 10:21 UTC

BunchSubject::getMediaRoles() A

↳ Parent: BunchSubject

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	0
cts	2
cp	0
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	0
crap	2

<?php

/**
 * TechDivision\Import\Product\Subjects\BunchSubject
 *
 * NOTICE OF LICENSE
 *
 * This source file is subject to the Open Software License (OSL 3.0)
 * that is available through the world-wide-web at this URL:
 * http://opensource.org/licenses/osl-3.0.php
 *
 * PHP version 5
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2016 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/techdivision/import-product
 * @link      http://www.techdivision.com
 */

namespace TechDivision\Import\Product\Subjects;

use Doctrine\Common\Collections\Collection;
use League\Event\EmitterInterface;
use TechDivision\Import\Loaders\LoaderInterface;
use TechDivision\Import\Services\RegistryProcessorInterface;
use TechDivision\Import\Utils\Generators\GeneratorInterface;
use TechDivision\Import\Utils\StoreViewCodes;
use TechDivision\Import\Product\Utils\MemberNames;
use TechDivision\Import\Product\Utils\RegistryKeys;
use TechDivision\Import\Product\Utils\VisibilityKeys;
use TechDivision\Import\Product\Utils\ConfigurationKeys;
use TechDivision\Import\Subjects\ExportableTrait;
use TechDivision\Import\Subjects\FileUploadTrait;
use TechDivision\Import\Subjects\ExportableSubjectInterface;
use TechDivision\Import\Subjects\FileUploadSubjectInterface;
use TechDivision\Import\Subjects\UrlKeyAwareSubjectInterface;
use TechDivision\Import\Subjects\CleanUpColumnsSubjectInterface;

/**
 * The subject implementation that handles the business logic to persist products.
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2016 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/techdivision/import-product
 * @link      http://www.techdivision.com
 */
class BunchSubject extends AbstractProductSubject implements ExportableSubjectInterface, FileUploadSubjectInterface, UrlKeyAwareSubjectInterface, CleanUpColumnsSubjectInterface
{

    /**
     * The trait that implements the export functionality.
     *
     * @var \TechDivision\Import\Subjects\ExportableTrait
     */
    use ExportableTrait;

    /**
     * The trait that provides file upload functionality.
     *
     * @var \TechDivision\Import\Subjects\FileUploadTrait
     */
    use FileUploadTrait;

    /**
     * The array with the pre-loaded entity IDs.
     *
     * @var array
     */
    protected $preLoadedEntityIds = array();

    /**
     * Mappings for the table column => CSV column header.
     *
     * @var array
     */
    protected $headerStockMappings = array(
        'qty'                         => array('qty', 'float'),
        'min_qty'                     => array('out_of_stock_qty', 'float'),
        'use_config_min_qty'          => array('use_config_min_qty', 'int'),
        'is_qty_decimal'              => array('is_qty_decimal', 'int'),
        'backorders'                  => array('allow_backorders', 'int'),
        'use_config_backorders'       => array('use_config_backorders', 'int'),
        'min_sale_qty'                => array('min_cart_qty', 'float'),
        'use_config_min_sale_qty'     => array('use_config_min_sale_qty', 'int'),
        'max_sale_qty'                => array('max_cart_qty', 'float'),
        'use_config_max_sale_qty'     => array('use_config_max_sale_qty', 'int'),
        'is_in_stock'                 => array('is_in_stock', 'int'),
        'notify_stock_qty'            => array('notify_on_stock_below', 'float'),
        'use_config_notify_stock_qty' => array('use_config_notify_stock_qty', 'int'),
        'manage_stock'                => array('manage_stock', 'int'),
        'use_config_manage_stock'     => array('use_config_manage_stock', 'int'),
        'use_config_qty_increments'   => array('use_config_qty_increments', 'int'),
        'qty_increments'              => array('qty_increments', 'float'),
        'use_config_enable_qty_inc'   => array('use_config_enable_qty_inc', 'int'),
        'enable_qty_increments'       => array('enable_qty_increments', 'int'),
        'is_decimal_divided'          => array('is_decimal_divided', 'int'),
    );

    /**
     * The array with the available visibility keys.
     *
     * @var array
     */
    protected $availableVisibilities = array(
        'Not Visible Individually' => VisibilityKeys::VISIBILITY_NOT_VISIBLE,
        'Catalog'                  => VisibilityKeys::VISIBILITY_IN_CATALOG,
        'Search'                   => VisibilityKeys::VISIBILITY_IN_SEARCH,
        'Catalog, Search'          => VisibilityKeys::VISIBILITY_BOTH
    );

    /**
     * The default callback mappings for the Magento standard product attributes.
     *
     * @var array
     */
    protected $defaultCallbackMappings = array(
        'visibility'           => array('import_product.callback.visibility'),
        'tax_class_id'         => array('import_product.callback.tax.class'),
        'bundle_price_type'    => array('import_product_bundle.callback.bundle.type'),
        'bundle_sku_type'      => array('import_product_bundle.callback.bundle.type'),
        'bundle_weight_type'   => array('import_product_bundle.callback.bundle.type'),
        'bundle_price_view'    => array('import_product_bundle.callback.bundle.price.view'),
        'bundle_shipment_type' => array('import_product_bundle.callback.bundle.shipment.type')
    );

    /**
     * The available entity types.
     *
     * @var array
     */
    protected $entityTypes = array();

    /**
     * The media roles loader instance.
     *
     * @var \TechDivision\Import\Loaders\LoaderInterface
     */
    protected $mediaRolesLoader;

    /**
     * BunchSubject constructor
     *
     * @param RegistryProcessorInterface $registryProcessor          The registry processor instance
     * @param GeneratorInterface         $coreConfigDataUidGenerator The generator instance
     * @param Collection                 $systemLoggers              The system logger collection
     * @param EmitterInterface           $emitter                    The emitter instance
     * @param LoaderInterface            $loader                     The media type loader instance
     */
    public function __construct(
        RegistryProcessorInterface $registryProcessor,
        GeneratorInterface $coreConfigDataUidGenerator,
        Collection $systemLoggers,
        EmitterInterface $emitter,
        LoaderInterface $loader
    ) {

        // set the loader for the media roles
        $this->mediaRolesLoader = $loader;

        // pass the other instances to the parent constructor
        parent::__construct($registryProcessor, $coreConfigDataUidGenerator, $systemLoggers, $emitter);
    }

    /**
     * Intializes the previously loaded global data for exactly one bunch.
     *
     * @param string $serial The serial of the actual import
     *
     * @return void
     */
    public function setUp($serial)
    {

        // load the status of the actual import
        $status = $this->getRegistryProcessor()->getAttribute(RegistryKeys::STATUS);

        // load the global data we've prepared initially
        $this->entityTypes = $status[RegistryKeys::GLOBAL_DATA][RegistryKeys::ENTITY_TYPES];

        // initialize the flag whether to copy images or not
        if ($this->getConfiguration()->hasParam(ConfigurationKeys::COPY_IMAGES)) {
            $this->setCopyImages($this->getConfiguration()->getParam(ConfigurationKeys::COPY_IMAGES));
        }

        // initialize media directory => can be absolute or relative
        if ($this->getConfiguration()->hasParam(ConfigurationKeys::MEDIA_DIRECTORY)) {

            try {
                $this->setMediaDir($this->resolvePath($this->getConfiguration()->getParam(ConfigurationKeys::MEDIA_DIRECTORY)));
            } catch (\InvalidArgumentException $iae) {
                $this->getSystemLogger()->warning($iae);
            }
        }

        // initialize images directory => can be absolute or relative
        if ($this->getConfiguration()->hasParam(ConfigurationKeys::IMAGES_FILE_DIRECTORY)) {

            try {
                $this->setImagesFileDir($this->resolvePath($this->getConfiguration()->getParam(ConfigurationKeys::IMAGES_FILE_DIRECTORY)));
            } catch (\InvalidArgumentException $iae) {
                $this->getSystemLogger()->warning($iae);
            }
        }

        // invoke the parent method
        parent::setUp($serial);
    }

    /**
     * Clean up the global data after importing the bunch.
     *
     * @param string $serial The serial of the actual import
     *
     * @return void
     */
    public function tearDown($serial)
    {

        // invoke the parent method
        parent::tearDown($serial);

        // load the registry processor
        $registryProcessor = $this->getRegistryProcessor();

        // update the status
        $registryProcessor->mergeAttributesRecursive(
            RegistryKeys::STATUS,
            array(
                RegistryKeys::PRE_LOADED_ENTITY_IDS => $this->preLoadedEntityIds,
            )
        );
    }

    /**
     * Return's the default callback mappings.
     *
     * @return array The default callback mappings
     */
    public function getDefaultCallbackMappings()
    {
        return $this->defaultCallbackMappings;
    }

    /**
     * Return's the mappings for the table column => CSV column header.
     *
     * @return array The header stock mappings
     */
    public function getHeaderStockMappings()
    {
        return $this->headerStockMappings;
    }

    /**
     * Return's the visibility key for the passed visibility string.
     *
     * @param string $visibility The visibility string to return the key for
     *
     * @return integer The requested visibility key
     * @throws \Exception Is thrown, if the requested visibility is not available
     */
    public function getVisibilityIdByValue($visibility)
    {

        // query whether or not, the requested visibility is available
        if (isset($this->availableVisibilities[$visibility])) {
            // load the visibility ID, add the mapping and return the ID
            return $this->availableVisibilities[$visibility];
        }

        // throw an exception, if not
        throw new \Exception(
            $this->appendExceptionSuffix(
                sprintf('Found invalid visibility %s', $visibility)
            )
        );
    }

    /**
     * Pre-load the entity ID for the passed product.
     *
     * @param array $product The product to be pre-loaded
     *
     * @return void
     */
    public function preLoadEntityId(array $product)
    {
        $this->preLoadedEntityIds[$product[MemberNames::SKU]] = $product[MemberNames::ENTITY_ID];
    }

    /**
     * Return's the entity type for the configured entity type code.
     *
     * @return array The requested entity type
     * @throws \Exception Is thrown, if the requested entity type is not available
     */
    public function getEntityType()

    {

        // query whether or not the entity type with the passed code is available
        if (isset($this->entityTypes[$entityTypeCode = $this->getEntityTypeCode()])) {
            return $this->entityTypes[$entityTypeCode];
        }

        // throw a new exception
        throw new \Exception(
            $this->appendExceptionSuffix(
                sprintf('Requested entity type "%s" is not available', $entityTypeCode)
            )
        );
    }

    /**
     * Return's TRUE, if the passed URL key varchar value IS related with the actual PK.
     *
     * @param array $productVarcharAttribute The varchar value to check
     *
     * @return boolean TRUE if the URL key is related, else FALSE
     */
    public function isUrlKeyOf(array $productVarcharAttribute)
    {
        return ((integer) $productVarcharAttribute[MemberNames::ENTITY_ID] === (integer) $this->getLastEntityId()) &&
               ((integer) $productVarcharAttribute[MemberNames::STORE_ID] === (integer) $this->getRowStoreId(StoreViewCodes::ADMIN));
    }

    /**
     * Loads and returns the media roles.
     *
     * @return array The array with the media roles
     */
    public function getMediaRoles(): array
    {
        return $this->mediaRolesLoader->load();
    }

    /**
     * Merge the columns from the configuration with all image type columns to define which
     * columns should be cleaned-up.
     *
     * @return array The columns that has to be cleaned-up
     */
    public function getCleanUpColumns()
    {

        // initialize the array for the columns that has to be cleaned-up
        $cleanUpColumns = array();


        // query whether or not an array has been specified in the configuration
        if ($this->getConfiguration()->hasParam(ConfigurationKeys::CLEAN_UP_EMPTY_COLUMNS)) {
            $cleanUpColumns = $this->getConfiguration()->getParam(ConfigurationKeys::CLEAN_UP_EMPTY_COLUMNS);
        }

        // return the array with the column names
        return $cleanUpColumns;
    }
}


1			<?php
2
3			/**
4			* TechDivision\Import\Product\Subjects\BunchSubject
5			*
6			* NOTICE OF LICENSE
7			*
8			* This source file is subject to the Open Software License (OSL 3.0)
9			* that is available through the world-wide-web at this URL:
10			* http://opensource.org/licenses/osl-3.0.php
11			*
12			* PHP version 5
13			*
14			* @author Tim Wagner <[email protected]>
15			* @copyright 2016 TechDivision GmbH <[email protected]>
16			* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
17			* @link https://github.com/techdivision/import-product
18			* @link http://www.techdivision.com
19			*/
20
21			namespace TechDivision\Import\Product\Subjects;
22
23			use Doctrine\Common\Collections\Collection;
24			use League\Event\EmitterInterface;
25			use TechDivision\Import\Loaders\LoaderInterface;
26			use TechDivision\Import\Services\RegistryProcessorInterface;
27			use TechDivision\Import\Utils\Generators\GeneratorInterface;
28			use TechDivision\Import\Utils\StoreViewCodes;
29			use TechDivision\Import\Product\Utils\MemberNames;
30			use TechDivision\Import\Product\Utils\RegistryKeys;
31			use TechDivision\Import\Product\Utils\VisibilityKeys;
32			use TechDivision\Import\Product\Utils\ConfigurationKeys;
33			use TechDivision\Import\Subjects\ExportableTrait;
34			use TechDivision\Import\Subjects\FileUploadTrait;
35			use TechDivision\Import\Subjects\ExportableSubjectInterface;
36			use TechDivision\Import\Subjects\FileUploadSubjectInterface;
37			use TechDivision\Import\Subjects\UrlKeyAwareSubjectInterface;
38			use TechDivision\Import\Subjects\CleanUpColumnsSubjectInterface;
39
40			/**
41			* The subject implementation that handles the business logic to persist products.
42			*
43			* @author Tim Wagner <[email protected]>
44			* @copyright 2016 TechDivision GmbH <[email protected]>
45			* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
46			* @link https://github.com/techdivision/import-product
47			* @link http://www.techdivision.com
48			*/
49			class BunchSubject extends AbstractProductSubject implements ExportableSubjectInterface, FileUploadSubjectInterface, UrlKeyAwareSubjectInterface, CleanUpColumnsSubjectInterface
50			{
51
52			/**
53			* The trait that implements the export functionality.
54			*
55			* @var \TechDivision\Import\Subjects\ExportableTrait
56			*/
57			use ExportableTrait;
58
59			/**
60			* The trait that provides file upload functionality.
61			*
62			* @var \TechDivision\Import\Subjects\FileUploadTrait
63			*/
64			use FileUploadTrait;
65
66			/**
67			* The array with the pre-loaded entity IDs.
68			*
69			* @var array
70			*/
71			protected $preLoadedEntityIds = array();
72
73			/**
74			* Mappings for the table column => CSV column header.
75			*
76			* @var array
77			*/
78			protected $headerStockMappings = array(
79			'qty' => array('qty', 'float'),
80			'min_qty' => array('out_of_stock_qty', 'float'),
81			'use_config_min_qty' => array('use_config_min_qty', 'int'),
82			'is_qty_decimal' => array('is_qty_decimal', 'int'),
83			'backorders' => array('allow_backorders', 'int'),
84			'use_config_backorders' => array('use_config_backorders', 'int'),
85			'min_sale_qty' => array('min_cart_qty', 'float'),
86			'use_config_min_sale_qty' => array('use_config_min_sale_qty', 'int'),
87			'max_sale_qty' => array('max_cart_qty', 'float'),
88			'use_config_max_sale_qty' => array('use_config_max_sale_qty', 'int'),
89			'is_in_stock' => array('is_in_stock', 'int'),
90			'notify_stock_qty' => array('notify_on_stock_below', 'float'),
91			'use_config_notify_stock_qty' => array('use_config_notify_stock_qty', 'int'),
92			'manage_stock' => array('manage_stock', 'int'),
93			'use_config_manage_stock' => array('use_config_manage_stock', 'int'),
94			'use_config_qty_increments' => array('use_config_qty_increments', 'int'),
95			'qty_increments' => array('qty_increments', 'float'),
96			'use_config_enable_qty_inc' => array('use_config_enable_qty_inc', 'int'),
97			'enable_qty_increments' => array('enable_qty_increments', 'int'),
98			'is_decimal_divided' => array('is_decimal_divided', 'int'),
99			);
100
101			/**
102			* The array with the available visibility keys.
103			*
104			* @var array
105			*/
106			protected $availableVisibilities = array(
107			'Not Visible Individually' => VisibilityKeys::VISIBILITY_NOT_VISIBLE,
108			'Catalog' => VisibilityKeys::VISIBILITY_IN_CATALOG,
109			'Search' => VisibilityKeys::VISIBILITY_IN_SEARCH,
110			'Catalog, Search' => VisibilityKeys::VISIBILITY_BOTH
111			);
112
113			/**
114			* The default callback mappings for the Magento standard product attributes.
115			*
116			* @var array
117			*/
118			protected $defaultCallbackMappings = array(
119			'visibility' => array('import_product.callback.visibility'),
120			'tax_class_id' => array('import_product.callback.tax.class'),
121			'bundle_price_type' => array('import_product_bundle.callback.bundle.type'),
122			'bundle_sku_type' => array('import_product_bundle.callback.bundle.type'),
123			'bundle_weight_type' => array('import_product_bundle.callback.bundle.type'),
124			'bundle_price_view' => array('import_product_bundle.callback.bundle.price.view'),
125			'bundle_shipment_type' => array('import_product_bundle.callback.bundle.shipment.type')
126			);
127
128			/**
129			* The available entity types.
130			*
131			* @var array
132			*/
133			protected $entityTypes = array();
134
135			/**
136			* The media roles loader instance.
137			*
138			* @var \TechDivision\Import\Loaders\LoaderInterface
139			*/
140			protected $mediaRolesLoader;
141
142			/**
143			* BunchSubject constructor
144			*
145			* @param RegistryProcessorInterface $registryProcessor The registry processor instance
146			* @param GeneratorInterface $coreConfigDataUidGenerator The generator instance
147			* @param Collection $systemLoggers The system logger collection
148			* @param EmitterInterface $emitter The emitter instance
149			* @param LoaderInterface $loader The media type loader instance
150			*/
151	18		public function __construct(
152			RegistryProcessorInterface $registryProcessor,
153			GeneratorInterface $coreConfigDataUidGenerator,
154			Collection $systemLoggers,
155			EmitterInterface $emitter,
156			LoaderInterface $loader
157			) {
158
159			// set the loader for the media roles
160	18		$this->mediaRolesLoader = $loader;
161
162			// pass the other instances to the parent constructor
163	18		parent::__construct($registryProcessor, $coreConfigDataUidGenerator, $systemLoggers, $emitter);
164	18		}
165
166			/**
167			* Intializes the previously loaded global data for exactly one bunch.
168			*
169			* @param string $serial The serial of the actual import
170			*
171			* @return void
172			*/
173	18		public function setUp($serial)
174			{
175
176			// load the status of the actual import
177	18		$status = $this->getRegistryProcessor()->getAttribute(RegistryKeys::STATUS);
178
179			// load the global data we've prepared initially
180	18		$this->entityTypes = $status[RegistryKeys::GLOBAL_DATA][RegistryKeys::ENTITY_TYPES];
181
182			// initialize the flag whether to copy images or not
183	18		if ($this->getConfiguration()->hasParam(ConfigurationKeys::COPY_IMAGES)) {
184			$this->setCopyImages($this->getConfiguration()->getParam(ConfigurationKeys::COPY_IMAGES));
185			}
186
187			// initialize media directory => can be absolute or relative
188	18	View Code Duplication	if ($this->getConfiguration()->hasParam(ConfigurationKeys::MEDIA_DIRECTORY)) {
			0 ignored issues – show Duplication introduced 2020-05-15 17:32 UTC by Report Bug Copy Issue Report This code seems to be duplicated across 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...
189			try {
190			$this->setMediaDir($this->resolvePath($this->getConfiguration()->getParam(ConfigurationKeys::MEDIA_DIRECTORY)));
191			} catch (\InvalidArgumentException $iae) {
192			$this->getSystemLogger()->warning($iae);
193			}
194			}
195
196			// initialize images directory => can be absolute or relative
197	18	View Code Duplication	if ($this->getConfiguration()->hasParam(ConfigurationKeys::IMAGES_FILE_DIRECTORY)) {
			0 ignored issues – show Duplication introduced 2020-05-15 17:32 UTC by Report Bug Copy Issue Report This code seems to be duplicated across 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...
198			try {
199			$this->setImagesFileDir($this->resolvePath($this->getConfiguration()->getParam(ConfigurationKeys::IMAGES_FILE_DIRECTORY)));
200			} catch (\InvalidArgumentException $iae) {
201			$this->getSystemLogger()->warning($iae);
202			}
203			}
204
205			// invoke the parent method
206	18		parent::setUp($serial);
207	18		}
208
209			/**
210			* Clean up the global data after importing the bunch.
211			*
212			* @param string $serial The serial of the actual import
213			*
214			* @return void
215			*/
216			public function tearDown($serial)
217			{
218
219			// invoke the parent method
220			parent::tearDown($serial);
221
222			// load the registry processor
223			$registryProcessor = $this->getRegistryProcessor();
224
225			// update the status
226			$registryProcessor->mergeAttributesRecursive(
227			RegistryKeys::STATUS,
228			array(
229			RegistryKeys::PRE_LOADED_ENTITY_IDS => $this->preLoadedEntityIds,
230			)
231			);
232			}
233
234			/**
235			* Return's the default callback mappings.
236			*
237			* @return array The default callback mappings
238			*/
239			public function getDefaultCallbackMappings()
240			{
241			return $this->defaultCallbackMappings;
242			}
243
244			/**
245			* Return's the mappings for the table column => CSV column header.
246			*
247			* @return array The header stock mappings
248			*/
249	1		public function getHeaderStockMappings()
250			{
251	1		return $this->headerStockMappings;
252			}
253
254			/**
255			* Return's the visibility key for the passed visibility string.
256			*
257			* @param string $visibility The visibility string to return the key for
258			*
259			* @return integer The requested visibility key
260			* @throws \Exception Is thrown, if the requested visibility is not available
261			*/
262			public function getVisibilityIdByValue($visibility)
263			{
264
265			// query whether or not, the requested visibility is available
266			if (isset($this->availableVisibilities[$visibility])) {
267			// load the visibility ID, add the mapping and return the ID
268			return $this->availableVisibilities[$visibility];
269			}
270
271			// throw an exception, if not
272			throw new \Exception(
273			$this->appendExceptionSuffix(
274			sprintf('Found invalid visibility %s', $visibility)
275			)
276			);
277			}
278
279			/**
280			* Pre-load the entity ID for the passed product.
281			*
282			* @param array $product The product to be pre-loaded
283			*
284			* @return void
285			*/
286			public function preLoadEntityId(array $product)
287			{
288			$this->preLoadedEntityIds[$product[MemberNames::SKU]] = $product[MemberNames::ENTITY_ID];
289			}
290
291			/**
292			* Return's the entity type for the configured entity type code.
293			*
294			* @return array The requested entity type
295			* @throws \Exception Is thrown, if the requested entity type is not available
296			*/
297	1	View Code Duplication	public function getEntityType()
			0 ignored issues – show Duplication introduced 2017-08-07 17:20 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...
298			{
299
300			// query whether or not the entity type with the passed code is available
301	1		if (isset($this->entityTypes[$entityTypeCode = $this->getEntityTypeCode()])) {
302	1		return $this->entityTypes[$entityTypeCode];
303			}
304
305			// throw a new exception
306			throw new \Exception(
307			$this->appendExceptionSuffix(
308			sprintf('Requested entity type "%s" is not available', $entityTypeCode)
309			)
310			);
311			}
312
313			/**
314			* Return's TRUE, if the passed URL key varchar value IS related with the actual PK.
315			*
316			* @param array $productVarcharAttribute The varchar value to check
317			*
318			* @return boolean TRUE if the URL key is related, else FALSE
319			*/
320			public function isUrlKeyOf(array $productVarcharAttribute)
321			{
322			return ((integer) $productVarcharAttribute[MemberNames::ENTITY_ID] === (integer) $this->getLastEntityId()) &&
323			((integer) $productVarcharAttribute[MemberNames::STORE_ID] === (integer) $this->getRowStoreId(StoreViewCodes::ADMIN));
324			}
325
326			/**
327			* Loads and returns the media roles.
328			*
329			* @return array The array with the media roles
330			*/
331			public function getMediaRoles(): array
332			{
333			return $this->mediaRolesLoader->load();
334			}
335
336			/**
337			* Merge the columns from the configuration with all image type columns to define which
338			* columns should be cleaned-up.
339			*
340			* @return array The columns that has to be cleaned-up
341			*/
342			public function getCleanUpColumns()
343			{
344
345			// initialize the array for the columns that has to be cleaned-up
346			$cleanUpColumns = array();
			0 ignored issues – show Bug Compatibility introduced 2020-05-19 10:23 UTC by Report Bug Copy Issue Report The expression `array();` of type `array` adds the type `array` to the return on line 354 which is incompatible with the return type of the parent method `TechDivision\Import\Prod...ject::getCleanUpColumns` of type `string`. Loading history...
347
348			// query whether or not an array has been specified in the configuration
349			if ($this->getConfiguration()->hasParam(ConfigurationKeys::CLEAN_UP_EMPTY_COLUMNS)) {
350			$cleanUpColumns = $this->getConfiguration()->getParam(ConfigurationKeys::CLEAN_UP_EMPTY_COLUMNS);
351			}
352
353			// return the array with the column names
354			return $cleanUpColumns;
355			}
356			}
357

techdivision / import-product

Push — 22.x ( 69d64a )

BunchSubject::getMediaRoles() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like