SettingsManager::save() - Code Metrics - satrun77/tinyissue - Measure and Improve Code Quality continuously with Scrutinizer

SettingsManager::save() A
last analyzed 2016-08-19 12:52 UTC

↳ Parent: SettingsManager

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	3

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	18
ccs	11
cts	11
cp	1
rs	9.4285
cc	3
eloc	11
nc	3
nop	1
crap	3

<?php

/*
 * This file is part of the Tinyissue package.
 *
 * (c) Mohamed Alsharaf <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tinyissue\Services;

use Illuminate\Support\Collection;
use Tinyissue\Model\Setting;

/**
 * SettingsManager singleton class to maintain a collection of all settings.
 *
 * @author Mohamed Alsharaf <[email protected]>
 *
 * @property int $id
 * @property string $name
 * @property string $value
 * @property string $key
 */
class SettingsManager extends Collection
{
    public function __construct($items = [])
    {
        parent::__construct($items);
        $this->load();
    }

    /**
     * Load all settings.
     *
     * @return \Illuminate\Database\Eloquent\Collection|bool
     */
    protected function load()
    {
        if ($this->count() === 0) {
            // Skip exception if table does not exists
            // This method called from RouteServiceProvider, which is called within command line 'artisan migrate'
            // before the table exists.
            try {
                $items       = Setting::all();
                $this->items = is_array($items) ? $items : $this->getArrayableItems($items);
            } catch (\Exception $e) {
                return false;
            }
        }

        return $this;
    }

    /**
     * Returns a setting value.
     *
     * @param string     $name
     * @param mixed|null $default
     *
     * @return mixed
     */
    public function get($name, $default = null)
    {
        if ($this->load()) {
            foreach ($this->all() as $setting) {
                if ($setting->key === $name) {
                    return $setting->value;
                }
            }
        }

        return $default;
    }

    /**
     * Whether or not the public projects enabled.
     *
     * @return bool
     */
    public function isPublicProjectsEnabled()
    {
        return (boolean) $this->get('enable_public_projects') === true;
    }

    /**
     * Returns date format.
     *
     * @return string
     */
    public function getDateFormat()
    {
        return (string) $this->get('date_format');
    }

    /**
     * Returns first status tag id.
     *
     * @return int
     */
    public function getFirstStatusTagId()
    {
        return (string) $this->get('first_status_tag');
    }

    /**
     * Returns users default language.
     *
     * @return string
     */
    public function getLanguage()
    {
        return (string) $this->get('language');
    }

    /**
     * Save a collection of settings.
     *
     * @param $values
     *
     * @return bool
     */
    public function save($values)
    {
        foreach ($values as $name => $value) {
            $settings = new Setting();
            $setting  = $settings->where('key', '=', $name)->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 ($setting) {
                $setting->value = $value;
                $setting->save();
            }
            unset($settings, $setting);
        }

        // Reload items
        $this->items = [];
        $this->load();

        return true;
    }
}


1		<?php
2
3		/*
4		* This file is part of the Tinyissue package.
5		*
6		* (c) Mohamed Alsharaf <[email protected]>
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 Tinyissue\Services;
13
14		use Illuminate\Support\Collection;
15		use Tinyissue\Model\Setting;
16
17		/**
18		* SettingsManager singleton class to maintain a collection of all settings.
19		*
20		* @author Mohamed Alsharaf <[email protected]>
21		*
22		* @property int $id
23		* @property string $name
24		* @property string $value
25		* @property string $key
26		*/
27		class SettingsManager extends Collection
28		{
29	63	public function __construct($items = [])
30		{
31	63	parent::__construct($items);
32	63	$this->load();
33	63	}
34
35		/**
36		* Load all settings.
37		*
38		* @return \Illuminate\Database\Eloquent\Collection\|bool
39		*/
40	73	protected function load()
41		{
42	73	if ($this->count() === 0) {
43		// Skip exception if table does not exists
44		// This method called from RouteServiceProvider, which is called within command line 'artisan migrate'
45		// before the table exists.
46		try {
47	64	$items = Setting::all();
48	64	$this->items = is_array($items) ? $items : $this->getArrayableItems($items);
49		} catch (\Exception $e) {
50		return false;
51		}
52		}
53
54	73	return $this;
55		}
56
57		/**
58		* Returns a setting value.
59		*
60		* @param string $name
61		* @param mixed\|null $default
62		*
63		* @return mixed
64		*/
65	73	public function get($name, $default = null)
66		{
67	73	if ($this->load()) {
68	73	foreach ($this->all() as $setting) {
69	73	if ($setting->key === $name) {
70	73	return $setting->value;
71		}
72		}
73		}
74
75	1	return $default;
76		}
77
78		/**
79		* Whether or not the public projects enabled.
80		*
81		* @return bool
82		*/
83	63	public function isPublicProjectsEnabled()
84		{
85	63	return (boolean) $this->get('enable_public_projects') === true;
86		}
87
88		/**
89		* Returns date format.
90		*
91		* @return string
92		*/
93	33	public function getDateFormat()
94		{
95	33	return (string) $this->get('date_format');
96		}
97
98		/**
99		* Returns first status tag id.
100		*
101		* @return int
102		*/
103	37	public function getFirstStatusTagId()
104		{
105	37	return (string) $this->get('first_status_tag');
106		}
107
108		/**
109		* Returns users default language.
110		*
111		* @return string
112		*/
113	5	public function getLanguage()
114		{
115	5	return (string) $this->get('language');
116		}
117
118		/**
119		* Save a collection of settings.
120		*
121		* @param $values
122		*
123		* @return bool
124		*/
125	7	public function save($values)
126		{
127	7	foreach ($values as $name => $value) {
128	7	$settings = new Setting();
129	7	$setting = $settings->where('key', '=', $name)->first();
		0 ignored issues – show Documentation Bug introduced 2016-01-11 07:49 UTC by Report Bug Copy Issue Report The method `where` does not exist on `object<Tinyissue\Model\Setting>`? 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...
130	7	if ($setting) {
131	7	$setting->value = $value;
132	7	$setting->save();
133		}
134	7	unset($settings, $setting);
135		}
136
137		// Reload items
138	7	$this->items = [];
139	7	$this->load();
140
141	7	return true;
142		}
143		}
144

satrun77 / tinyissue

SettingsManager::save() A last analyzed 2016-08-19 12:52 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

SettingsManager::save() A
last analyzed 2016-08-19 12:52 UTC