SettingRepository::update() - Code Metrics - Inspection of "Add Settings Management in Admin Panel" - XetaIO/Xetaravel - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — settings-management ( 58b7ea )

by Fèvre

created 2022-03-28 00:54 UTC

SettingRepository::update() A

↳ Parent: SettingRepository

Complexity

Conditions	5
Paths	16

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	7
c	1
b	0
f	0
nc	16
nop	2
dl	0
loc	10
rs	9.6111

<?php
namespace Xetaravel\Models\Repositories;

use Illuminate\Support\Str;
use Xetaravel\Models\Setting;

class SettingRepository
{
    /**
     * Create a new setting instance.
     *
     * @param array $data The data used to create the setting.
     *
     * @return \Xetaravel\Models\Setting
     */
    public static function create(array $data): Setting
    {
        return Setting::create([
            'name' => Str::slug($data['name'], '.'),
            'value_int' => $data['type'] ==  'value_int' ? (int) $data['value'] : null,
            'value_str' => $data['type'] ==  'value_str' ? (string) $data['value'] : null,
            'value_bool' => $data['type'] ==  'value_bool' ? (bool) $data['value'] : null,
            'description' => $data['description'],
            'is_deletable' => isset($data['is_deletable']) ? true : false,
        ]);
    }

    /**
     * Update the Setting informations after a valid update request.
     *
     * @param array $data The data used to update the setting.
     * @param \Xetaravel\Models\Setting $setting The setting to update.
     *
     * @return bool
     */
    public static function update(array $data, Setting $setting): bool
    {
        $setting->name = Str::slug($data['name'], '.');

        $setting->value_int = $data['type'] ==  'value_int' ? (int) $data['value'] : null;

        $setting->value_str = $data['type'] ==  'value_str' ? (string) $data['value'] : null;

        $setting->value_bool = $data['type'] ==  'value_bool' ? (bool) $data['value'] : null;

        $setting->description = $data['description'];

        $setting->is_deletable = isset($data['is_deletable']) ? true : false;


        return $setting->save();
    }
}


1			<?php
2			namespace Xetaravel\Models\Repositories;
3
4			use Illuminate\Support\Str;
5			use Xetaravel\Models\Setting;
6
7			class SettingRepository
8			{
9			/**
10			* Create a new setting instance.
11			*
12			* @param array $data The data used to create the setting.
13			*
14			* @return \Xetaravel\Models\Setting
15			*/
16			public static function create(array $data): Setting
17			{
18			return Setting::create([
19			'name' => Str::slug($data['name'], '.'),
20			'value_int' => $data['type'] == 'value_int' ? (int) $data['value'] : null,
21			'value_str' => $data['type'] == 'value_str' ? (string) $data['value'] : null,
22			'value_bool' => $data['type'] == 'value_bool' ? (bool) $data['value'] : null,
23			'description' => $data['description'],
24			'is_deletable' => isset($data['is_deletable']) ? true : false,
25			]);
26			}
27
28			/**
29			* Update the Setting informations after a valid update request.
30			*
31			* @param array $data The data used to update the setting.
32			* @param \Xetaravel\Models\Setting $setting The setting to update.
33			*
34			* @return bool
35			*/
36			public static function update(array $data, Setting $setting): bool
37			{
38			$setting->name = Str::slug($data['name'], '.');
			0 ignored issues – show Bug Best Practice introduced 2022-03-28 01:01 UTC by Report Bug Copy Issue Report The property `name` does not exist on `Xetaravel\Models\Setting`. Since you implemented `__set`, consider adding a @property annotation. Loading history...
39			$setting->value_int = $data['type'] == 'value_int' ? (int) $data['value'] : null;
			0 ignored issues – show Bug introduced 2022-03-28 01:01 UTC by Report Bug Copy Issue Report The property `value_int` does not seem to exist on `Xetaravel\Models\Setting`. Are you sure there is no database migration missing? Checks if undeclared accessed properties appear in database migrations and if the creating migration is correct. Loading history...
40			$setting->value_str = $data['type'] == 'value_str' ? (string) $data['value'] : null;
			0 ignored issues – show Bug introduced 2022-03-28 01:01 UTC by Report Bug Copy Issue Report The property `value_str` does not seem to exist on `Xetaravel\Models\Setting`. Are you sure there is no database migration missing? Checks if undeclared accessed properties appear in database migrations and if the creating migration is correct. Loading history...
41			$setting->value_bool = $data['type'] == 'value_bool' ? (bool) $data['value'] : null;
			0 ignored issues – show Bug introduced 2022-03-28 01:01 UTC by Report Bug Copy Issue Report The property `value_bool` does not seem to exist on `Xetaravel\Models\Setting`. Are you sure there is no database migration missing? Checks if undeclared accessed properties appear in database migrations and if the creating migration is correct. Loading history...
42			$setting->description = $data['description'];
			0 ignored issues – show Bug Best Practice introduced 2022-03-28 01:01 UTC by Report Bug Copy Issue Report The property `description` does not exist on `Xetaravel\Models\Setting`. Since you implemented `__set`, consider adding a @property annotation. Loading history...
43			$setting->is_deletable = isset($data['is_deletable']) ? true : false;
			0 ignored issues – show Bug Best Practice introduced 2022-03-28 01:01 UTC by Report Bug Copy Issue Report The property `is_deletable` does not exist on `Xetaravel\Models\Setting`. Since you implemented `__set`, consider adding a @property annotation. Loading history...
44
45			return $setting->save();
46			}
47			}
48

XetaIO / Xetaravel

Push — settings-management ( 58b7ea )

SettingRepository::update() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like