BlogServiceProvider::boot() - Code Metrics - wingsline/laravel-blog - Measure and Improve Code Quality continuously with Scrutinizer

BlogServiceProvider::boot() A
last analyzed 2020-09-08 17:53 UTC

↳ Parent: BlogServiceProvider

Complexity

Conditions	2
Paths	2

Size

Total Lines	52
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	32
nc	2
nop	0
dl	0
loc	52
c	1
b	0
f	0
cc	2
rs	9.408

How to fix Long Method

<?php

namespace Wingsline\Blog;

use Illuminate\Support\Facades\Route;
use Illuminate\Support\ServiceProvider;
use Spatie\Flash\Flash;
use Spatie\Menu\Laravel\Facades\Menu;
use Spatie\Menu\Laravel\View;
use Spatie\ResponseCache\Middlewares\CacheResponse;
use Spatie\Tags\Tag;
use Wingsline\Blog\Console\InstallCommand;
use Wingsline\Blog\Console\PublishCommand;
use Wingsline\Blog\Console\ThemePublishCommand;
use Wingsline\Blog\Http\Middleware\Authenticate;
use Wingsline\Blog\Http\Middleware\NoHttpCache;
use Wingsline\Blog\Posts\Post;
use Wingsline\Blog\Posts\PostObserver;

class BlogServiceProvider extends ServiceProvider
{
    public function addFeed()
    {
        $this->app['config']
            ->set(
                'feed.feeds.blog',
                $this->app['config']->get('blog.feed')
            );
    }

    /**
     * Perform post-registration booting of services.
     */
    public function boot()
    {
        // prepend the views from the theme
        $this->app['view']->getFinder()->prependLocation(base_path('theme/views'));

        $this->loadViewsFrom(__DIR__.'/../resources/views', 'blog');
        $this->loadMigrationsFrom(__DIR__.'/../database/migrations');
        $this->loadRoutesFrom(__DIR__.'/../routes/web.php');

        Flash::levels([
            'success' => '',
            'error' => 'bg-red-500',
        ]);

        Menu::macro('blogAdminNavHeader', function () {
            return Menu::build(
                config('blog.navHeader'),
                function (\Spatie\Menu\Laravel\Menu $menu, $view) {
                    $menu->add(View::create($view)->addParentClass('ml-5 lg:ml-6'));
                })
                ->addClass('flex items-end lg:items-center lg:pr-2')
                ->addItemClass('p-2')
                ->setActiveFromRequest('/');
        });

        Menu::macro('blogAdminNav', function () {
            return Menu::build(
                config('blog.navAdmin'),
                function (\Spatie\Menu\Laravel\Menu $menu, $view) {
                    $menu->add(View::create($view)->addParentClass(''));
                })
                ->addItemClass('p-2')
                ->setActiveFromRequest('/');
        });

        // register the route middleware groups
        $this->app['router']->middlewareGroup('blog-auth',
            [Authenticate::class]);
        $this->app['router']->middlewareGroup('blog-nocache',
            [NoHttpCache::class]);
        $this->app['router']->middlewareGroup('blog-cacheResponse',
            [CacheResponse::class]);

        // router bindings
        $this->registerRouteModelBindings();

        // model event observers
        Post::observe(PostObserver::class);

        // Publishing is only necessary when using the CLI.
        if ($this->app->runningInConsole()) {
            $this->bootForConsole();
        }
    }

    /**
     * Register any package services.
     */
    public function register()
    {
        $this->mergeConfigFrom(__DIR__.'/../config/blog.php', 'blog');

        if (file_exists(base_path('theme/config.php'))) {
            $this->mergeConfigFrom(base_path('theme/config.php'), 'theme');
        }

        $this->addFeed();

        $this->commands([
            InstallCommand::class,
            PublishCommand::class,
            ThemePublishCommand::class,
        ]);
    }

    public function registerRouteModelBindings()
    {
        Route::bind('postSlug', function ($slug) {
            if (auth()->check()) {
                return Post::where('slug', $slug)->first() ?? abort(404);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            }

            $post = Post::where('slug', $slug)->public()->first() ?? abort(404);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

            if (! $post->published) {
                abort(404);
            }

            return $post;
        });

        Route::bind('tagSlug', function ($slug) {
            return Tag::where('slug->en', $slug)->first() ?? abort(404);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        });
    }

    /**
     * Console-specific booting.
     */
    protected function bootForConsole()
    {
        // Publishing the configuration file.
        $this->publishes([
            __DIR__.'/../config/blog.php' => config_path('blog.php'),
            __DIR__.'/../config/theme.php' => config_path('theme.php'),
        ], 'blog.config');

        // Publishing the views.
        $this->publishes([
            __DIR__.'/../resources/views' => base_path('resources/views/vendor/wingsline'),
        ], 'blog.views');

        // Publishing the migrations.
        $this->publishes([
            __DIR__.'/../database/migrations' => database_path('migrations'),
        ], 'blog.migrations');

        // Publishing assets.
        $this->publishes([
            __DIR__.'/../public' => public_path('vendor/wingsline-blog'),
        ], 'blog.assets');

        // Publishing the translation files.
        /*$this->publishes([
            __DIR__.'/../resources/lang' => resource_path('lang/vendor/wingsline'),
        ], 'blog.views');*/

        // Registering package commands.
        // $this->commands([]);
    }
}


