hipanel\modules\server\widgets\Expires

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	init()	0	21	5

<?php
/**
 * Server module for HiPanel
 *
 * @link      https://github.com/hiqdev/hipanel-module-server
 * @package   hipanel-module-server
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2019, HiQDev (http://hiqdev.com/)
 */

/**
 * @see    http://hiqdev.com/hipanel-module-domain
 * @license http://hiqdev.com/hipanel-module-domain/license
 * @copyright Copyright (c) 2015 HiQDev
 */

namespace hipanel\modules\server\widgets;

use hipanel\modules\server\models\Server;
use Yii;

class Expires extends \hipanel\widgets\Label
{
    /**
     * @var Server
     */
    public $model;

    public function init()
    {
        $expires = $this->model->expires;

The property expires does not exist on hipanel\modules\server\models\Server. Since you implemented __get, consider adding a @property annotation.

        $this->label = Yii::t('hipanel:server', '(not sold)');

The property label does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        if (!empty($expires)) {
            if (strtotime($expires) < time()) {
                $class = 'danger';
            } elseif (strtotime($expires) < strtotime('+5 days', time())) {
                $class = 'warning';
            } elseif (strtotime($expires) < strtotime('+30 days', time())) {
                $class = 'info';
            } else {
                $class = 'default';
            }

            $this->color = $class;

The property color does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            $this->label = Yii::$app->formatter->asDate($expires);
        }

        parent::init();
    }
}