NotificationChannels\ExpoPushNotifications\ExpoPushNotificationsServiceProvider

Complexity

Total Complexity

8

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

81.25%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	16	1
A	register()	0	4	1
A	getInterestsDriver()	0	12	2
A	setupConfig()	0	8	1
A	shouldPublishMigrations()	0	9	3

<?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);

It seems like $repository defined by $this->getInterestsDriver() on line 23 can also be of type null; however, NotificationChannels\ExpoPushNotifications\ExpoPushNotificationsServiceProvider::shouldPublishMigrations() does only seem to accept object<ExponentPhpSDK\ExpoRepository>, maybe add an additional type check?

        $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;

break is not strictly necessary here and could be removed.

            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');
        }
    }
}