ExpoPushNotificationsServiceProvider::shouldPublishMigrations() - Code Metrics - Inspection of "Merge pull request #73 from Alymosul/analysis-nNGk..." - Alymosul/laravel-exponent-push-notifications - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 9db785...3caf5f )

by Aly

created 2021-02-08 11:17 UTC

shouldPublishMigrations() A

↳ Parent: ExpoPushNotificationsServiceProvider

Complexity

Conditions	3
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	4.679

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	3
cts	7
cp	0.4286
rs	9.9666
c	0
b	0
f	0
cc	3
nc	2
nop	1
crap	4.679

<?php

namespace NotificationChannels\ExpoPushNotifications;

use ExponentPhpSDK\Expo;
use ExponentPhpSDK\ExpoRegistrar;
use ExponentPhpSDK\ExpoRepository;
use ExponentPhpSDK\Repositories\ExpoFileDriver;
use Illuminate\Support\ServiceProvider;
use NotificationChannels\ExpoPushNotifications\Repositories\ExpoDatabaseDriver;

class ExpoPushNotificationsServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap the application services.
     *
     * @return void
     */
    public function boot()
    {
        $this->setupConfig();

        $repository = $this->getInterestsDriver();

        $this->shouldPublishMigrations($repository);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

        $this->app->when(ExpoChannel::class)
            ->needs(Expo::class)
            ->give(function () use ($repository) {
                return new Expo(new ExpoRegistrar($repository));
            });

        $router = $this->app['router'];
        $router->middlewareGroup('expo.middleware', config('exponent-push-notifications')['middleware']);

        $this->loadRoutesFrom(__DIR__.'/Http/routes.php');
    }

    /**
     * Register the application services.
     *
     * @return void
     */
    public function register()
    {
        $this->app->bind(ExpoRepository::class, get_class($this->getInterestsDriver()));
    }

    /**
     * Gets the Expo repository driver based on config.
     *
     * @return ExpoRepository
     */
    public function getInterestsDriver()
    {
        $driver = config('exponent-push-notifications.interests.driver');

        switch ($driver) {
            case 'database':
                return new ExpoDatabaseDriver();
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            default:
                return new ExpoFileDriver();
        }
    }

    /**
     * Publishes the configuration files for the package.
     *
     * @return void
     */
    protected function setupConfig()
    {
        $this->publishes([
            __DIR__.'/../config/exponent-push-notifications.php' => config_path('exponent-push-notifications.php'),
        ], 'config');

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

    /**
     * Publishes the migration files needed in the package.
     *
     * @param ExpoRepository $repository
     *
     * @return void
     */
    private function shouldPublishMigrations(ExpoRepository $repository)
    {
        if ($repository instanceof ExpoDatabaseDriver && ! class_exists('CreateExponentPushNotificationInterestsTable')) {
            $timestamp = date('Y_m_d_His', time());
            $this->publishes([
                __DIR__.'/../migrations/create_exponent_push_notification_interests_table.php.stub' => database_path("/migrations/{$timestamp}_create_exponent_push_notification_interests_table.php"),
            ], 'migrations');
        }
    }
}


1		<?php
2
3		namespace NotificationChannels\ExpoPushNotifications;
4
5		use ExponentPhpSDK\Expo;
6		use ExponentPhpSDK\ExpoRegistrar;
7		use ExponentPhpSDK\ExpoRepository;
8		use ExponentPhpSDK\Repositories\ExpoFileDriver;
9		use Illuminate\Support\ServiceProvider;
10		use NotificationChannels\ExpoPushNotifications\Repositories\ExpoDatabaseDriver;
11
12		class ExpoPushNotificationsServiceProvider extends ServiceProvider
13		{
14		/**
15		* Bootstrap the application services.
16		*
17		* @return void
18		*/
19	23	public function boot()
20		{
21	23	$this->setupConfig();
22
23	23	$repository = $this->getInterestsDriver();
24
25	23	$this->shouldPublishMigrations($repository);
		0 ignored issues – show Bug introduced 2018-09-27 13:02 UTC by Report Bug Copy Issue Report It seems like `$repository` defined by `$this->getInterestsDriver()` on line `23` can also be of type `null`; however, `NotificationChannels\Exp...ouldPublishMigrations()` does only seem to accept `object<ExponentPhpSDK\ExpoRepository>`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
26
27	23	$this->app->when(ExpoChannel::class)
28	23	->needs(Expo::class)
29		->give(function () use ($repository) {
30		return new Expo(new ExpoRegistrar($repository));
31	23	});
32
33	23	$router = $this->app['router'];
34	23	$router->middlewareGroup('expo.middleware', config('exponent-push-notifications')['middleware']);
35
36	23	$this->loadRoutesFrom(__DIR__.'/Http/routes.php');
37	23	}
38
39		/**
40		* Register the application services.
41		*
42		* @return void
43		*/
44	23	public function register()
45		{
46	23	$this->app->bind(ExpoRepository::class, get_class($this->getInterestsDriver()));
47	23	}
48
49		/**
50		* Gets the Expo repository driver based on config.
51		*
52		* @return ExpoRepository
53		*/
54	23	public function getInterestsDriver()
55		{
56	23	$driver = config('exponent-push-notifications.interests.driver');
57
58	23	switch ($driver) {
59	23	case 'database':
60		return new ExpoDatabaseDriver();
61		break;
		0 ignored issues – show Unused Code introduced 2018-09-03 07:38 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
62		default:
63	23	return new ExpoFileDriver();
64		}
65		}
66
67		/**
68		* Publishes the configuration files for the package.
69		*
70		* @return void
71		*/
72	23	protected function setupConfig()
73		{
74	23	$this->publishes([
75	23	__DIR__.'/../config/exponent-push-notifications.php' => config_path('exponent-push-notifications.php'),
76	23	], 'config');
77
78	23	$this->mergeConfigFrom(__DIR__.'/../config/exponent-push-notifications.php', 'exponent-push-notifications');
79	23	}
80
81		/**
82		* Publishes the migration files needed in the package.
83		*
84		* @param ExpoRepository $repository
85		*
86		* @return void
87		*/
88	23	private function shouldPublishMigrations(ExpoRepository $repository)
89		{
90	23	if ($repository instanceof ExpoDatabaseDriver && ! class_exists('CreateExponentPushNotificationInterestsTable')) {
91		$timestamp = date('Y_m_d_His', time());
92		$this->publishes([
93		__DIR__.'/../migrations/create_exponent_push_notification_interests_table.php.stub' => database_path("/migrations/{$timestamp}_create_exponent_push_notification_interests_table.php"),
94		], 'migrations');
95		}
96	23	}
97		}
98

Alymosul / laravel-exponent-push-notifications

Push — master ( 9db785...3caf5f )

shouldPublishMigrations() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like