DaemonCommand::execute() - Code Metrics - Inspection of "install script" - startwind/inventorio-command-line - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 202822...ce601f )

by Nils

created 2025-04-24 17:25 UTC

DaemonCommand::execute() A

↳ Parent: DaemonCommand

Complexity

Conditions	3
Paths	3

Size

Total Lines	22
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	13
c	1
b	0
f	0
nc	3
nop	2
dl	0
loc	22
rs	9.8333

<?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);
        }
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
    }
}


1			<?php
2
3			namespace Startwind\Inventorio\Command;
4
5			use Startwind\Inventorio\Remote\RemoteConnect;
6			use Symfony\Component\Console\Input\InputInterface;
7			use Symfony\Component\Console\Output\OutputInterface;
8
9			class DaemonCommand extends InventorioCommand
10			{
11			protected static $defaultName = 'daemon';
12			protected static $defaultDescription = 'Start long running daemon';
13
14			/**
15			* @inheritDoc
16			*/
17			protected function execute(InputInterface $input, OutputInterface $output): int
18			{
19			$this->initConfiguration($input->getOption('configFile'));
20
21			$lastRun = time() - 7200;
22
23			$remoteConnect = new RemoteConnect(
24			$this->config->getInventorioServer(),
25			$this->getServerId(),
26			$this->config->getCommands()
27			);
28
29			while (true) {
30			if ($lastRun < time() - 3600) {
31			$command = $this->getApplication()->find('collect');
32			$command->run($input, $output);
33			$lastRun = time();
34			}
35
36			$remoteConnect->run();
37
38			sleep(10);
39			}
			0 ignored issues – show Bug Best Practice introduced 2025-04-24 17:27 UTC by Report Bug Copy Issue Report 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. For hinted functions/methods where all return statements with the correct type are only reachable via conditions, ?null? gets implicitly returned which may be incompatible with the hinted type. Let?s take a look at an example: interface ReturnsInt { public function returnsIntHinted(): int; } class MyClass implements ReturnsInt { public function returnsIntHinted(): int { if (foo()) { return 123; } // here: null is implicitly returned } } Loading history...
40			}
41			}
42

startwind / inventorio-command-line

Push — master ( 202822...ce601f )

DaemonCommand::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like