NotifynderServiceProvider::migration() - Code Metrics - Inspection of "clean repo to absolute basics" - fenos/Notifynder - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — version-4 ( 5bb021 )

unknown

created 2016-06-17 14:20 UTC

NotifynderServiceProvider::migration() F

↳ Parent: NotifynderServiceProvider

Complexity

Conditions	10
Paths	512

Size

Total Lines	30
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes	6
Bugs	1	Features	1

Metric	Value
cc	10
eloc	19
c	6
b	1
f	1
nc	512
nop	0
dl	0
loc	30
rs	3.2187

How to fix Complexity

<?php

namespace Fenos\Notifynder;

use Fenos\Notifynder\Collections\Config;
use Fenos\Notifynder\Contracts\ConfigContract;
use Fenos\Notifynder\Contracts\NotifynderContract;
use Fenos\Notifynder\Managers\NotifynderManager;
use Illuminate\Support\ServiceProvider;

class NotifynderServiceProvider extends ServiceProvider
{
    public function register()
    {
        $this->bindContracts();
        $this->bindConfig();
        $this->bindNotifynder();
    }

    public function boot()
    {
        $this->config();
        $this->migration();
    }

    /**
     * Bind contracts.
     */
    protected function bindContracts()
    {
        $this->app->bind(NotifynderContract::class, 'notifynder');
        $this->app->bind(ConfigContract::class, 'notifynder.config');
    }

    /**
     * Bind notifynder config.
     */
    protected function bindConfig()
    {
        $this->app->singleton('notifynder.config', function ($app) {

            return new Config();
        });
    }

    /**
     * Bind notifynder manager.
     */
    protected function bindNotifynder()
    {
        $this->app->singleton('notifynder', function ($app) {
            return new NotifynderManager(
                $app['notifynder.category'],

                $app['notifynder.sender'],
                $app['notifynder.notification'],
                $app['notifynder.dispatcher'],
                $app['notifynder.group']
            );
        });
    }

    /**
     * Publish config file.
     */
    protected function config()
    {
        $this->publishes([
            __DIR__.'/../config/notifynder.php' => config_path('notifynder.php'),
        ]);

        $this->mergeConfigFrom(__DIR__.'/../config/notifynder.php', 'notifynder');
    }

    /**
     * Publish migration files.
     */
    protected function migration()
    {
        if (! class_exists('NotificationCategories')) {
            $this->publishMigration('2014_02_10_145728_notification_categories');
        }
        if (! class_exists('CreateNotificationGroupsTable')) {
            $this->publishMigration('2014_08_01_210813_create_notification_groups_table');
        }
        if (! class_exists('CreateNotificationCategoryNotificationGroupTable')) {
            $this->publishMigration('2014_08_01_211045_create_notification_category_notification_group_table');
        }
        if (! class_exists('CreateNotificationsTable')) {
            $this->publishMigration('2015_05_05_212549_create_notifications_table');
        }
        if (! class_exists('AddExpireTimeColumnToNotificationTable')) {
            $this->publishMigration('2015_06_06_211555_add_expire_time_column_to_notification_table');
        }
        if (! class_exists('ChangeTypeToExtraInNotificationsTable')) {
            $this->publishMigration('2015_06_06_211555_change_type_to_extra_in_notifications_table');
        }
        if (! class_exists('AlterCategoryNameToUnique')) {
            $this->publishMigration('2015_06_07_211555_alter_category_name_to_unique');
        }
        if (! class_exists('MakeNotificationUrlNullable')) {
            $this->publishMigration('2016_04_19_200827_make_notification_url_nullable');
        }
        if (! class_exists('AddStackIdToNotifications')) {
            $this->publishMigration('2016_05_19_144531_add_stack_id_to_notifications');
        }
    }

    /**
     * @param string $filename
     */
    protected function publishMigration($filename)
    {
        $extension = '.php';
        $filename = trim($filename, $extension).$extension;
        $stub = __DIR__.'/../migrations/'.$filename;
        $target = $this->migrationFilepath($filename);
        $this->publishes([$stub => $target], 'migrations');
    }

    /**
     * @param string $filename
     * @return string
     */
    protected function migrationFilepath($filename)
    {
        if (function_exists('database_path')) {
            return database_path('/migrations/'.$filename);
        } else {
            return base_path('/database/migrations/'.$filename);
        }
    }
}


