LizardMedia\CartRuleMobile\Model\Rule\Condition\Mobile

Complexity

Total Complexity

8

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	validate()	0	7	1
A	getInputType()	0	4	1
A	getValueElementType()	0	4	1
A	getValueSelectOptions()	0	10	2
A	loadAttributeOptions()	0	10	1
A	callParentConstructor()	0	4	1

<?php
declare(strict_types=1);

/**
 * File:Mobile.php
 *
 * @author Maciej Sławik <maciej.slawik@lizardmedia.pl>
 * @copyright Copyright (C) 2018 Lizard Media (http://lizardmedia.pl)
 */

namespace LizardMedia\CartRuleMobile\Model\Rule\Condition;

use LizardMedia\CartRuleMobile\Api\MobileDetectorInterface;
use Magento\Config\Model\Config\Source\Yesno;
use Magento\Framework\Model\AbstractModel;
use Magento\Rule\Model\Condition\AbstractCondition;
use Magento\Rule\Model\Condition\Context;

/**
 * Class Mobile
 * @package LizardMedia\CartRuleMobile\Model\Rule\Condition
 */
class Mobile extends AbstractCondition
{
    /**
     * @var Yesno
     */
    protected $sourceYesNo;

    /**
     * @var MobileDetectorInterface
     */
    private $mobileDetector;

    /**
     * Constructor
     * @param Context $context
     * @param Yesno $sourceYesNo
     * @param MobileDetectorInterface $mobileDetector
     * @param array $data
     */
    public function __construct(
        Context $context,
        Yesno $sourceYesNo,
        MobileDetectorInterface $mobileDetector,
        array $data = []
    ) {
        $this->callParentConstructor($context, $data);
        $this->sourceYesNo = $sourceYesNo;
        $this->mobileDetector = $mobileDetector;
    }

    /**
     * @param AbstractModel $model
     * @return bool
     */
    public function validate(AbstractModel $model)
    {
        $enabledOnMobile = (bool)$this->getValue();
        $isMobile = $this->mobileDetector->isMobileDevice();

        return $enabledOnMobile === $isMobile;
    }

    /**
     * @return string
     */
    public function getInputType()
    {
        return 'select';
    }

    /**
     * @return string
     */
    public function getValueElementType()
    {
        return 'select';
    }

    /**
     * @return array|mixed
     */
    public function getValueSelectOptions()
    {
        if (!$this->hasData('value_select_options')) {
            $this->setData(
                'value_select_options',
                $this->sourceYesNo->toOptionArray()
            );
        }
        return $this->getData('value_select_options');
    }

    /**
     * @return Mobile
     */
    public function loadAttributeOptions()
    {
        $this->setData(
            'attribute_option',
            [
                'is_mobile' => __('Mobile device')
            ]
        );
        return $this;
    }


    /**
     * @param Context $context
     * @param array $data
     * @codeCoverageIgnore
     */
    protected function callParentConstructor(Context $context, array $data): void
    {
        parent::__construct($context, $data);

It seems like you call parent on a different method (__construct() instead of callParentConstructor()). Are you sure this is correct? If so, you might want to change this to $this->__construct().

    }
}