InventoryUpdateWriter::writeItem() - Code Metrics - WeareJH/data-import-magento - Measure and Improve Code Quality continuously with Scrutinizer

InventoryUpdateWriter::writeItem() C
last analyzed 2016-10-31 17:16 UTC

↳ Parent: InventoryUpdateWriter

Complexity

Conditions	11
Paths	57

Size

Total Lines	60
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	60
rs	6.2926
cc	11
eloc	36
nc	57
nop	1

How to fix Long Method Complexity

<?php

namespace Jh\DataImportMagento\Writer;

use Ddeboer\DataImport\Exception\WriterException;
use Ddeboer\DataImport\Writer\AbstractWriter;
use Guzzle\Common\Exception\InvalidArgumentException;
use Jh\DataImportMagento\Exception\MagentoSaveException;
use Jh\DataImportMagento\Options\OptionsParseTrait;

/**
 * Class InventoryUpdateWriter
 * @author Aydin Hassan <[email protected]>
 * @package Jh\DataImportMagento\Writer
 */
class InventoryUpdateWriter extends AbstractWriter
{
    use OptionsParseTrait;

    /**
     * Whether to add qty to existing qty
     */
    const STOCK_UPDATE_TYPE_ADD = 'add';

    /**
     * Whether to set qty and override current
     */
    const STOCK_UPDATE_TYPE_SET = 'set';

    /**
     * @var \Mage_Catalog_Model_Product
     */
    private $productModel;

    /**
     * @var array
     */
    protected $options = [
        'productIdField'                => 'sku',
        'stockUpdateType'               => self::STOCK_UPDATE_TYPE_SET,
        'updateStockStatusIfInStock'    => true
    ];

    /**
     * @param \Mage_Catalog_Model_Product $productModel
     * @param array $options
     */
    public function __construct(
        \Mage_Catalog_Model_Product $productModel,
        array $options = array()
    ) {
        $this->productModel = $productModel;
        $this->setOptions($options);
    }

    /**
     * @param $options
     */
    public function setOptions(array $options)
    {
        if (isset($options['stockUpdateType'])) {
            //check the type passed in, is actually type we support
            if ($options['stockUpdateType'] !== self::STOCK_UPDATE_TYPE_SET
                && $options['stockUpdateType'] !== self::STOCK_UPDATE_TYPE_ADD
            ) {
                throw new \InvalidArgumentException(
                    sprintf("'%s' is not a valid value for 'stockUpdateType'", $options['stockUpdateType'])
                );
            }
        }
        $this->options = $this->parseOptions($this->options, $options);
    }

    /**
     * @return \Ddeboer\DataImport\Writer\WriterInterface
     */
    public function prepare()
    {
        return $this;
    }

    /**
     * @param array $item
     * @return \Ddeboer\DataImport\Writer\WriterInterface
     * @throws \Ddeboer\DataImport\Exception\WriterException
     * @throws \Jh\DataImportMagento\Exception\MagentoSaveException
     */
    public function writeItem(array $item)
    {
        if (!isset($item['product_id'])) {
            throw new WriterException("No product Id Found");
        }

        $id = $item['product_id'];

        if (!isset($item['qty'])) {
            throw new WriterException(
                sprintf('No Quantity found for Product: "%s". Using field "qty"', $id)
            );
        }

        //If Given a sku as the Product ID field, we need to get the product ID
        //from the actual product
        $product = clone $this->productModel;
        switch ($this->options['productIdField']) {
            case 'sku':
                $productId = $product->getIdBySku($id);
                if (!$productId) {
                    throw new WriterException(
                        sprintf('Product not found with SKU: "%s"', $id)
                    );
                }
                break;
            case 'id':
            default:
                //default to assume just using product_id
                $productId = $id;
                break;
        }

        $product->load($productId);
        $stockItem = $product->getData('stock_item');

        switch ($this->options['stockUpdateType']) {
            case self::STOCK_UPDATE_TYPE_ADD:
                $stockItem->setData('qty', $stockItem->getData('qty') + $item['qty']);
                break;
            case self::STOCK_UPDATE_TYPE_SET:
                $stockItem->setData('qty', $item['qty']);
                break;
        }

        if ($this->options['updateStockStatusIfInStock']) {
            // set item to in stock if the new qty matches or is greater than min qty in the config
            if ($item['qty'] >= $stockItem->getMinQty()) {
                $stockItem->setData('is_in_stock', 1);
            }
        }

        try {
            $stockItem->save();
        } catch (\Exception $e) {
            throw new MagentoSaveException($e);
        }

        return $this;
    }
}


