ThemeSettingsStorage - Code Metrics - Inspection of "Updated config to use latest components" - hiqdev/hipanel-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 01e3d1...48bdd7 )

by Dmitry

created 2016-10-19 12:32 UTC

ThemeSettingsStorage A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	56
ccs	0
cts	30
cp	0
rs	10
c	1
b	0
f	0
wmc	7
lcom	1
cbo	5

5 Methods

Rating	Name	Size	Complexity
A	getStorage()	4	1
A	init()	6	2
A	getCache()	4	1
A	set()	7	1
A	get()	11	2

<?php

namespace hipanel\components;

use hiqdev\thememanager\storage\SettingsStorageInterface;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use Yii;
use yii\base\Component;
use yii\base\Model;
use yii\caching\Cache;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php

class ThemeSettingsStorage extends Component implements SettingsStorageInterface
{
    /**
     * @return SettingsStorage
     */
    protected function getStorage()
    {
        return Yii::$app->get('settingsStorage');
    }

    /**
     * @var string
     */
    public $cacheKey;

    public function init()
    {
        if ($this->cacheKey === null) {
            $this->cacheKey = ['themeSettingsStorage', Yii::$app->user->id];

        }
    }

    /**
     * @return Cache
     */
    protected function getCache()
    {
        return Yii::$app->get('cache');
    }

    /**
     * @inheritdoc
     */
    public function set(Model $model)
    {
        $data = $model->toArray();

        $this->getStorage()->setBounded('theme', $data);
        $this->getCache()->set($this->cacheKey, $data, 86400); // 1 day
    }

    /**
     * @return mixed
     */
    public function get()
    {
        if (($cache = $this->getCache()->get($this->cacheKey)) !== false) {
            return $cache;
        }

        $storage = $this->getStorage()->getBounded('theme');
        $this->getCache()->set($this->cacheKey, $storage, 86400); // 1 day

        return $storage;
    }
}


1			<?php
2
3			namespace hipanel\components;
4
5			use hiqdev\thememanager\storage\SettingsStorageInterface;
			0 ignored issues – show Bug introduced 2016-10-19 12:37 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `hipanel\components\SettingsStorageInterface`. Let’s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let’s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
6			use Yii;
7			use yii\base\Component;
8			use yii\base\Model;
9			use yii\caching\Cache;
			0 ignored issues – show Bug introduced 2016-10-19 12:37 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `hipanel\components\Cache`. Let’s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let’s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
10
11			class ThemeSettingsStorage extends Component implements SettingsStorageInterface
12			{
13			/**
14			* @return SettingsStorage
15			*/
16			protected function getStorage()
17			{
18			return Yii::$app->get('settingsStorage');
19			}
20
21			/**
22			* @var string
23			*/
24			public $cacheKey;
25
26			public function init()
27			{
28			if ($this->cacheKey === null) {
29			$this->cacheKey = ['themeSettingsStorage', Yii::$app->user->id];
			0 ignored issues – show Documentation Bug introduced 2016-10-19 12:37 UTC by Report Bug Copy Issue Report It seems like `array('themeSettingsStor..., \Yii::$app->user->id)` of type `array<integer,?,{"0":"string","1":"?"}>` is incompatible with the declared type `string` of property `$cacheKey`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
30			}
31			}
32
33			/**
34			* @return Cache
35			*/
36			protected function getCache()
37			{
38			return Yii::$app->get('cache');
39			}
40
41			/**
42			* @inheritdoc
43			*/
44			public function set(Model $model)
45			{
46			$data = $model->toArray();
47
48			$this->getStorage()->setBounded('theme', $data);
49			$this->getCache()->set($this->cacheKey, $data, 86400); // 1 day
50			}
51
52			/**
53			* @return mixed
54			*/
55			public function get()
56			{
57			if (($cache = $this->getCache()->get($this->cacheKey)) !== false) {
58			return $cache;
59			}
60
61			$storage = $this->getStorage()->getBounded('theme');
62			$this->getCache()->set($this->cacheKey, $storage, 86400); // 1 day
63
64			return $storage;
65			}
66			}
67

hiqdev / hipanel-core

Push — master ( 01e3d1...48bdd7 )

ThemeSettingsStorage A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

5 Methods

Duplication Side-by-Side

Filter issues like