Bludata\Lumen\Application\BaseApplication

Complexity

Total Complexity

11

Size/Duplication

Total Lines	83
Duplicated Lines	19.28 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
	getRepositoryInterface()	0	1	?
A	getCurrentUser()	0	4	1
A	setCurrentUser()	0	6	1
C	resource()	16	61	9

<?php

namespace Bludata\Lumen\Application;

use Closure;
use Laravel\Lumen\Application;

abstract class BaseApplication extends Application
{
    protected $currentUser;

    abstract public function getRepositoryInterface($entity);

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

    public function getCurrentUser()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->currentUser;
    }

    public function setCurrentUser($currentUser)
    {
        $this->currentUser = $currentUser;

        return $this;
    }

    /**
     * @return Bludata\Lumen\Application\BaseApplication

Should the return type not be BaseApplication?

     */
    public function resource($descriptionGroup, $prefix, $controller, array $except = [], Closure $routes = null)

This line exceeds maximum limit of 80 characters; contains 113 characters

    {
        $exceptAll = false;

        if (isset($except[0])) {
            $exceptAll = $except[0] == '*';
        }

        if (!$exceptAll) {
            if (!in_array('index', $except)) {

This code seems to be duplicated across your project.

                $this->get($prefix, [
                    'as'               => $prefix.'.index',
                    'uses'             => $controller.'@index',
                    'description'      => 'Buscar todos',
                    'descriptionGroup' => $descriptionGroup,
                ]);
            }

            if (!in_array('show', $except)) {
                $this->get($prefix.'/{id:[0-9]+}', [
                    'as'               => $prefix.'.show',
                    'uses'             => $controller.'@show',
                    'description'      => 'Buscar um',
                    'descriptionGroup' => $descriptionGroup,
                ]);
            }

            if (!in_array('store', $except)) {
                $this->post($prefix, [
                    'as'               => $prefix.'.store',
                    'uses'             => $controller.'@store',
                    'description'      => 'Cadastrar',
                    'descriptionGroup' => $descriptionGroup,
                ]);
            }

            if (!in_array('update', $except)) {

This code seems to be duplicated across your project.

                $this->put($prefix.'/{id:[0-9]+}', [
                    'as'               => $prefix.'.update',
                    'uses'             => $controller.'@update',
                    'description'      => 'Editar',
                    'descriptionGroup' => $descriptionGroup,
                ]);
            }

            if (!in_array('destroy', $except)) {
                $this->delete($prefix.'/{id:[0-9]+}', [
                    'as'               => $prefix.'.destroy',
                    'uses'             => $controller.'@destroy',
                    'description'      => 'Excluir',
                    'descriptionGroup' => $descriptionGroup,
                ]);
            }
        }

        if ($routes instanceof Closure) {
            $routes($descriptionGroup, $prefix, $controller);
        }

        return $this;
    }
}