Bantenprov\Nilai\NilaiServiceProvider

Complexity

Total Complexity

19

Size/Duplication

Total Lines	196
Duplicated Lines	10.2 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	12	1
A	register()	0	12	1
A	provides()	0	7	1
A	configHandle()	0	13	2
A	routeHandle()	0	4	1
A	langHandle()	10	10	2
A	viewHandle()	10	10	2
A	assetHandle()	0	8	2
A	migrationHandle()	0	10	2
A	publicHandle()	0	8	2
A	seedHandle()	0	8	2
A	publishHandle()	0	13	1

<?php

namespace Bantenprov\Nilai;

use Illuminate\Support\ServiceProvider;
use Bantenprov\Nilai\Console\Commands\NilaiCommand;

/**
 * The NilaiServiceProvider class
 *
 * @package Bantenprov\Nilai
 * @author  bantenprov <developer.bantenprov@gmail.com>
 */
class NilaiServiceProvider extends ServiceProvider
{
    /**
     * Indicates if loading of the provider is deferred.
     *
     * @var bool
     */
    protected $defer = false;

    /**
     * Bootstrap the application events.
     *
     * @return void
     */
    public function boot()
    {
        $this->routeHandle();
        $this->configHandle();
        $this->langHandle();
        $this->viewHandle();
        $this->assetHandle();
        $this->migrationHandle();
        $this->publicHandle();
        $this->seedHandle();
        $this->publishHandle();
    }

    /**
     * Register the service provider.
     *
     * @return void
     */
    public function register()
    {
        $this->app->singleton('nilai', function ($app) {

The parameter $app is not used and could be removed.

            return new Nilai;
        });

        $this->app->singleton('command.nilai', function ($app) {

The parameter $app is not used and could be removed.

            return new NilaiCommand;
        });

        $this->commands('command.nilai');
    }

    /**
     * Get the services provided by the provider.
     *
     * @return array
     */
    public function provides()
    {
        return [
            'nilai',
            'command.nilai',
        ];
    }

    /**
     * Loading and publishing package's config
     *
     * @return void
     */
    protected function configHandle($publish = '')
    {
        $packageConfigPath = __DIR__.'/config';
        $appConfigPath     = config_path('bantenprov/nilai');

        $this->mergeConfigFrom($packageConfigPath.'/nilai.php', 'nilai');
        $this->mergeConfigFrom($packageConfigPath.'/akademik.php', 'akademik');

        $this->publishes([
            $packageConfigPath.'/nilai.php' => $appConfigPath.'/nilai.php',
            $packageConfigPath.'/akademik.php' => $appConfigPath.'/akademik.php',
        ], $publish ? $publish : 'nilai-config');
    }

    /**
     * Loading package routes
     *
     * @return void
     */
    protected function routeHandle()
    {
        $this->loadRoutesFrom(__DIR__.'/routes/routes.php');
    }

    /**
     * Loading and publishing package's translations
     *
     * @return void
     */
    protected function langHandle($publish = '')

This method seems to be duplicated in your project.

    {
        $packageTranslationsPath = __DIR__.'/resources/lang';

        $this->loadTranslationsFrom($packageTranslationsPath, 'nilai');

        $this->publishes([
            $packageTranslationsPath => resource_path('lang/vendor/nilai'),
        ], $publish ? $publish : 'nilai-lang');
    }

    /**
     * Loading and publishing package's views
     *
     * @return void
     */
    protected function viewHandle($publish = '')

This method seems to be duplicated in your project.

    {
        $packageViewsPath = __DIR__.'/resources/views';

        $this->loadViewsFrom($packageViewsPath, 'nilai');

        $this->publishes([
            $packageViewsPath => resource_path('views/vendor/nilai'),
        ], $publish ? $publish : 'nilai-views');
    }

    /**
     * Publishing package's assets (JavaScript, CSS, images...)
     *
     * @return void
     */
    protected function assetHandle($publish = '')
    {
        $packageAssetsPath = __DIR__.'/resources/assets';

        $this->publishes([
            $packageAssetsPath => resource_path('assets'),
        ], $publish ? $publish : 'nilai-assets');
    }

    /**
     * Publishing package's migrations
     *
     * @return void
     */
    protected function migrationHandle($publish = '')
    {
        $packageMigrationsPath = __DIR__.'/database/migrations';

        $this->loadMigrationsFrom($packageMigrationsPath);

        $this->publishes([
            $packageMigrationsPath => database_path('migrations')
        ], $publish ? $publish : 'nilai-migrations');
    }

    /**
     * Publishing package's publics (JavaScript, CSS, images...)
     *
     * @return void
     */
    public function publicHandle($publish = '')
    {
        $packagePublicPath = __DIR__.'/public';

        $this->publishes([
            $packagePublicPath => base_path('public')
        ], $publish ? $publish : 'nilai-public');
    }

    /**
     * Publishing package's seeds
     *
     * @return void
     */
    public function seedHandle($publish = '')
    {
        $packageSeedPath = __DIR__.'/database/seeds';

        $this->publishes([
            $packageSeedPath => base_path('database/seeds')
        ], $publish ? $publish : 'nilai-seeds');
    }

    /**
     * Publishing package's all files
     *
     * @return void
     */
    public function publishHandle()
    {
        $publish = 'nilai-publish';

        $this->routeHandle($publish);

The call to NilaiServiceProvider::routeHandle() has too many arguments starting with $publish.

        $this->configHandle($publish);
        $this->langHandle($publish);
        $this->viewHandle($publish);
        $this->assetHandle($publish);
        // $this->migrationHandle($publish);

75% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->publicHandle($publish);
        $this->seedHandle($publish);
    }
}