Thinktomorrow\AssetLibrary\AssetLibraryServiceProvider

Complexity

Total Complexity

10

Size/Duplication

Total Lines	106
Duplicated Lines	16.98 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

97.62%

Importance

Changes	4
Bugs	2	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	13	1
A	registerEloquentFactoriesFrom()	0	4	1
A	setupRoutes()	0	6	1
A	register()	0	6	1
A	registerModelBindings()	0	4	1
A	registerAssetLibrary()	0	6	1
B	publishMigrations()	18	23	4

<?php

namespace Thinktomorrow\AssetLibrary;

use Illuminate\Database\Eloquent\Factory as EloquentFactory;
use Illuminate\Routing\Router;
use Illuminate\Support\ServiceProvider;
use Thinktomorrow\AssetLibrary\Models\Asset;

class AssetLibraryServiceProvider extends ServiceProvider
{
    /**
     * Indicates if loading of the provider is deferred.
     *
     * @var bool
     */
    protected $defer = false;

    /**
     * Perform post-registration booting of services.
     *
     * @return void
     */
    public function boot()
    {
        $this->publishes([
            __DIR__.'/../config/assetlibrary.php' => config_path('assetlibrary.php'),
        ], 'config');

        $this->setupRoutes($this->app->router);

Accessing router on the interface Illuminate\Contracts\Foundation\Application suggest that you code against a concrete implementation. How about adding an instanceof check?

        $this->publishMigrations();

        $this->registerModelBindings();
        $this->registerEloquentFactoriesFrom(__DIR__.'/../database/factories');
    }

    /**
     * Register factories.
     *
     * @param  string  $path
     * @return void
     */
    protected function registerEloquentFactoriesFrom($path)
    {
        $this->app->make(EloquentFactory::class)->load($path);
    }

    /**
     * Define the routes for the application.
     *
     * @param  \Illuminate\Routing\Router  $router
     * @return void
     */
    public function setupRoutes(Router $router)
    {
        $router->group(['namespace' => 'Thinktomorrow\AssetLibrary\Http\Controllers'], function ($router) {
            require __DIR__.'/Http/routes.php';
        });
    }

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

        $this->registerAssetLibrary();
    }

    protected function registerModelBindings()
    {
        //TODO implement this
    }

    /**
     *
     */
    private function registerAssetLibrary()
    {
        $this->app->singleton('asset', function ($app) {
            return new Asset($app);
        });
    }

    /**
     *
     */
    public function publishMigrations(): void
    {
        if (!class_exists('CreateAssetTable')) {

This code seems to be duplicated across your project.

            $this->publishes([
                __DIR__ . '/../database/migrations/create_asset_table.php' => database_path('migrations/' . date('Y_m_d_His',

This line exceeds maximum limit of 120 characters; contains 125 characters

                        time()) . '_create_asset_table.php'),
            ], 'migrations');

        }

        if (!class_exists('CreateAssetPivotTable')) {

This code seems to be duplicated across your project.

            $this->publishes([
                __DIR__ . '/../database/migrations/create_asset_pivot_table.php' => database_path('migrations/' . date('Y_m_d_His',

This line exceeds maximum limit of 120 characters; contains 131 characters

                        time()) . '_create_asset_pivot_table.php'),
            ], 'migrations');
        }

        if (! class_exists('CreateMediaTable')) {

This code seems to be duplicated across your project.

            $this->publishes([
                __DIR__.'/../../../spatie/laravel-medialibrary/database/migrations/create_media_table.php.stub' => database_path('migrations/'.date('Y_m_d_His', time()).'_create_media_table.php'),

This line exceeds maximum limit of 120 characters; contains 196 characters

            ], 'migrations');
        }
    }
}