ConnectionBasedLockAdapter::doAcquireLock() - Code Metrics - Inspection of "Implemented proper support for lock payloads to be..." - storeman-io/storeman - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9913ec...45b4e9 )

by Arne

created 2017-08-12 02:06 UTC

ConnectionBasedLockAdapter::doAcquireLock() A

↳ Parent: ConnectionBasedLockAdapter

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	14
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	7
nc	2
nop	1

<?php

namespace Archivr\LockAdapter;

use Archivr\ConnectionAdapter\ConnectionAdapterInterface;

class ConnectionBasedLockAdapter extends AbstractLockAdapter
{
    /**
     * @var ConnectionAdapterInterface
     */
    protected $connectionAdapter;

    public function __construct(ConnectionAdapterInterface $connectionAdapter)
    {
        $this->connectionAdapter = $connectionAdapter;
    }

    protected function doGetLock(string $name)
    {
        if (!$this->connectionAdapter->exists($this->getLockFileName($name)))
        {
            return null;
        }

        return Lock::fromPayload($this->connectionAdapter->read($this->getLockFileName($name)));
    }

    protected function doAcquireLock(string $name): bool
    {
        $lockFileName = $this->getLockFileName($name);
        $payload = $this->getNewLockPayload($name);

        if ($this->connectionAdapter->exists($lockFileName))
        {
            return false;
        }

        $this->connectionAdapter->write($lockFileName, $payload);

        return true;
    }

    protected function doReleaseLock(string $name)
    {
        $this->connectionAdapter->unlink($this->getLockFileName($name));
    }

    protected function getLockFileName(string $lockName): string
    {
        return $lockName . '.lock';
    }
}


1			<?php
2
3			namespace Archivr\LockAdapter;
4
5			use Archivr\ConnectionAdapter\ConnectionAdapterInterface;
6
7			class ConnectionBasedLockAdapter extends AbstractLockAdapter
8			{
9			/**
10			* @var ConnectionAdapterInterface
11			*/
12			protected $connectionAdapter;
13
14			public function __construct(ConnectionAdapterInterface $connectionAdapter)
15			{
16			$this->connectionAdapter = $connectionAdapter;
17			}
18
19			protected function doGetLock(string $name)
20			{
21			if (!$this->connectionAdapter->exists($this->getLockFileName($name)))
22			{
23			return null;
24			}
25
26			return Lock::fromPayload($this->connectionAdapter->read($this->getLockFileName($name)));
27			}
28
29			protected function doAcquireLock(string $name): bool
30			{
31			$lockFileName = $this->getLockFileName($name);
32			$payload = $this->getNewLockPayload($name);
33
34			if ($this->connectionAdapter->exists($lockFileName))
35			{
36			return false;
37			}
38
39			$this->connectionAdapter->write($lockFileName, $payload);
40
41			return true;
42			}
43
44			protected function doReleaseLock(string $name)
45			{
46			$this->connectionAdapter->unlink($this->getLockFileName($name));
47			}
48
49			protected function getLockFileName(string $lockName): string
50			{
51			return $lockName . '.lock';
52			}
53			}
54

storeman-io / storeman

Push — master ( 9913ec...45b4e9 )

ConnectionBasedLockAdapter::doAcquireLock() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like