PermissionServiceProvider::booted() - Code Metrics - Inspection of "Move Permissions Registration to booted method" - spatie/laravel-permission - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#143)

by Ian

created 2016-12-12 23:01 UTC

PermissionServiceProvider::booted() A

↳ Parent: PermissionServiceProvider

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	1

<?php

namespace Spatie\Permission;

use Illuminate\Support\ServiceProvider;
use Illuminate\View\Compilers\BladeCompiler;
use Spatie\Permission\Contracts\Permission as PermissionContract;
use Spatie\Permission\Contracts\Role as RoleContract;

class PermissionServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap the application services.
     */
    public function boot()
    {
        $this->publishes([
            __DIR__.'/../resources/config/laravel-permission.php' => $this->app->configPath().'/'.'laravel-permission.php',
        ], 'config');

        if (!class_exists('CreatePermissionTables')) {
            // Publish the migration
            $timestamp = date('Y_m_d_His', time());
            $this->publishes([
                __DIR__.'/../resources/migrations/create_permission_tables.php.stub' => $this->app->databasePath().'/migrations/'.$timestamp.'_create_permission_tables.php',

            ], 'migrations');
        }
    }

    /**
     * Finish configuring the application.
     *
     * @param PermissionRegistrar $permissionLoader
     */
    public function booted(PermissionRegistrar $permissionLoader)
    {
        $permissionLoader->registerPermissions();
    }

    /**
     * Register the application services.
     */
    public function register()
    {
        $this->mergeConfigFrom(__DIR__.'/../resources/config/laravel-permission.php', 'laravel-permission');

        $this->registerModelBindings();

        $this->registerBladeExtensions();
    }

    /**
     * Bind the Permission and Role model into the IoC.
     */
    protected function registerModelBindings()
    {
        $config = $this->app->config['laravel-permission.models'];
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}

        $this->app->bind(PermissionContract::class, $config['permission']);
        $this->app->bind(RoleContract::class, $config['role']);
    }

    /**
     * Register the blade extensions.
     */
    protected function registerBladeExtensions()
    {
        $this->app->afterResolving('blade.compiler', function (BladeCompiler $bladeCompiler) {
            $bladeCompiler->directive('role', function ($role) {
                return "<?php if(auth()->check() && auth()->user()->hasRole({$role})): ?>";
            });
            $bladeCompiler->directive('endrole', function () {
                return '<?php endif; ?>';
            });

            $bladeCompiler->directive('hasrole', function ($role) {
                return "<?php if(auth()->check() && auth()->user()->hasRole({$role})): ?>";
            });
            $bladeCompiler->directive('endhasrole', function () {
                return '<?php endif; ?>';
            });

            $bladeCompiler->directive('hasanyrole', function ($roles) {
                return "<?php if(auth()->check() && auth()->user()->hasAnyRole({$roles})): ?>";
            });
            $bladeCompiler->directive('endhasanyrole', function () {
                return '<?php endif; ?>';
            });

            $bladeCompiler->directive('hasallroles', function ($roles) {
                return "<?php if(auth()->check() && auth()->user()->hasAllRoles({$roles})): ?>";
            });
            $bladeCompiler->directive('endhasallroles', function () {
                return '<?php endif; ?>';
            });
        });
    }
}


1			<?php
2
3			namespace Spatie\Permission;
4
5			use Illuminate\Support\ServiceProvider;
6			use Illuminate\View\Compilers\BladeCompiler;
7			use Spatie\Permission\Contracts\Permission as PermissionContract;
8			use Spatie\Permission\Contracts\Role as RoleContract;
9
10			class PermissionServiceProvider extends ServiceProvider
11			{
12			/**
13			* Bootstrap the application services.
14			*/
15			public function boot()
16			{
17			$this->publishes([
18			__DIR__.'/../resources/config/laravel-permission.php' => $this->app->configPath().'/'.'laravel-permission.php',
19			], 'config');
20
21			if (!class_exists('CreatePermissionTables')) {
22			// Publish the migration
23			$timestamp = date('Y_m_d_His', time());
24			$this->publishes([
25			__DIR__.'/../resources/migrations/create_permission_tables.php.stub' => $this->app->databasePath().'/migrations/'.$timestamp.'_create_permission_tables.php',
			0 ignored issues – show Bug introduced 2015-12-21 23:32 UTC by Report Bug Copy Issue Report The method `databasePath()` does not exist on `Illuminate\Contracts\Foundation\Application`. Did you maybe mean `basePath()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
26			], 'migrations');
27			}
28			}
29
30			/**
31			* Finish configuring the application.
32			*
33			* @param PermissionRegistrar $permissionLoader
34			*/
35			public function booted(PermissionRegistrar $permissionLoader)
36			{
37			$permissionLoader->registerPermissions();
38			}
39
40			/**
41			* Register the application services.
42			*/
43			public function register()
44			{
45			$this->mergeConfigFrom(__DIR__.'/../resources/config/laravel-permission.php', 'laravel-permission');
46
47			$this->registerModelBindings();
48
49			$this->registerBladeExtensions();
50			}
51
52			/**
53			* Bind the Permission and Role model into the IoC.
54			*/
55			protected function registerModelBindings()
56			{
57			$config = $this->app->config['laravel-permission.models'];
			0 ignored issues – show Bug introduced 2015-12-21 23:32 UTC by Report Bug Copy Issue Report Accessing `config` on the interface `Illuminate\Contracts\Foundation\Application` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
58
59			$this->app->bind(PermissionContract::class, $config['permission']);
60			$this->app->bind(RoleContract::class, $config['role']);
61			}
62
63			/**
64			* Register the blade extensions.
65			*/
66			protected function registerBladeExtensions()
67			{
68			$this->app->afterResolving('blade.compiler', function (BladeCompiler $bladeCompiler) {
69			$bladeCompiler->directive('role', function ($role) {
70			return "<?php if(auth()->check() && auth()->user()->hasRole({$role})): ?>";
71			});
72			$bladeCompiler->directive('endrole', function () {
73			return '<?php endif; ?>';
74			});
75
76			$bladeCompiler->directive('hasrole', function ($role) {
77			return "<?php if(auth()->check() && auth()->user()->hasRole({$role})): ?>";
78			});
79			$bladeCompiler->directive('endhasrole', function () {
80			return '<?php endif; ?>';
81			});
82
83			$bladeCompiler->directive('hasanyrole', function ($roles) {
84			return "<?php if(auth()->check() && auth()->user()->hasAnyRole({$roles})): ?>";
85			});
86			$bladeCompiler->directive('endhasanyrole', function () {
87			return '<?php endif; ?>';
88			});
89
90			$bladeCompiler->directive('hasallroles', function ($roles) {
91			return "<?php if(auth()->check() && auth()->user()->hasAllRoles({$roles})): ?>";
92			});
93			$bladeCompiler->directive('endhasallroles', function () {
94			return '<?php endif; ?>';
95			});
96			});
97			}
98			}
99

spatie / laravel-permission

Pull Request — master (#143)

PermissionServiceProvider::booted() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like