AppServiceProvider - Code Metrics - Inspection of "User controller" - uidaho/smartsettia - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#8)

by Brandon

created 2017-09-11 05:26 UTC

AppServiceProvider A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

84.62%

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
wmc	3
c	1
b	1
f	0
lcom	0
cbo	1
dl	0
loc	37
ccs	11
cts	13
cp	0.8462
rs	10

2 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	19	2
A	register()	0	4	1

<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use Illuminate\Support\Facades\Schema;
use Illuminate\Support\Facades\Validator;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        // Fixes incompatibility with MySQL < 5.7.7
        Schema::defaultStringLength(191);

        // Create {{ $view_name }} provider in blade templates
        view()->composer('*', function($view) {
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.
}
            $view_name = str_replace('.', '-', $view->getName());
            view()->share('view_name', $view_name);
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.
}
        });

        Validator::extend('phone', function($attribute, $value, $parameters, $validator) {

            return preg_match('%^(?:(?:\(?(?:00|\+)([1-4]\d\d|[1-9]\d?)\)?)?[\-\.\ \\\/]?)?((?:\(?\d{1,}\)?[\-\.\ \\\/]?){0,})(?:[\-\.\ \\\/]?(?:#|ext\.?|extension|x)[\-\.\ \\\/]?(\d+))?$%i', $value) && strlen($value) >= 10;
        });

        Validator::replacer('phone', function($message, $attribute, $rule, $parameters) {

            return str_replace(':attribute',$attribute, ':attribute is invalid phone number');
        });
    }

    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        //
    }
}


Pull Request — master (#8)

AppServiceProvider A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

2 Methods

Available Fixes

Available Fixes

1		<?php
2
3		namespace App\Providers;
4
5		use Illuminate\Support\ServiceProvider;
6		use Illuminate\Support\Facades\Schema;
7		use Illuminate\Support\Facades\Validator;
8
9		class AppServiceProvider extends ServiceProvider
10		{
11		/**
12		* Bootstrap any application services.
13		*
14		* @return void
15		*/
16	4	public function boot()
17		{
18		// Fixes incompatibility with MySQL < 5.7.7
19	4	Schema::defaultStringLength(191);
20
21		// Create {{ $view_name }} provider in blade templates
22		view()->composer('*', function($view) {
		0 ignored issues – show Bug introduced 2017-08-19 20:52 UTC by Report Bug Copy Issue Report The method `composer` does only exist in `Illuminate\Contracts\View\Factory`, but not in `Illuminate\View\View`. 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...
23	1	$view_name = str_replace('.', '-', $view->getName());
24	1	view()->share('view_name', $view_name);
		0 ignored issues – show Bug introduced 2017-08-19 20:52 UTC by Report Bug Copy Issue Report The method `share` does only exist in `Illuminate\Contracts\View\Factory`, but not in `Illuminate\View\View`. 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...
25	4	});
26
27		Validator::extend('phone', function($attribute, $value, $parameters, $validator) {
		0 ignored issues – show Unused Code introduced 2017-09-11 05:30 UTC by Report Bug Copy Issue Report The parameter `$parameters` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-09-11 05:30 UTC by Report Bug Copy Issue Report The parameter `$validator` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
28		return preg_match('%^(?:(?:\(?(?:00\|\+)([1-4]\d\d\|[1-9]\d?)\)?)?[\-\.\ \\\/]?)?((?:\(?\d{1,}\)?[\-\.\ \\\/]?){0,})(?:[\-\.\ \\\/]?(?:#\|ext\.?\|extension\|x)[\-\.\ \\\/]?(\d+))?$%i', $value) && strlen($value) >= 10;
29	4	});
30
31	4	Validator::replacer('phone', function($message, $attribute, $rule, $parameters) {
		0 ignored issues – show Unused Code introduced 2017-09-11 05:30 UTC by Report Bug Copy Issue Report The parameter `$rule` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-09-11 05:30 UTC by Report Bug Copy Issue Report The parameter `$parameters` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
32		return str_replace(':attribute',$attribute, ':attribute is invalid phone number');
33	4	});
34	4	}
35
36		/**
37		* Register any application services.
38		*
39		* @return void
40		*/
41	4	public function register()
42		{
43		//
44	4	}
45		}
46

uidaho / smartsettia

Pull Request — master (#8)

AppServiceProvider A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

2 Methods

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like