1			<?php
2
3			namespace Fenos\Notifynder;
4
5			use Fenos\Notifynder\Collections\Config;
6			use Fenos\Notifynder\Contracts\ConfigContract;
7			use Fenos\Notifynder\Contracts\NotifynderContract;
8			use Fenos\Notifynder\Managers\NotifynderManager;
9			use Illuminate\Support\ServiceProvider;
10
11			class NotifynderServiceProvider extends ServiceProvider
12			{
13			public function register()
14			{
15			$this->bindContracts();
16			$this->bindConfig();
17			$this->bindNotifynder();
18			}
19
20			public function boot()
21			{
22			$this->config();
23			$this->migration();
24			}
25
26			/**
27			* Bind contracts.
28			*/
29			protected function bindContracts()
30			{
31			$this->app->bind(NotifynderContract::class, 'notifynder');
32			$this->app->bind(ConfigContract::class, 'notifynder.config');
33			}
34
35			/**
36			* Bind notifynder config.
37			*/
38			protected function bindConfig()
39			{
40			$this->app->singleton('notifynder.config', function ($app) {
			0 ignored issues – show Unused Code introduced 2016-06-17 14:28 UTC by Report Bug Copy Issue Report The parameter `$app` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
41			return new Config();
42			});
43			}
44
45			/**
46			* Bind notifynder manager.
47			*/
48			protected function bindNotifynder()
49			{
50			$this->app->singleton('notifynder', function ($app) {
51			return new NotifynderManager(
52			$app['notifynder.category'],
			0 ignored issues – show Unused Code introduced 2016-06-17 14:28 UTC by Report Bug Copy Issue Report The call to `NotifynderManager::__construct()` has too many arguments starting with `$app['notifynder.category']`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
53			$app['notifynder.sender'],
54			$app['notifynder.notification'],
55			$app['notifynder.dispatcher'],
56			$app['notifynder.group']
57			);
58			});
59			}
60
61			/**
62			* Publish config file.
63			*/
64			protected function config()
65			{
66			$this->publishes([
67			__DIR__.'/../config/notifynder.php' => config_path('notifynder.php'),
68			]);
69
70			$this->mergeConfigFrom(__DIR__.'/../config/notifynder.php', 'notifynder');
71			}
72
73			/**
74			* Publish migration files.
75			*/
76			protected function migration()
77			{
78			if (! class_exists('NotificationCategories')) {
79			$this->publishMigration('2014_02_10_145728_notification_categories');
80			}
81			if (! class_exists('CreateNotificationGroupsTable')) {
82			$this->publishMigration('2014_08_01_210813_create_notification_groups_table');
83			}
84			if (! class_exists('CreateNotificationCategoryNotificationGroupTable')) {
85			$this->publishMigration('2014_08_01_211045_create_notification_category_notification_group_table');
86			}
87			if (! class_exists('CreateNotificationsTable')) {
88			$this->publishMigration('2015_05_05_212549_create_notifications_table');
89			}
90			if (! class_exists('AddExpireTimeColumnToNotificationTable')) {
91			$this->publishMigration('2015_06_06_211555_add_expire_time_column_to_notification_table');
92			}
93			if (! class_exists('ChangeTypeToExtraInNotificationsTable')) {
94			$this->publishMigration('2015_06_06_211555_change_type_to_extra_in_notifications_table');
95			}
96			if (! class_exists('AlterCategoryNameToUnique')) {
97			$this->publishMigration('2015_06_07_211555_alter_category_name_to_unique');
98			}
99			if (! class_exists('MakeNotificationUrlNullable')) {
100			$this->publishMigration('2016_04_19_200827_make_notification_url_nullable');
101			}
102			if (! class_exists('AddStackIdToNotifications')) {
103			$this->publishMigration('2016_05_19_144531_add_stack_id_to_notifications');
104			}
105			}
106
107			/**
108			* @param string $filename
109			*/
110			protected function publishMigration($filename)
111			{
112			$extension = '.php';
113			$filename = trim($filename, $extension).$extension;
114			$stub = __DIR__.'/../migrations/'.$filename;
115			$target = $this->migrationFilepath($filename);
116			$this->publishes([$stub => $target], 'migrations');
117			}
118
119			/**
120			* @param string $filename
121			* @return string
122			*/
123			protected function migrationFilepath($filename)
124			{
125			if (function_exists('database_path')) {
126			return database_path('/migrations/'.$filename);
127			} else {
128			return base_path('/database/migrations/'.$filename);
129			}
130			}
131			}
132

fenos / Notifynder

Push — version-4 ( 5bb021 )

NotifynderServiceProvider::migration() F

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like