SquantoManagerServiceProvider::boot() - Code Metrics - Inspection of "Ft/remove htmlawed for htmlpurifier" - thinktomorrow/squanto - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#12)

by Philippe

created 2019-07-26 10:13 UTC

SquantoManagerServiceProvider::boot() A

↳ Parent: SquantoManagerServiceProvider

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2

Importance

Changes

Metric	Value
cc	2
nc	2
nop	0
dl	0
loc	12
ccs	6
cts	6
cp	1
crap	2
rs	9.8666
c	0
b	0
f	0

<?php

namespace Thinktomorrow\Squanto;

use Illuminate\Support\ServiceProvider;

class SquantoManagerServiceProvider extends ServiceProvider
{
    protected $defer = false;

    /**
     * @return array
     */
    public function provides()
    {
        return [];
    }

    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        require_once dirname(__FILE__) . "/Manager/utils/helpers.php";

        if (! $this->app->routesAreCached()) {
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            require __DIR__ . '/Manager/Http/routes/web.php';
        }

        // Register squanto viewfiles under squanto:: namespace
        // allow to override them by making a view file under the resources/views/vendor/squanto location
        $this->loadViewsFrom(realpath(__DIR__ . '/Manager/Http/views'), 'squanto');
    }

    /**
     * Register our translator
     *
     * @return void
     */
    public function register()
    {
        //
    }
}


1		<?php
2
3		namespace Thinktomorrow\Squanto;
4
5		use Illuminate\Support\ServiceProvider;
6
7		class SquantoManagerServiceProvider extends ServiceProvider
8		{
9		protected $defer = false;
10
11		/**
12		* @return array
13		*/
14		public function provides()
15		{
16		return [];
17		}
18
19		/**
20		* Bootstrap any application services.
21		*
22		* @return void
23		*/
24	62	public function boot()
25		{
26	62	require_once dirname(__FILE__) . "/Manager/utils/helpers.php";
27
28	62	if (! $this->app->routesAreCached()) {
		0 ignored issues – show Bug introduced 2018-10-31 16:04 UTC by Report Bug Copy Issue Report The method `routesAreCached` does only exist in `Illuminate\Foundation\Application`, but not in `Illuminate\Contracts\Foundation\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
29	62	require __DIR__ . '/Manager/Http/routes/web.php';
30		}
31
32		// Register squanto viewfiles under squanto:: namespace
33		// allow to override them by making a view file under the resources/views/vendor/squanto location
34	62	$this->loadViewsFrom(realpath(__DIR__ . '/Manager/Http/views'), 'squanto');
35	62	}
36
37		/**
38		* Register our translator
39		*
40		* @return void
41		*/
42	62	public function register()
43		{
44		//
45	62	}
46		}
47

thinktomorrow / squanto

Pull Request — master (#12)

SquantoManagerServiceProvider::boot() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like