ExpoPushNotificationsServiceProvider::setupConfig() - Code Metrics - Inspection of "Added support for Laravel 6.0" - Alymosul/laravel-exponent-push-notifications - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Pull Request — master (#35)

by Peter

created 2019-09-16 10:45 UTC

setupConfig() A

↳ Parent: ExpoPushNotificationsServiceProvider

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	6
cts	6
cp	1
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	0
crap	1

<?php

namespace NotificationChannels\ExpoPushNotifications;

use ExponentPhpSDK\Expo;
use ExponentPhpSDK\ExpoRegistrar;
use ExponentPhpSDK\ExpoRepository;
use Illuminate\Support\ServiceProvider;
use ExponentPhpSDK\Repositories\ExpoFileDriver;
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));
            });

        $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 Illuminate\Support\ServiceProvider;
9		use ExponentPhpSDK\Repositories\ExpoFileDriver;
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	$this->loadRoutesFrom(__DIR__.'/Http/routes.php');
34	23	}
35
36		/**
37		* Register the application services.
38		*
39		* @return void
40		*/
41	23	public function register()
42		{
43	23	$this->app->bind(ExpoRepository::class, get_class($this->getInterestsDriver()));
44	23	}
45
46		/**
47		* Gets the Expo repository driver based on config.
48		*
49		* @return ExpoRepository
50		*/
51	23	public function getInterestsDriver()
52		{
53	23	$driver = config('exponent-push-notifications.interests.driver');
54
55	23	switch ($driver) {
56	23	case 'database':
57		return new ExpoDatabaseDriver();
58		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...
59		default:
60	23	return new ExpoFileDriver();
61		}
62		}
63
64		/**
65		* Publishes the configuration files for the package.
66		*
67		* @return void
68		*/
69	23	protected function setupConfig()
70		{
71	23	$this->publishes([
72	23	__DIR__.'/../config/exponent-push-notifications.php' => config_path('exponent-push-notifications.php'),
73	23	], 'config');
74
75	23	$this->mergeConfigFrom(__DIR__.'/../config/exponent-push-notifications.php', 'exponent-push-notifications');
76	23	}
77
78		/**
79		* Publishes the migration files needed in the package.
80		*
81		* @param ExpoRepository $repository
82		*
83		* @return void
84		*/
85	23	private function shouldPublishMigrations(ExpoRepository $repository)
86		{
87	23	if ($repository instanceof ExpoDatabaseDriver && ! class_exists('CreateExponentPushNotificationInterestsTable')) {
88		$timestamp = date('Y_m_d_His', time());
89		$this->publishes([
90		__DIR__.'/../migrations/create_exponent_push_notification_interests_table.php.stub' => database_path("/migrations/{$timestamp}_create_exponent_push_notification_interests_table.php"),
91		], 'migrations');
92		}
93	23	}
94		}
95

Alymosul / laravel-exponent-push-notifications

Pull Request — master (#35)

setupConfig() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like