Completed
Push — master ( 01e3d1...48bdd7 )
by Dmitry
04:56
created

ThemeSettingsStorage::getStorage()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 2
nc 1
nop 0
dl 0
loc 4
ccs 0
cts 4
cp 0
crap 2
rs 10
c 1
b 0
f 0
1
<?php
2
3
namespace hipanel\components;
4
5
use hiqdev\thememanager\storage\SettingsStorageInterface;
0 ignored issues
show
Bug introduced by
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 by
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 by
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