Funivan\Console\SingleState\SingleStateConfigurator

Complexity

Total Complexity

7

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
C	configureSingleState()	0	36	7

<?php

  namespace Funivan\Console\SingleState;

  use Funivan\Console\ConsoleCommand;
  use Symfony\Component\Console\ConsoleEvents;
  use Symfony\Component\Console\Event\ConsoleCommandEvent;
  use Symfony\Component\Console\Event\ConsoleTerminateEvent;
  use Symfony\Component\EventDispatcher\EventDispatcher;

  /**
   * @author Ivan Shcherbak <dev@funivan.com>
   */
  class SingleStateConfigurator implements ConsoleSingleStateConfiguratorInterface {

    /**
     * @inheritdoc
     */
    public function configureSingleState(EventDispatcher $dispatcher) {
      $dispatcher->addListener(ConsoleEvents::COMMAND, function (ConsoleCommandEvent $event) {

        /** @var ConsoleCommand $command */
        $command = $event->getCommand();

        if (!($command instanceof ConsoleCommand) or ($command->isSingleInstance() == false)) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

          return;
        }


        $commandState = (new CommandState($command));
        if ($commandState->isAlive()) {
          $event->getOutput()->writeln('Already running: ' . $command->getName() . ' Pid:' . $commandState->getPid());
          $event->disableCommand();
        } else {
          $commandState->setPid(getmypid());
        }

      });

      $dispatcher->addListener(ConsoleEvents::TERMINATE, function (ConsoleTerminateEvent $event) {
        /** @var ConsoleCommand $command */
        $command = $event->getCommand();

        if (!($command instanceof ConsoleCommand) or ($command->isSingleInstance() == false)) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

          return;
        }

        if ($event->getExitCode() === 0) {
          (new CommandState($command))->unlink();
        }

      });
      return $dispatcher;
    }
  }