Bludata\Framework\Laravel\Application\BaseApplication::resourceRoutes()

Complexity

Conditions	18
Paths	516

Size

Total Lines	50
Code Lines	29

Duplication

Lines	28
Ratio	56 %

Importance

Changes

0

<?php
    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;
    }