Chuckbe\Chuckcms\Models\Module

Complexity

Total Complexity

9

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Importance

Changes	3
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getSetting()	0	5	2
A	resolveSetting()	0	13	3
A	getSettingsAttribute()	0	15	4

<?php

namespace Chuckbe\Chuckcms\Models;

use Eloquent;

class Module extends Eloquent
{
    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'slug', 'hintpath', 'path', 'type', 'version', 'author', 'json',
    ];

    /**
     * The attributes that are castable.
     *
     * @var array
     */
    protected $casts = [
        'json' => 'array',
    ];

    public function getSettingsAttribute()
    {
        if (array_key_exists('settings', $this->json)) {

The property json does not exist on Chuckbe\Chuckcms\Models\Module. Since you implemented __get, consider adding a @property annotation.

            return $this->json['settings'];
        }

        if (!array_key_exists('admin', $this->json)) {
            return [];
        }

        if (!array_key_exists('settings', $this->json['admin'])) {
            return [];
        }

        return $this->json['admin']['settings'];
    }

    public function getSetting(string $string)
    {
        $setting = $this->resolveSetting($string);

        return !is_null($setting) ? $setting : null;
    }

    private function resolveSetting($var)
    {
        $setting = $this->settings;

The property settings does not exist on Chuckbe\Chuckcms\Models\Module. Since you implemented __get, consider adding a @property annotation.

        $split = explode('.', $var);
        foreach ($split as $value) {
            if (array_key_exists($value, $setting)) {
                $setting = $setting[$value];
            } else {
                return null;
            }
        }

        return $setting;
    }
}