Completed
Push — master ( 79372b...1f082f )
by recca
02:25
created

TerminalServiceProvider::handleRoutes()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 12
Code Lines 8

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 10
CRAP Score 2.003

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 2
eloc 8
c 1
b 0
f 0
nc 2
nop 1
dl 0
loc 12
ccs 10
cts 11
cp 0.9091
crap 2.003
rs 9.4285
1
<?php
2
3
namespace Recca0120\Terminal;
4
5
use Illuminate\Support\Arr;
6
use Illuminate\Http\Request;
7
use Illuminate\Routing\Router;
8
use Illuminate\Support\ServiceProvider;
9
10
class TerminalServiceProvider extends ServiceProvider
11
{
12
    /**
13
     * namespace.
14
     *
15
     * @var string
16
     */
17
    protected $namespace = 'Recca0120\Terminal\Http\Controllers';
18
19
    /**
20
     * Bootstrap any application services.
21
     *
22
     * @param \Illuminate\Http\Request $request
23
     * @param \Illuminate\Routing\Router $router
24
     */
25 1
    public function boot(Request $request, Router $router)
26
    {
27 1
        $config = $this->app['config']['terminal'];
28 1
        if (in_array($request->getClientIp(), Arr::get($config, 'whitelists', [])) === true || Arr::get($config, 'enabled') === true) {
29 1
            $this->loadViewsFrom(__DIR__.'/../resources/views', 'terminal');
30 1
            $this->handleRoutes($router, $config);
0 ignored issues
show
Unused Code introduced by
The call to TerminalServiceProvider::handleRoutes() has too many arguments starting with $config.

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress.

In this case you can add the @ignore PhpDoc annotation to the duplicate definition and it will be ignored.

Loading history...
31 1
        }
32
33 1
        if ($this->app->runningInConsole() === true) {
34 1
            $this->handlePublishes();
35 1
        }
36 1
    }
37
38
    /**
39
     * Register any application services.
40
     */
41 1
    public function register()
42
    {
43 1
        $this->mergeConfigFrom(__DIR__.'/../config/terminal.php', 'terminal');
44
45
        $this->app->singleton(Application::class, function ($app) {
46 1
            $config = $app['config']['terminal'];
47 1
            $commands = $config['commands'];
48 1
            $artisan = new Application($app, $app['events'], $app->version());
49 1
            $artisan->resolveCommands($commands, true);
50
51 1
            return $artisan;
52 1
        });
53
54
        $this->app->singleton(Kernel::class, function ($app) {
55 1
            $config = $app['config']['terminal'];
56
57 1
            return new Kernel($app[Application::class], array_merge($config, [
58 1
                'basePath' => $app->basePath(),
59 1
                'environment' => $app->environment(),
60 1
                'version' => $app->version(),
61 1
                'endpoint' => $app['url']->route(Arr::get($config, 'route.as').'endpoint'),
62 1
            ]));
63 1
        });
64 1
    }
65
66
    /**
67
     * register routes.
68
     *
69
     * @param \Illuminate\Routing\Router $router
70
     * @param array $config
0 ignored issues
show
Bug introduced by
There is no parameter named $config. Was it maybe removed?

This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function.

Consider the following example. The parameter $italy is not defined by the method finale(...).

/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}

The most likely cause is that the parameter was removed, but the annotation was not.

Loading history...
71
     */
72 1
    protected function handleRoutes(Router $router)
73
    {
74 1
        if ($this->app->routesAreCached() === false) {
75 1
            $router->group([
76 1
                'namespace' => $this->namespace,
77 1
                'prefix' => 'terminal',
78 1
                'as' => 'terminal.',
79 1
            ], function (Router $router) {
80
                require __DIR__.'/Http/routes.php';
81 1
            });
82 1
        }
83 1
    }
84
85
    /**
86
     * handle publishes.
87
     */
88 1
    protected function handlePublishes()
89
    {
90 1
        $this->publishes([
91 1
            __DIR__.'/../config/terminal.php' => config_path('terminal.php'),
92 1
        ], 'config');
93
94 1
        $this->publishes([
95 1
            __DIR__.'/../resources/views' => base_path('resources/views/vendor/terminal'),
96 1
        ], 'views');
97
98 1
        $this->publishes([
99 1
            __DIR__.'/../public' => public_path('vendor/terminal'),
100 1
        ], 'public');
101 1
    }
102
}
103