app\modules\admin\widgets\ChangeInfoBox

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	9	3
A	run()	0	8	1

<?php
/**
 * Created for IG Monitoring.
 * User: jakim <pawel@jakimowski.info>
 * Date: 17.02.2018
 */

namespace app\modules\admin\widgets;


use Yii;
use yii\base\Widget;

class ChangeInfoBox extends Widget
{
    public $header = 'Bookmarks';
    public $number = '41';
    public $format = 'integer';

    protected $bgCssClass = 'bg-blue';

    /**
     * @var \app\components\Formatter
     */
    protected $formatter;

    public function init()
    {
        $this->formatter = Yii::$app->formatter;

Yii::app->formatter is of type yii\i18n\Formatter, but the property $formatter was declared to be of type app\components\Formatter. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        parent::init();
        $this->number = $this->formatter->asChange($this->number, false, $this->format);
        if ($this->number > 0) {
            $this->bgCssClass = 'bg-green';
        } elseif ($this->number < 0) {
            $this->bgCssClass = 'bg-red';
        }
    }

    public function run()
    {
        echo "<div class=\"small-box {$this->bgCssClass}\">";
        echo "<div class=\"inner\">";
        echo "<span class=\"info-box-text\">{$this->header}</span>";
        echo "<span class=\"info-box-number\">{$this->number}</span>";
        echo "</div>";
        echo "</div>";
    }
}