Konekt\Concord\Concerns\LoadsSubmodules

Complexity

Total Complexity

6

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getDefaultSubModuleConfiguration()	0	7	1
A	loadSubModules()	0	15	5

<?php

declare(strict_types=1);

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

namespace Konekt\Concord\Concerns;

use JetBrains\PhpStorm\ArrayShape;

trait LoadsSubmodules
{
    private function loadSubModules(array $modules): void
    {
        foreach ($modules as $module => $configuration) {
            if (is_int($module) && is_string($configuration)) { // means no configuration was set for module
                $module = $configuration;
                $configuration = $this->getDefaultSubModuleConfiguration();
            } else {
                $configuration = array_merge(
                    $this->getDefaultSubModuleConfiguration(),
                    is_array($configuration) ? $configuration : []
                );
                $configuration = array_replace_recursive($configuration, $this->getCascadeModuleConfiguration());

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

            }

            $this->concord->registerModule($module, $configuration);
        }
    }

    #[ArrayShape(['implicit' => "bool", 'migrations' => "mixed", 'views' => "mixed", 'routes' => "mixed"])] private function getDefaultSubModuleConfiguration(): array
    {
        return [
            'implicit' => true,
            'migrations' => $this->areMigrationsEnabled(),

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

            'views' => $this->areViewsEnabled(),

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

            'routes' => $this->areRoutesEnabled()

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

        ];
    }
}