Spatie\UptimeMonitor\Commands\CreateMonitor

Complexity

Total Complexity

5

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	24	5

<?php

namespace Spatie\UptimeMonitor\Commands;

use Spatie\UptimeMonitor\Models\Monitor;
use Spatie\Url\Url;

class CreateMonitor extends BaseCommand
{
    protected $signature = 'monitor:create {url}';

    protected $description = 'Create a monitor';

    public function handle()
    {
        $url = Url::fromString($this->argument('url'));

It seems like $this->argument('url') targeting Illuminate\Console\Concerns\InteractsWithIO::argument() can also be of type array or null; however, Spatie\Url\Url::fromString() does only seem to accept string, maybe add an additional type check?

        if (! in_array($url->getScheme(), ['http', 'https'])) {
            if ($scheme = $this->choice("Which protocol needs to be used for checking `{$url}`?", [1 => 'https', 2 => 'http'], 1)) {
                $url = $url->withScheme($scheme);

It seems like $scheme defined by $this->choice("Which protocol needs to be used for checking `{$url}`?", array(1 => 'https', 2 => 'http'), 1) on line 19 can also be of type array; however, Spatie\Url\Url::withScheme() does only seem to accept string, maybe add an additional type check?

            }
        }

        if ($this->confirm('Should we look for a specific string on the response?')) {
            $lookForString = $this->ask('Which string?');
        }

        $monitor = Monitor::create([
            'url' => trim($url, '/'),
            'look_for_string' => $lookForString ?? '',
            'uptime_check_method' => isset($lookForString) ? 'get' : 'head',
            'certificate_check_enabled' => $url->getScheme() === 'https',
            'uptime_check_interval_in_minutes' => config('uptime-monitor.uptime_check.run_interval_in_minutes'),
        ]);

        $this->warn("{$monitor->url} will be monitored!");
    }
}