Bludata\Framework\Laravel\Application\BaseApplication

Complexity

Total Complexity

20

Size/Duplication

Total Lines	66
Duplicated Lines	42.42 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	routesWithPermission()	0	5	1
F	resourceRoutes()	28	50	18
A	publicRoutes()	0	5	1

<?php

namespace Bludata\Framework\Laravel\Application;

use Closure;
use Illuminate\Foundation\Application;
use Illuminate\Support\Facades\Route;

class BaseApplication extends Application
{
    public function publicRoutes(string $uri, string $controller, Closure $routes = null, array $options = [])
    {
        $options['only'] = $options['only'] ?? ['index', 'show', 'destroyed'];

        return $this->resourceRoutes($uri, $controller, $routes, $options);
    }

    public function routesWithPermission(string $uri, string $controller, Closure $routes = null, array $options = [])
    {
        $options['only'] = $options['only'] ?? ['store', 'update', 'destroyed', 'restoreDestroyed'];

        return $this->resourceRoutes($uri, $controller, $routes, $options);
    }

    public function resourceRoutes(string $uri, string $controller, Closure $routes = null, array $options = [])
    {
        $options['name']   = $options['name']   ?? $uri;
        $options['only']   = $options['only']   ?? [];
        $options['except'] = $options['except'] ?? [];

        $exceptAll = isset($options['except'][0]) && $options['except'][0] == '*';

        if (!$exceptAll) {
            if (in_array('index', $options['only']) && !in_array('index', $options['except'])) {

It seems like $options['only'] can also be of type string; however, parameter $haystack of in_array() does only seem to accept array, maybe add an additional type check?

This code seems to be duplicated across your project.

                Route::get($uri, $controller.'@index')
                     ->name($options['name'].'.index');
            }

            if (in_array('show', $options['only']) && !in_array('show', $options['except'])) {

This code seems to be duplicated across your project.

                Route::get($uri.'/{uuid}', $controller.'@show')
                     ->name($options['name'].'.show');
            }

            if (in_array('store', $options['only']) && !in_array('store', $options['except'])) {

This code seems to be duplicated across your project.

                Route::post($uri, $controller.'@store')
                     ->name($options['name'].'.store');
            }

            if (in_array('update', $options['only']) && !in_array('update', $options['except'])) {

This code seems to be duplicated across your project.

                Route::put($uri.'/{uuid}', $controller.'@update')
                     ->name($options['name'].'.update');
            }

            if (in_array('destroy', $options['only']) && !in_array('destroy', $options['except'])) {

This code seems to be duplicated across your project.

                Route::delete($uri.'/{uuid}', $controller.'@destroy')
                     ->name($options['name'].'.destroy');
            }

            if (in_array('destroyed', $options['only']) && !in_array('destroyed', $options['except'])) {

This code seems to be duplicated across your project.

                Route::get($uri.'/destroyed', $controller.'@destroyed')
                     ->name($options['name'].'.destroyed');
            }

            if (in_array('restoreDestroyed', $options['only']) && !in_array('restoreDestroyed', $options['except'])) {

This code seems to be duplicated across your project.

                Route::post($uri.'/{uuid}', $controller.'@restoreDestroyed')
                     ->name($options['name'].'.restoreDestroyed');
            }
        }

        if ($routes instanceof Closure) {
            $routes($uri, $controller, $options);
        }

        return $this;
    }
}