Products::getPersonalisedProductCollection() - Code Metrics - Inspection of "feature/filter-visibility" - richdynamix/personalised-products - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#18)

by Steven

created 2015-12-20 23:23 UTC

Products::getPersonalisedProductCollection() A

↳ Parent: Products

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	8
rs	9.4286
cc	1
eloc	6
nc	1
nop	1

<?php

namespace Richdynamix\PersonalisedProducts\Model;

use \Magento\Catalog\Model\ProductFactory as ProductFactory;
use \Magento\Catalog\Model\Product\Visibility as Visibility;

/**
 * Loads product collections from predicted product ids
 *
 * @category  Richdynamix
 * @package   PersonalisedProducts
 * @author    Steven Richardson ([email protected]) @mage_gizmo
 */
class Products
{
    /**
     * @var ProductFactory
     */
    private $_productFactory;

    /**
     * Products constructor.
     * @param ProductFactory $productFactory
     * @param Visibility $productVisibility
     */
    public function __construct(ProductFactory $productFactory, Visibility $productVisibility)

    {
        $this->_productFactory = $productFactory;
    }

    /**
     * We only want to show visible and enabled products.
     *
     * @param $personalisedIds
     * @return $this
     */
    public function getPersonalisedProductCollection($personalisedIds)
    {
        $collection = $this->_productFactory->create()->getCollection()
            ->addAttributeToFilter('entity_id', ['in', $personalisedIds])
            ->addAttributeToFilter('visibility', Visibility::VISIBILITY_BOTH)
            ->addAttributeToFilter('status', array('eq' => 1));
        return $collection;
    }
}

1			<?php
2
3			namespace Richdynamix\PersonalisedProducts\Model;
4
5			use \Magento\Catalog\Model\ProductFactory as ProductFactory;
6			use \Magento\Catalog\Model\Product\Visibility as Visibility;
7
8			/**
9			* Loads product collections from predicted product ids
10			*
11			* @category Richdynamix
12			* @package PersonalisedProducts
13			* @author Steven Richardson ([email protected]) @mage_gizmo
14			*/
15			class Products
16			{
17			/**
18			* @var ProductFactory
19			*/
20			private $_productFactory;
21
22			/**
23			* Products constructor.
24			* @param ProductFactory $productFactory
25			* @param Visibility $productVisibility
26			*/
27			public function __construct(ProductFactory $productFactory, Visibility $productVisibility)
			0 ignored issues – show Unused Code introduced 2015-12-20 23:25 UTC by Report Bug Copy Issue Report The parameter `$productVisibility` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
28			{
29			$this->_productFactory = $productFactory;
30			}
31
32			/**
33			* We only want to show visible and enabled products.
34			*
35			* @param $personalisedIds
36			* @return $this
37			*/
38			public function getPersonalisedProductCollection($personalisedIds)
39			{
40			$collection = $this->_productFactory->create()->getCollection()
41			->addAttributeToFilter('entity_id', ['in', $personalisedIds])
42			->addAttributeToFilter('visibility', Visibility::VISIBILITY_BOTH)
43			->addAttributeToFilter('status', array('eq' => 1));
44			return $collection;
45			}
46			}

richdynamix / personalised-products

Pull Request — develop (#18)

Products::getPersonalisedProductCollection() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like