EricMakesStuff\ServerMonitor\Notifications\EventHandler

Complexity

Total Complexity

6

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	2
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	whenDiskUsageAlarm()	0	4	1
A	whenDiskUsageHealthy()	0	4	1
A	whenHttpPingDown()	0	4	1
A	whenHttpPingUp()	0	4	1
A	subscribe()	0	22	1

<?php

namespace EricMakesStuff\ServerMonitor\Notifications;

use EricMakesStuff\ServerMonitor\Events\HttpPingDown;
use EricMakesStuff\ServerMonitor\Events\HttpPingUp;
use EricMakesStuff\ServerMonitor\Events\DiskUsageAlarm;
use EricMakesStuff\ServerMonitor\Events\DiskUsageHealthy;
use Illuminate\Events\Dispatcher;

class EventHandler
{
    /**
     * @var \EricMakesStuff\ServerMonitor\Notifications\Notifier
     */
    protected $notifier;

    public function __construct()
    {
        $notifierClass = config('server-monitor.notifications.handler');

        $this->notifier = app($notifierClass);
    }

    /**
     * @param \EricMakesStuff\ServerMonitor\Events\DiskUsageAlarm $event
     */
    public function whenDiskUsageAlarm(DiskUsageAlarm $event)
    {
        $this->notifier->diskUsageAlarm($event->diskUsageMonitor);

It seems like $event->diskUsageMonitor can be null; however, diskUsageAlarm() does not accept null, maybe add an additional type check?

    }

    /**
     * @param \EricMakesStuff\ServerMonitor\Events\DiskUsageHealthy $event
     */
    public function whenDiskUsageHealthy(DiskUsageHealthy $event)
    {
        $this->notifier->diskUsageHealthy($event->diskUsageMonitor);

It seems like $event->diskUsageMonitor can be null; however, diskUsageHealthy() does not accept null, maybe add an additional type check?

    }

    /**
     * @param \EricMakesStuff\ServerMonitor\Events\HttpPingDown $event
     */
    public function whenHttpPingDown(HttpPingDown $event)
    {
        $this->notifier->httpPingDown($event->httpPingMonitor);

It seems like $event->httpPingMonitor can be null; however, httpPingDown() does not accept null, maybe add an additional type check?

    }

    /**
     * @param \EricMakesStuff\ServerMonitor\Events\HttpPingUp $event
     */
    public function whenHttpPingUp(HttpPingUp $event)
    {
        $this->notifier->httpPingUp($event->httpPingMonitor);

It seems like $event->httpPingMonitor can be null; however, httpPingUp() does not accept null, maybe add an additional type check?

    }

    /**
     * Register the listeners for the subscriber.
     *
     * @param \Illuminate\Events\Dispatcher $events
     *
     * @return array
     */
    public function subscribe(Dispatcher $events)
    {
        $events->listen(
            DiskUsageHealthy::class,
            static::class.'@whenDiskUsageHealthy'
        );

        $events->listen(
            DiskUsageAlarm::class,
            static::class.'@whenDiskUsageAlarm'
        );

        $events->listen(
            HttpPingUp::class,
            static::class.'@whenHttpPingUp'
        );

        $events->listen(
            HttpPingDown::class,
            static::class.'@whenHttpPingDown'
        );
    }
}