Startwind\Inventorio\Command\DaemonCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	execute()	0	22	3

<?php

namespace Startwind\Inventorio\Command;

use Startwind\Inventorio\Remote\RemoteConnect;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class DaemonCommand extends InventorioCommand
{
    protected static $defaultName = 'daemon';
    protected static $defaultDescription = 'Start long running daemon';

    /**
     * @inheritDoc
     */
    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $this->initConfiguration($input->getOption('configFile'));

        $lastRun = time() - 7200;

        $remoteConnect = new RemoteConnect(
            $this->config->getInventorioServer(),
            $this->getServerId(),
            $this->config->getCommands()
        );

        while (true) {
            if ($lastRun < time() - 3600) {
                $command = $this->getApplication()->find('collect');
                $command->run($input, $output);
                $lastRun = time();
            }

            $remoteConnect->run();

            sleep(10);
        }

In this branch, the function will implicitly return null which is incompatible with the type-hinted return integer. Consider adding a return statement or allowing null as return value.

    }
}