AbstractLockedCommand::execute() - Code Metrics - Inspection of "Merge pull request #849 from acelaya-forks/feature..." - shlinkio/shlink - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 8109ce...b15e90 )

by Alejandro

created 2020-09-27 10:59 UTC

AbstractLockedCommand::execute() A

↳ Parent: AbstractLockedCommand

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	2

Importance

Changes

Metric	Value
eloc	10
dl	0
loc	16
ccs	9
cts	9
cp	1
rs	9.9332
c	0
b	0
f	0
cc	2
nc	2
nop	2
crap	2

<?php

declare(strict_types=1);

namespace Shlinkio\Shlink\CLI\Command\Util;

use Shlinkio\Shlink\CLI\Util\ExitCodes;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Lock\LockFactory;

use function sprintf;

abstract class AbstractLockedCommand extends Command
{
    private LockFactory $locker;

    public function __construct(LockFactory $locker)
    {
        parent::__construct();
        $this->locker = $locker;
    }

    final protected function execute(InputInterface $input, OutputInterface $output): ?int
    {
        $lockConfig = $this->getLockConfig();
        $lock = $this->locker->createLock($lockConfig->lockName(), $lockConfig->ttl(), $lockConfig->isBlocking());

        if (! $lock->acquire($lockConfig->isBlocking())) {
            $output->writeln(
                sprintf('<comment>Command "%s" is already in progress. Skipping.</comment>', $lockConfig->lockName()),
            );
            return ExitCodes::EXIT_WARNING;
        }

        try {
            return $this->lockedExecute($input, $output);
        } finally {
            $lock->release();
        }
    }

    abstract protected function lockedExecute(InputInterface $input, OutputInterface $output): int;

    abstract protected function getLockConfig(): LockedCommandConfig;
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Shlinkio\Shlink\CLI\Command\Util;
6
7		use Shlinkio\Shlink\CLI\Util\ExitCodes;
8		use Symfony\Component\Console\Command\Command;
9		use Symfony\Component\Console\Input\InputInterface;
10		use Symfony\Component\Console\Output\OutputInterface;
11		use Symfony\Component\Lock\LockFactory;
12
13		use function sprintf;
14
15		abstract class AbstractLockedCommand extends Command
16		{
17		private LockFactory $locker;
18
19	19	public function __construct(LockFactory $locker)
20		{
21	19	parent::__construct();
22	19	$this->locker = $locker;
23	19	}
24
25	16	final protected function execute(InputInterface $input, OutputInterface $output): ?int
26		{
27	16	$lockConfig = $this->getLockConfig();
28	16	$lock = $this->locker->createLock($lockConfig->lockName(), $lockConfig->ttl(), $lockConfig->isBlocking());
29
30	16	if (! $lock->acquire($lockConfig->isBlocking())) {
31	1	$output->writeln(
32	1	sprintf('<comment>Command "%s" is already in progress. Skipping.</comment>', $lockConfig->lockName()),
33		);
34	1	return ExitCodes::EXIT_WARNING;
35		}
36
37		try {
38	15	return $this->lockedExecute($input, $output);
39		} finally {
40	15	$lock->release();
41		}
42		}
43
44		abstract protected function lockedExecute(InputInterface $input, OutputInterface $output): int;
45
46		abstract protected function getLockConfig(): LockedCommandConfig;
47		}
48

shlinkio / shlink

Push — develop ( 8109ce...b15e90 )

AbstractLockedCommand::execute() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like