Settings::get() - Code Metrics - Inspection of "WIP homepage setting" - thinktomorrow/chief - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — ft/fields-refactor ( 9f047d...e6f9ef )

by Ben

created 2019-07-15 15:10 UTC

Settings::get() A

↳ Parent: Settings

Complexity

Conditions	4
Paths	4

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	8
dl	0
loc	17
rs	10
c	0
b	0
f	0
cc	4
nc	4
nop	3

<?php

namespace Thinktomorrow\Chief\Settings;

use Illuminate\Support\Collection;
use Illuminate\Support\Facades\Schema;

class Settings extends Collection
{
    public static function configValues(): array
    {
        return config('thinktomorrow.chief-settings');
    }

    public function get($key, $locale = null, $default = null)
    {
        $this->fetch();

        if (! isset($this->items[$key])) {
            return $default;
        }
        
        if (is_array($this->items[$key])) {
            if ($this->items[$key]['value'] == null) {
                return $default;
            }

            return $this->items[$key]['value'];
        }

        return $this->items[$key];
    }

    public function set($key, $value)
    {
        $this->items[$key] = $value;
    }

    private function fetch()
    {
        if ($this->items) {

            return;
        }

        $config_values = static::configValues();

        $database_values = Schema::hasTable((new Setting)->getTable())
            ? Setting::all()->pluck('value', 'key')->toArray()
            : [];

        $this->items = array_merge($config_values, $database_values);
    }

    public function fresh()
    {
        $this->items = null;

        return $this;
    }
}


1			<?php
2
3			namespace Thinktomorrow\Chief\Settings;
4
5			use Illuminate\Support\Collection;
6			use Illuminate\Support\Facades\Schema;
7
8			class Settings extends Collection
9			{
10			public static function configValues(): array
11			{
12			return config('thinktomorrow.chief-settings');
13			}
14
15			public function get($key, $locale = null, $default = null)
16			{
17			$this->fetch();
18
19			if (! isset($this->items[$key])) {
20			return $default;
21			}
22
23			if (is_array($this->items[$key])) {
24			if ($this->items[$key]['value'] == null) {
25			return $default;
26			}
27
28			return $this->items[$key]['value'];
29			}
30
31			return $this->items[$key];
32			}
33
34			public function set($key, $value)
35			{
36			$this->items[$key] = $value;
37			}
38
39			private function fetch()
40			{
41			if ($this->items) {
			0 ignored issues – show Bug Best Practice introduced 2019-07-15 20:25 UTC by Report Bug Copy Issue Report The expression `$this->items` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
42			return;
43			}
44
45			$config_values = static::configValues();
46
47			$database_values = Schema::hasTable((new Setting)->getTable())
48			? Setting::all()->pluck('value', 'key')->toArray()
49			: [];
50
51			$this->items = array_merge($config_values, $database_values);
52			}
53
54			public function fresh()
55			{
56			$this->items = null;
57
58			return $this;
59			}
60			}
61

thinktomorrow / chief

Push — ft/fields-refactor ( 9f047d...e6f9ef )

Settings::get() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like