Bantenprov\Siswa\SiswaServiceProvider

Complexity

Total Complexity

19

Size/Duplication

Total Lines	194
Duplicated Lines	10.31 %

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	11	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\Siswa;

use Illuminate\Support\ServiceProvider;
use Bantenprov\Siswa\Console\Commands\SiswaCommand;

/**
 * The SiswaServiceProvider class
 *
 * @package Bantenprov\Siswa
 * @author  bantenprov <developer.bantenprov@gmail.com>
 */
class SiswaServiceProvider 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('siswa', function ($app) {

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

            return new Siswa;
        });

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

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

            return new SiswaCommand;
        });

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

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

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

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

        $this->publishes([
            $packageConfigPath.'/siswa.php' => $appConfigPath.'/siswa.php',
        ], $publish ? $publish : 'siswa-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, 'siswa');

        $this->publishes([
            $packageTranslationsPath => resource_path('lang/vendor/siswa'),
        ], $publish ? $publish : 'siswa-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, 'siswa');

        $this->publishes([
            $packageViewsPath => resource_path('views/vendor/siswa'),
        ], $publish ? $publish : 'siswa-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 : 'siswa-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 : 'siswa-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 : 'siswa-public');
    }

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

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

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

        $this->routeHandle($publish);

The call to SiswaServiceProvider::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);
    }
}