BackupGridRow::generateBackupingLink() - Code Metrics - Inspection of "Fixed link to nowhere when backups are disabled fo..." - hiqdev/hipanel-module-hosting - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 287fd3...85b31d )

by Dmitry

created 2016-11-11 12:22 UTC

BackupGridRow::generateBackupingLink() A

↳ Parent: BackupGridRow

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	9
ccs	0
cts	7
cp	0
rs	9.6666
cc	1
eloc	5
nc	1
nop	0
crap	2

<?php

/*
 * Hosting Plugin for HiPanel
 *
 * @link      https://github.com/hiqdev/hipanel-module-hosting
 * @package   hipanel-module-hosting
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2016, HiQDev (http://hiqdev.com/)
 */

namespace hipanel\modules\hosting\widgets\backup;

use hipanel\modules\hosting\models\Backuping;
use hipanel\modules\hosting\models\Hdomain;
use Yii;
use yii\base\Widget;
use yii\helpers\Html;

class BackupGridRow extends Widget
{
    /**
     * @var \hipanel\base\Model
     */
    public $model;

    public function run()
    {
        if ($this->model->backuping_exists) {
            return $this->generateBackupingLink();
        } else {
            return $this->generateBackupingEnableLink();
        }
    }

    protected function getRealObjectId()
    {
        $id = $this->model->id;

        if ($this->model instanceof Hdomain) {
            $id = $this->model->isAlias() ? $this->model->vhost_id : $this->model->id;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        }

        return $id;
    }

    protected function generateBackupingLink()
    {
        $id = $this->getRealObjectId();

        $linkToBackup = Html::a('<i class="fa fa-archive" aria-hidden="true"></i>&nbsp;&nbsp;' .
            Yii::t('hipanel/hosting', 'Backup settings'), ['@backuping/view', 'id' => $id]);

        return $linkToBackup;
    }

    protected function generateBackupingEnableLink()
    {
        $text = '<i class="fa fa-check" aria-hidden="true"></i>&nbsp;&nbsp;';
        $text .= Yii::t('hipanel/hosting', 'Enable backuping');

        $linkToBackup = Html::a($text, ['@hdomain/enable-backuping'], [
            'data-method' => 'POST',
            'data-params' => [
                'Hdomain[id]' => $this->getRealObjectId()
            ]
        ]);

        return $linkToBackup;
    }
}


1			<?php
2
3			/*
4			* Hosting Plugin for HiPanel
5			*
6			* @link https://github.com/hiqdev/hipanel-module-hosting
7			* @package hipanel-module-hosting
8			* @license BSD-3-Clause
9			* @copyright Copyright (c) 2015-2016, HiQDev (http://hiqdev.com/)
10			*/
11
12			namespace hipanel\modules\hosting\widgets\backup;
13
14			use hipanel\modules\hosting\models\Backuping;
15			use hipanel\modules\hosting\models\Hdomain;
16			use Yii;
17			use yii\base\Widget;
18			use yii\helpers\Html;
19
20			class BackupGridRow extends Widget
21			{
22			/**
23			* @var \hipanel\base\Model
24			*/
25			public $model;
26
27			public function run()
28			{
29			if ($this->model->backuping_exists) {
30			return $this->generateBackupingLink();
31			} else {
32			return $this->generateBackupingEnableLink();
33			}
34			}
35
36			protected function getRealObjectId()
37			{
38			$id = $this->model->id;
39
40			if ($this->model instanceof Hdomain) {
41			$id = $this->model->isAlias() ? $this->model->vhost_id : $this->model->id;
			0 ignored issues – show Documentation introduced 2016-07-14 14:55 UTC by Report Bug Copy Issue Report The property `vhost_id` does not exist on `object<hipanel\modules\hosting\models\Hdomain>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text / class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history... Documentation introduced 2016-07-14 14:55 UTC by Report Bug Copy Issue Report The property `id` does not exist on `object<hipanel\modules\hosting\models\Hdomain>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /* * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
42			}
43
44			return $id;
45			}
46
47			protected function generateBackupingLink()
48			{
49			$id = $this->getRealObjectId();
50
51			$linkToBackup = Html::a('<i class="fa fa-archive" aria-hidden="true"></i>  ' .
52			Yii::t('hipanel/hosting', 'Backup settings'), ['@backuping/view', 'id' => $id]);
53
54			return $linkToBackup;
55			}
56
57			protected function generateBackupingEnableLink()
58			{
59			$text = '<i class="fa fa-check" aria-hidden="true"></i>  ';
60			$text .= Yii::t('hipanel/hosting', 'Enable backuping');
61
62			$linkToBackup = Html::a($text, ['@hdomain/enable-backuping'], [
63			'data-method' => 'POST',
64			'data-params' => [
65			'Hdomain[id]' => $this->getRealObjectId()
66			]
67			]);
68
69			return $linkToBackup;
70			}
71			}
72

hiqdev / hipanel-module-hosting

Push — master ( 287fd3...85b31d )

BackupGridRow::generateBackupingLink() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like