ServiceProvider::boot() - Code Metrics - Inspection of "Add new AbstractWidget." - benrowe/laravel-widgets - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3a775b...45e4bb )

by Ben

created 2016-06-14 22:11 UTC

ServiceProvider::boot() A

↳ Parent: ServiceProvider

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	6
rs	9.4285
cc	1
eloc	4
nc	1
nop	0

<?php

namespace Benrowe\Laravel\Widgets;

use Arrilot\Widgets\ServiceProvider as BaseServiceProvider;
use Benrowe\Laravel\Widgets\Factories\WidgetFactory;
use Arrilot\Widgets\Misc\LaravelApplicationWrapper;

/**
 * @package Benrowe\Laravel\Widgets
 */
class ServiceProvider extends BaseServiceProvider
{
    /**
     * Suplement the base provider with a different version of the widget factory
     *
     * @return nil

     */
    public function register()
    {
        parent::boot();
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}

        $this->app->bind('arrilot.widget', function () {
            return new WidgetFactory(new LaravelApplicationWrapper());
        });

        $this->app->alias('arrilot.widget', 'Arrilot\Widgets\Factories\WidgetFactory');

    }

    /**
     * Register two additional blade directives
     *
     * @return nil

     */
    public function boot()
    {
        parent::boot();
        $this->registerBladeDirective('widget-begin', '$1<?php echo app("arrilot.widget")->begin$2; ?>');
        $this->registerBladeDirective('widget-end', '$1<?php echo app("arrilot.widget")->endw$2; ?>');
    }
}


1			<?php
2
3			namespace Benrowe\Laravel\Widgets;
4
5			use Arrilot\Widgets\ServiceProvider as BaseServiceProvider;
6			use Benrowe\Laravel\Widgets\Factories\WidgetFactory;
7			use Arrilot\Widgets\Misc\LaravelApplicationWrapper;
8
9			/**
10			* @package Benrowe\Laravel\Widgets
11			*/
12			class ServiceProvider extends BaseServiceProvider
13			{
14			/**
15			* Suplement the base provider with a different version of the widget factory
16			*
17			* @return nil
			0 ignored issues – show Documentation introduced 2016-06-14 22:15 UTC by Report Bug Copy Issue Report Should the return type not be `nil\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
18			*/
19			public function register()
20			{
21			parent::boot();
			0 ignored issues – show Comprehensibility Bug introduced 2016-06-14 22:15 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`boot()` instead of `register()`). Are you sure this is correct? If so, you might want to change this to `$this->boot()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
22
23			$this->app->bind('arrilot.widget', function () {
24			return new WidgetFactory(new LaravelApplicationWrapper());
25			});
26
27			$this->app->alias('arrilot.widget', 'Arrilot\Widgets\Factories\WidgetFactory');
28
29			}
30
31			/**
32			* Register two additional blade directives
33			*
34			* @return nil
			0 ignored issues – show Documentation introduced 2016-06-14 22:15 UTC by Report Bug Copy Issue Report Should the return type not be `nil\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
35			*/
36			public function boot()
37			{
38			parent::boot();
39			$this->registerBladeDirective('widget-begin', '$1<?php echo app("arrilot.widget")->begin$2; ?>');
40			$this->registerBladeDirective('widget-end', '$1<?php echo app("arrilot.widget")->endw$2; ?>');
41			}
42			}
43

benrowe / laravel-widgets

Push — master ( 3a775b...45e4bb )

ServiceProvider::boot() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like