1			<?php
2
3			namespace Wingsline\Blog;
4
5			use Illuminate\Support\Facades\Route;
6			use Illuminate\Support\ServiceProvider;
7			use Spatie\Flash\Flash;
8			use Spatie\Menu\Laravel\Facades\Menu;
9			use Spatie\Menu\Laravel\View;
10			use Spatie\ResponseCache\Middlewares\CacheResponse;
11			use Spatie\Tags\Tag;
12			use Wingsline\Blog\Console\InstallCommand;
13			use Wingsline\Blog\Console\PublishCommand;
14			use Wingsline\Blog\Console\ThemePublishCommand;
15			use Wingsline\Blog\Http\Middleware\Authenticate;
16			use Wingsline\Blog\Http\Middleware\NoHttpCache;
17			use Wingsline\Blog\Posts\Post;
18			use Wingsline\Blog\Posts\PostObserver;
19
20			class BlogServiceProvider extends ServiceProvider
21			{
22			public function addFeed()
23			{
24			$this->app['config']
25			->set(
26			'feed.feeds.blog',
27			$this->app['config']->get('blog.feed')
28			);
29			}
30
31			/**
32			* Perform post-registration booting of services.
33			*/
34			public function boot()
35			{
36			// prepend the views from the theme
37			$this->app['view']->getFinder()->prependLocation(base_path('theme/views'));
38
39			$this->loadViewsFrom(__DIR__.'/../resources/views', 'blog');
40			$this->loadMigrationsFrom(__DIR__.'/../database/migrations');
41			$this->loadRoutesFrom(__DIR__.'/../routes/web.php');
42
43			Flash::levels([
44			'success' => '',
45			'error' => 'bg-red-500',
46			]);
47
48			Menu::macro('blogAdminNavHeader', function () {
49			return Menu::build(
50			config('blog.navHeader'),
51			function (\Spatie\Menu\Laravel\Menu $menu, $view) {
52			$menu->add(View::create($view)->addParentClass('ml-5 lg:ml-6'));
53			})
54			->addClass('flex items-end lg:items-center lg:pr-2')
55			->addItemClass('p-2')
56			->setActiveFromRequest('/');
57			});
58
59			Menu::macro('blogAdminNav', function () {
60			return Menu::build(
61			config('blog.navAdmin'),
62			function (\Spatie\Menu\Laravel\Menu $menu, $view) {
63			$menu->add(View::create($view)->addParentClass(''));
64			})
65			->addItemClass('p-2')
66			->setActiveFromRequest('/');
67			});
68
69			// register the route middleware groups
70			$this->app['router']->middlewareGroup('blog-auth',
71			[Authenticate::class]);
72			$this->app['router']->middlewareGroup('blog-nocache',
73			[NoHttpCache::class]);
74			$this->app['router']->middlewareGroup('blog-cacheResponse',
75			[CacheResponse::class]);
76
77			// router bindings
78			$this->registerRouteModelBindings();
79
80			// model event observers
81			Post::observe(PostObserver::class);
82
83			// Publishing is only necessary when using the CLI.
84			if ($this->app->runningInConsole()) {
85			$this->bootForConsole();
86			}
87			}
88
89			/**
90			* Register any package services.
91			*/
92			public function register()
93			{
94			$this->mergeConfigFrom(__DIR__.'/../config/blog.php', 'blog');
95
96			if (file_exists(base_path('theme/config.php'))) {
97			$this->mergeConfigFrom(base_path('theme/config.php'), 'theme');
98			}
99
100			$this->addFeed();
101
102			$this->commands([
103			InstallCommand::class,
104			PublishCommand::class,
105			ThemePublishCommand::class,
106			]);
107			}
108
109			public function registerRouteModelBindings()
110			{
111			Route::bind('postSlug', function ($slug) {
112			if (auth()->check()) {
113			return Post::where('slug', $slug)->first() ?? abort(404);
			0 ignored issues – show Bug introduced 2019-11-25 17:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `abort(404)` is correct as it seems to always return `null`. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
114			}
115
116			$post = Post::where('slug', $slug)->public()->first() ?? abort(404);
			0 ignored issues – show Bug introduced 2019-11-25 17:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `abort(404)` is correct as it seems to always return `null`. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
117
118			if (! $post->published) {
119			abort(404);
120			}
121
122			return $post;
123			});
124
125			Route::bind('tagSlug', function ($slug) {
126			return Tag::where('slug->en', $slug)->first() ?? abort(404);
			0 ignored issues – show Bug introduced 2019-11-25 17:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `abort(404)` is correct as it seems to always return `null`. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
127			});
128			}
129
130			/**
131			* Console-specific booting.
132			*/
133			protected function bootForConsole()
134			{
135			// Publishing the configuration file.
136			$this->publishes([
137			__DIR__.'/../config/blog.php' => config_path('blog.php'),
138			__DIR__.'/../config/theme.php' => config_path('theme.php'),
139			], 'blog.config');
140
141			// Publishing the views.
142			$this->publishes([
143			__DIR__.'/../resources/views' => base_path('resources/views/vendor/wingsline'),
144			], 'blog.views');
145
146			// Publishing the migrations.
147			$this->publishes([
148			__DIR__.'/../database/migrations' => database_path('migrations'),
149			], 'blog.migrations');
150
151			// Publishing assets.
152			$this->publishes([
153			__DIR__.'/../public' => public_path('vendor/wingsline-blog'),
154			], 'blog.assets');
155
156			// Publishing the translation files.
157			/*$this->publishes([
158			__DIR__.'/../resources/lang' => resource_path('lang/vendor/wingsline'),
159			], 'blog.views');*/
160
161			// Registering package commands.
162			// $this->commands([]);
163			}
164			}
165

wingsline / laravel-blog

BlogServiceProvider::boot() A last analyzed 2020-09-08 17:53 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

BlogServiceProvider::boot() A
last analyzed 2020-09-08 17:53 UTC