Glorand\Model\Settings\Managers\TableSettingsManager

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	apply()	0	10	2
A	set()	0	6	1
A	delete()	0	14	3

<?php

namespace Glorand\Model\Settings\Managers;

use Glorand\Model\Settings\Contracts\SettingsManagerContract;
use Glorand\Model\Settings\Models\ModelSettings;

/**
 * Class TableSettingsManager
 * @package Glorand\Model\Settings\Managers
 * @property  \Illuminate\Database\Eloquent\Model|\Glorand\Model\Settings\Traits\HasSettingsTable $model
 */
class TableSettingsManager extends AbstractSettingsManager
{
    /**
     * @param array $settings
     * @return \Glorand\Model\Settings\Contracts\SettingsManagerContract
     */
    public function apply(array $settings = []): SettingsManagerContract
    {
        if (!$modelSettings = $this->model->modelSettings()->first()) {
            $modelSettings = new ModelSettings();
            $modelSettings->model()->associate($this->model);

It seems like $this->model can also be of type Glorand\Model\Settings\Traits\HasSettingsTable; however, parameter $model of Illuminate\Database\Eloquent\Relations\MorphTo::associate() does only seem to accept Illuminate\Database\Eloquent\Model, maybe add an additional type check?

        }
        $modelSettings->settings = $settings;
        $modelSettings->save();

        return $this;
    }

    /**
     * @param string|null $path
     * @return \Glorand\Model\Settings\Contracts\SettingsManagerContract
     * @throws \Exception
     */
    public function delete(string $path = null): SettingsManagerContract
    {
        if (!$path) {
            /** @var ModelSettings $modelSettings */
            if ($modelSettings = $this->model->modelSettings()->first()) {
                $modelSettings->delete();
            }
        } else {
            $settings = $this->all();
            array_forget($settings, $path);

The function array_forget() has been deprecated: Arr::forget() should be used directly instead. Will be removed in Laravel 5.9.

            $this->apply($settings);
        }

        return $this;
    }

    /**
     * @param string $path
     * @param mixed $value
     * @return \Glorand\Model\Settings\Contracts\SettingsManagerContract
     */
    public function set(string $path, $value): SettingsManagerContract
    {
        $settings = $this->all();
        array_set($settings, $path, $value);

The function array_set() has been deprecated: Arr::set() should be used directly instead. Will be removed in Laravel 5.9.

        return $this->apply($settings);
    }
}