Spatie\ServerMonitor\Commands\AddHost::handle()

Complexity

Conditions	5
Paths	16

Size

Total Lines

45

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle()
    {
        $this->info("Let's add a host!");

        $hostName = $this->ask('What is the name of the host');

        $sshUser = $this->confirm('Should a custom ssh user be used?')
            ? $this->ask('Which user?')
            : null;

        $port = $this->confirm('Should a custom port be used?')
            ? $this->ask('Which port?')
            : null;

        $ip = $this->confirm('Should a specific ip address be used?')
            ? $this->ask('Which ip address?')
            : null;

        $checkNames = array_merge([static::$allChecksLabel], $this->getAllCheckNames());

        $chosenChecks = $this->choice('Which checks should be performed?', $checkNames, 0, null, true);

        $chosenChecks = $this->determineChecks($chosenChecks, $checkNames);

It seems like $chosenChecks can also be of type string; however, Spatie\ServerMonitor\Commands\AddHost::determineChecks() does only seem to accept array, maybe add an additional type check?

        if ($this->determineHostModelClass()::where('name', $hostName)->first()) {

The method where cannot be called on $this->determineHostModelClass() (of type string).

            throw new InvalidArgumentException("Host `{$hostName}` already exists");
        }

        $this->determineHostModelClass()::create([

The method create cannot be called on $this->determineHostModelClass() (of type string).

            'name' => $hostName,
            'ssh_user' => $sshUser,
            'port' => $port,
            'ip' => $ip,
        ])->checks()->saveMany(collect($chosenChecks)->map(function (string $checkName) {
            $checkModel = $this->determineCheckModelClass();

            return new $checkModel([
                'type' => $checkName,
                'status' => CheckStatus::NOT_YET_CHECKED,
                'custom_properties' => [],
            ]);
        }));

        $this->info("Host `{$hostName}` added");
    }