App\BxConsole\LockableTrait::lock()

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function lock($blocking = false) {

        if (null !== $this->_lock) {
            throw new LogicException('A lock is already in place.');
        }

        $lockStore = new FlockStore(pathinfo(EnvHelper::getCrontabFile(), PATHINFO_DIRNAME));

It seems like pathinfo(App\BxConsole\EnvHelper::getCrontabFile(), App\BxConsole\PATHINFO_DIRNAME) can also be of type array; however, parameter $lockPath of Symfony\Component\Lock\Store\FlockStore::__construct() does only seem to accept null|string, maybe add an additional type check?

        $lockFactory = new LockFactory($lockStore);

        $this->_lock = $lockFactory->createLock($this->getLockName($this->getName()), EnvHelper::getCrontabTimeout());

The method getName() does not exist on App\BxConsole\LockableTrait. Did you maybe mean getLockName()?

        if (!$this->_lock->acquire($blocking)) {
            $this->_lock = null;

            return false;
        }

        return true;
    }
}