EloquentSetting - Code Metrics - Inspection of "update testbench to 3.5" - ibrandcc/setting - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( be843f...36b93f )

by shjchen

created 2018-05-04 14:22 UTC

EloquentSetting A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

83.33%

Importance

Changes

Metric	Value
dl	0
loc	66
ccs	20
cts	24
cp	0.8333
rs	10
c	0
b	0
f	0
wmc	8
lcom	1
cbo	1

4 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	getSetting()	9	2
A	allToArray()	9	1
A	setSetting()	18	4

<?php

/*
 * This file is part of ibrand/setting.
 *
 * (c) iBrand <https://www.ibrand.cc>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace iBrand\Component\Setting\Repositories;

use iBrand\Component\Setting\Models\SystemSetting;

/**
 * Class EloquentSetting
 * @package iBrand\Component\Setting\Repositories
 */
class EloquentSetting implements SettingInterface
{
    /**
     * @var SystemSetting
     */
    private $model;

    /**
     * EloquentSetting constructor.
     * @param SystemSetting $model
     */
    public function __construct(SystemSetting $model)
    {
        $this->model = $model;
    }

    /**
     * @param array $settings
     * @return bool
     */
    public function setSetting(array $settings)
    {
        if (!is_array($settings)) {
            return false;
        }

        foreach ($settings as $key => $val) {
            $var = $this->model->where('key', $key)->first();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            if ($var) {
                $var->value = $val;
                $var->save();
            } else {
                $var = $this->model->create(['key' => $key, 'value' => $val]);

            }
        }

        return $var;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
    }

    /**
     * @param $key
     * @return string
     */
    public function getSetting($key)
    {
        $value = $this->model->where('key', $key)->get(['value'])->first();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        if (!is_null($value)) {
            return $value->value;
        }

        return '';
    }

    /**
     * @return array
     */
    public function allToArray()
    {
        $collection = $this->model->all();
        $keyed = $collection->mapWithKeys(function ($item) {
            return [$item->key => $item->value];
        });

        return $keyed->toArray();
    }
}


1		<?php
2
3		/*
4		* This file is part of ibrand/setting.
5		*
6		* (c) iBrand <https://www.ibrand.cc>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace iBrand\Component\Setting\Repositories;
13
14		use iBrand\Component\Setting\Models\SystemSetting;
15
16		/**
17		* Class EloquentSetting
18		* @package iBrand\Component\Setting\Repositories
19		*/
20		class EloquentSetting implements SettingInterface
21		{
22		/**
23		* @var SystemSetting
24		*/
25		private $model;
26
27		/**
28		* EloquentSetting constructor.
29		* @param SystemSetting $model
30		*/
31	6	public function __construct(SystemSetting $model)
32		{
33	6	$this->model = $model;
34	6	}
35
36		/**
37		* @param array $settings
38		* @return bool
39		*/
40	6	public function setSetting(array $settings)
41		{
42	6	if (!is_array($settings)) {
43		return false;
44		}
45
46	6	foreach ($settings as $key => $val) {
47	6	$var = $this->model->where('key', $key)->first();
		0 ignored issues – show Documentation Bug introduced 2018-01-29 10:20 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<iBrand\Component\...g\Models\SystemSetting>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
48	6	if ($var) {
49		$var->value = $val;
50		$var->save();
51		} else {
52	6	$var = $this->model->create(['key' => $key, 'value' => $val]);
		0 ignored issues – show Bug introduced 2018-01-29 10:20 UTC by Report Bug Copy Issue Report The method `create()` does not exist on `iBrand\Component\Setting\Models\SystemSetting`. Did you maybe mean `created()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
53		}
54		}
55
56	6	return $var;
		0 ignored issues – show Bug introduced 2018-01-29 10:20 UTC by Report Bug Copy Issue Report The variable `$var` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
57		}
58
59		/**
60		* @param $key
61		* @return string
62		*/
63	1	public function getSetting($key)
64		{
65	1	$value = $this->model->where('key', $key)->get(['value'])->first();
		0 ignored issues – show Documentation Bug introduced 2018-01-29 10:20 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<iBrand\Component\...g\Models\SystemSetting>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
66	1	if (!is_null($value)) {
67	1	return $value->value;
68		}
69
70		return '';
71		}
72
73		/**
74		* @return array
75		*/
76	2	public function allToArray()
77		{
78	2	$collection = $this->model->all();
79	2	$keyed = $collection->mapWithKeys(function ($item) {
80	2	return [$item->key => $item->value];
81	2	});
82
83	2	return $keyed->toArray();
84		}
85		}
86

ibrandcc / setting

Push — master ( be843f...36b93f )

EloquentSetting A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

4 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like