1			<?php
2
3			namespace Jh\DataImportMagento\Writer;
4
5			use Ddeboer\DataImport\Exception\WriterException;
6			use Ddeboer\DataImport\Writer\AbstractWriter;
7			use Guzzle\Common\Exception\InvalidArgumentException;
8			use Jh\DataImportMagento\Exception\MagentoSaveException;
9			use Jh\DataImportMagento\Options\OptionsParseTrait;
10
11			/**
12			* Class InventoryUpdateWriter
13			* @author Aydin Hassan <[email protected]>
14			* @package Jh\DataImportMagento\Writer
15			*/
16			class InventoryUpdateWriter extends AbstractWriter
17			{
18			use OptionsParseTrait;
19
20			/**
21			* Whether to add qty to existing qty
22			*/
23			const STOCK_UPDATE_TYPE_ADD = 'add';
24
25			/**
26			* Whether to set qty and override current
27			*/
28			const STOCK_UPDATE_TYPE_SET = 'set';
29
30			/**
31			* @var \Mage_Catalog_Model_Product
32			*/
33			private $productModel;
34
35			/**
36			* @var array
37			*/
38			protected $options = [
39			'productIdField' => 'sku',
40			'stockUpdateType' => self::STOCK_UPDATE_TYPE_SET,
41			'updateStockStatusIfInStock' => true
42			];
43
44			/**
45			* @param \Mage_Catalog_Model_Product $productModel
46			* @param array $options
47			*/
48			public function __construct(
49			\Mage_Catalog_Model_Product $productModel,
50			array $options = array()
51			) {
52			$this->productModel = $productModel;
53			$this->setOptions($options);
54			}
55
56			/**
57			* @param $options
58			*/
59			public function setOptions(array $options)
60			{
61			if (isset($options['stockUpdateType'])) {
62			//check the type passed in, is actually type we support
63			if ($options['stockUpdateType'] !== self::STOCK_UPDATE_TYPE_SET
64			&& $options['stockUpdateType'] !== self::STOCK_UPDATE_TYPE_ADD
65			) {
66			throw new \InvalidArgumentException(
67			sprintf("'%s' is not a valid value for 'stockUpdateType'", $options['stockUpdateType'])
68			);
69			}
70			}
71			$this->options = $this->parseOptions($this->options, $options);
72			}
73
74			/**
75			* @return \Ddeboer\DataImport\Writer\WriterInterface
76			*/
77			public function prepare()
78			{
79			return $this;
80			}
81
82			/**
83			* @param array $item
84			* @return \Ddeboer\DataImport\Writer\WriterInterface
85			* @throws \Ddeboer\DataImport\Exception\WriterException
86			* @throws \Jh\DataImportMagento\Exception\MagentoSaveException
87			*/
88			public function writeItem(array $item)
89			{
90			if (!isset($item['product_id'])) {
91			throw new WriterException("No product Id Found");
92			}
93
94			$id = $item['product_id'];
95
96			if (!isset($item['qty'])) {
97			throw new WriterException(
98			sprintf('No Quantity found for Product: "%s". Using field "qty"', $id)
99			);
100			}
101
102			//If Given a sku as the Product ID field, we need to get the product ID
103			//from the actual product
104			$product = clone $this->productModel;
105			switch ($this->options['productIdField']) {
106			case 'sku':
107			$productId = $product->getIdBySku($id);
108			if (!$productId) {
109			throw new WriterException(
110			sprintf('Product not found with SKU: "%s"', $id)
111			);
112			}
113			break;
114			case 'id':
115			default:
116			//default to assume just using product_id
117			$productId = $id;
118			break;
119			}
120
121			$product->load($productId);
122			$stockItem = $product->getData('stock_item');
123
124			switch ($this->options['stockUpdateType']) {
125			case self::STOCK_UPDATE_TYPE_ADD:
126			$stockItem->setData('qty', $stockItem->getData('qty') + $item['qty']);
127			break;
128			case self::STOCK_UPDATE_TYPE_SET:
129			$stockItem->setData('qty', $item['qty']);
130			break;
131			}
132
133			if ($this->options['updateStockStatusIfInStock']) {
134			// set item to in stock if the new qty matches or is greater than min qty in the config
135			if ($item['qty'] >= $stockItem->getMinQty()) {
136			$stockItem->setData('is_in_stock', 1);
137			}
138			}
139
140			try {
141			$stockItem->save();
142			} catch (\Exception $e) {
143			throw new MagentoSaveException($e);
144			}
145
146			return $this;
147			}
148			}
149

WeareJH / data-import-magento

InventoryUpdateWriter::writeItem() C last analyzed 2016-10-31 17:16 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

InventoryUpdateWriter::writeItem() C
last analyzed 2016-10-31 17:16 UTC