dev7ch\slick\SlickWidget

Complexity

Total Complexity

3

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

95.24%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	init()	0	29	2
A	run()	0	4	1

<?php

namespace dev7ch\slick;

use dev7ch\slick\assets\PicturefillAsset;
use dev7ch\slick\assets\ResourcesAsset;
use dev7ch\slick\assets\SlickAsset;

\Yii::setAlias('@slick', '@bower/slick-carousel');
\Yii::setAlias('@resources', dirname(__DIR__).'/src/resources');

class SlickWidget extends \luya\base\Widget
{
    public $baseUrl;
    public $images;
    public $slickConfig = [];
    public $slickConfigWidget;

    public function init()
    {
        PicturefillAsset::register($this->getView());
        SlickAsset::register($this->getView());
        ResourcesAsset::register($this->getView());

        if (Module::slickConfig() != false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison !== instead.

            $this->slickConfig = Module::slickConfig();

It seems like \dev7ch\slick\Module::slickConfig() of type boolean is incompatible with the declared type array of property $slickConfig.

        } else {
            $this->slickConfig = $this->slickConfigWidget;
        }

        $this->view->registerJs(
            "var slickSlider = $('.slick-slider').slick({"
                .implode(', ',
                    array_map(
                        function ($config, $option) {
                            return sprintf('%s:%s', $option, $config);
                        },
                        $this->slickConfig, array_keys($this->slickConfig)
                    )
                ).
            '});',
            \luya\web\View::POS_READY,
            'slickSlider'
        );

        parent::init();
    }

    public function run()
    {
        return $this->render('SlickSlider', ['widget' => $this]);
    }
}