Konekt\Concord\Concerns\HasModuleConfig

Complexity

Total Complexity

6

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	areModelsEnabled()	0	3	1
A	areViewsEnabled()	0	3	1
A	areRoutesEnabled()	0	3	1
A	config()	0	5	2
A	areMigrationsEnabled()	0	3	1

<?php

declare(strict_types=1);

/**
 * Contains the HasModuleConfig trait.
 *
 * @copyright   Copyright (c) 2021 Attila Fulop
 * @author      Attila Fulop
 * @license     MIT
 * @since       2021-11-11
 *
 */

namespace Konekt\Concord\Concerns;

trait HasModuleConfig
{
    public function config(string $key = null, $default = null): mixed
    {
        $key = $key ? sprintf('%s.%s', $this->getId(), $key) : $this->getId();

It seems like getId() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return config($key, $default);
    }

    public function areMigrationsEnabled(): bool
    {
        return (bool) $this->config('migrations', true);
    }

    public function areModelsEnabled(): bool
    {
        return (bool) $this->config('models', true);
    }

    public function areViewsEnabled(): bool
    {
        return (bool) $this->config('views', true);
    }

    public function areRoutesEnabled(): bool
    {
        return (bool) $this->config('routes', true);
    }
}