refactor(core): style improvements

Author: Guikingone

.php-cs-fixer.dist.php

@@ -37,12 +37,17 @@
     'no_trailing_comma_in_singleline_array' => true,
     'no_unused_imports' => true,
     'no_useless_return' => true,
+    'ordered_imports' => true,
     'return_assignment' => true,
+    'semicolon_after_instruction' => true,
     'short_scalar_cast' => true,
     'simplified_null_return' => true,
     'single_line_throw' => true,
+    'static_lambda' => true,
     'strict_comparison' => true,
     'strict_param' => true,
+    'ternary_to_null_coalescing' => true,
     'use_arrow_functions' => true,
     'void_return' => true,
+    'yoda_style' => true,
 ])->setFinder($finder);

src/Bridge/Doctrine/Transport/Configuration/Connection.php

@@ -4,6 +4,9 @@
 
 namespace SchedulerBundle\Bridge\Doctrine\Transport\Configuration;
 
+use function array_map;
+use function array_walk;
+
 use Closure;
 use Doctrine\DBAL\Connection as DbalConnection;
 use Doctrine\DBAL\Exception;
@@ -18,12 +21,10 @@
 use SchedulerBundle\Exception\LogicException;
 use SchedulerBundle\Exception\RuntimeException;
 use SchedulerBundle\Exception\TransportException;
+
 use SchedulerBundle\Transport\Configuration\ExternalConnectionInterface;
 use Throwable;
 
-use function array_map;
-use function array_walk;
-
 /**
  * @author Guillaume Loulier <[email protected]>
  */

src/Bridge/Doctrine/Transport/Configuration/DoctrineConfigurationFactory.php

@@ -11,10 +11,11 @@
 use SchedulerBundle\Exception\InvalidArgumentException;
 use SchedulerBundle\Transport\Configuration\ConfigurationFactoryInterface;
 use SchedulerBundle\Transport\Dsn;
-use Symfony\Component\Serializer\SerializerInterface;
 
 use function sprintf;
 
+use Symfony\Component\Serializer\SerializerInterface;
+
 /**
  * @author Guillaume Loulier <[email protected]>
  */

src/Bridge/Doctrine/Transport/Connection.php

@@ -4,13 +4,20 @@
 
 namespace SchedulerBundle\Bridge\Doctrine\Transport;
 
+use function array_map;
+
 use Doctrine\DBAL\Connection as DBALConnection;
 use Doctrine\DBAL\ParameterType;
 use Doctrine\DBAL\Result;
 use Doctrine\DBAL\Schema\Schema;
 use Doctrine\DBAL\Types\Types;
 use Doctrine\ORM\Query\Expr;
 use Exception;
+
+use const FILTER_VALIDATE_BOOLEAN;
+
+use function filter_var;
+
 use SchedulerBundle\Bridge\Doctrine\Connection\AbstractDoctrineConnection;
 use SchedulerBundle\Exception\InvalidArgumentException;
 use SchedulerBundle\Exception\LogicException;
@@ -20,15 +27,14 @@
 use SchedulerBundle\Task\TaskList;
 use SchedulerBundle\Task\TaskListInterface;
 use SchedulerBundle\Transport\Configuration\ConfigurationInterface;
+
 use SchedulerBundle\Transport\ConnectionInterface;
-use Symfony\Component\Serializer\SerializerInterface;
-use Throwable;
 
-use function array_map;
-use function filter_var;
 use function sprintf;
 
-use const FILTER_VALIDATE_BOOLEAN;
+use Symfony\Component\Serializer\SerializerInterface;
+
+use Throwable;
 
 /**
  * @author Guillaume Loulier <[email protected]>
@@ -49,7 +55,7 @@ public function __construct(
      */
     public function list(): TaskListInterface
     {
-        $existingTasksCount = $this->createQueryBuilder(table: $this->configuration->get(key: 'table_name'), alias: 't')
+        $existingTasksCount = $this->createQueryBuilder(table: (string) $this->configuration->get(key: 'table_name'), alias: 't')
             ->select(select: (new Expr())->countDistinct(x: 't.id'))
         ;
 

src/Bridge/Doctrine/Transport/DoctrineTransportFactory.php

@@ -13,10 +13,11 @@
 use SchedulerBundle\Transport\Configuration\ConfigurationInterface;
 use SchedulerBundle\Transport\Dsn;
 use SchedulerBundle\Transport\TransportFactoryInterface;
-use Symfony\Component\Serializer\SerializerInterface;
 
 use function sprintf;
 
+use Symfony\Component\Serializer\SerializerInterface;
+
 /**
  * @author Guillaume Loulier <[email protected]>
  */

src/Bridge/Redis/Transport/Connection.php

@@ -4,6 +4,8 @@
 
 namespace SchedulerBundle\Bridge\Redis\Transport;
 
+use function array_map;
+
 use Redis;
 use SchedulerBundle\Exception\InvalidArgumentException;
 use SchedulerBundle\Exception\TransportException;
@@ -12,12 +14,12 @@
 use SchedulerBundle\Task\TaskListInterface;
 use SchedulerBundle\Transport\Configuration\ConfigurationInterface;
 use SchedulerBundle\Transport\ConnectionInterface;
-use Symfony\Component\Serializer\SerializerInterface;
-use Throwable;
 
-use function array_map;
 use function sprintf;
 
+use Symfony\Component\Serializer\SerializerInterface;
+use Throwable;
+
 /**
  * @author Guillaume Loulier <[email protected]>
  */

src/Bridge/Redis/Transport/RedisTransportFactory.php

@@ -4,17 +4,19 @@
 
 namespace SchedulerBundle\Bridge\Redis\Transport;
 
+use function class_exists;
+use function phpversion;
+
 use Redis;
 use SchedulerBundle\Exception\LogicException;
 use SchedulerBundle\Exception\RuntimeException;
 use SchedulerBundle\SchedulePolicy\SchedulePolicyOrchestratorInterface;
 use SchedulerBundle\Transport\Configuration\ConfigurationInterface;
 use SchedulerBundle\Transport\Dsn;
+
 use SchedulerBundle\Transport\TransportFactoryInterface;
 use Symfony\Component\Serializer\SerializerInterface;
 
-use function class_exists;
-use function phpversion;
 use function version_compare;
 
 /**

src/Command/ConsumeTasksCommand.php

@@ -4,35 +4,40 @@
 
 namespace SchedulerBundle\Command;
 
+use function array_pop;
+
 use DateTimeImmutable;
+
+use function implode;
+use function in_array;
+
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
+use SchedulerBundle\Event\TaskExecutedEvent;
 use SchedulerBundle\Event\WorkerSleepingEvent;
+use SchedulerBundle\EventListener\StopWorkerOnFailureLimitSubscriber;
+use SchedulerBundle\EventListener\StopWorkerOnTaskLimitSubscriber;
+use SchedulerBundle\EventListener\StopWorkerOnTimeLimitSubscriber;
+use SchedulerBundle\SchedulerInterface;
+use SchedulerBundle\Task\Output;
 use SchedulerBundle\Task\ProbeTask;
 use SchedulerBundle\Task\TaskInterface;
 use SchedulerBundle\Worker\WorkerConfiguration;
+use SchedulerBundle\Worker\WorkerInterface;
+
+use function sprintf;
+
 use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\Helper;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
+
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
-use SchedulerBundle\Event\TaskExecutedEvent;
-use SchedulerBundle\EventListener\StopWorkerOnFailureLimitSubscriber;
-use SchedulerBundle\EventListener\StopWorkerOnTaskLimitSubscriber;
-use SchedulerBundle\EventListener\StopWorkerOnTimeLimitSubscriber;
-use SchedulerBundle\SchedulerInterface;
-use SchedulerBundle\Task\Output;
-use SchedulerBundle\Worker\WorkerInterface;
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Throwable;
 
-use function array_pop;
-use function implode;
-use function in_array;
-use function sprintf;
-
 /**
  * @author Guillaume Loulier <[email protected]>
  */
@@ -123,7 +128,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         if (false === $force) {
-            $nonPausedTasks = $dueTasks->filter(static fn (TaskInterface $task): bool => $task->getState() !== TaskInterface::PAUSED);
+            $nonPausedTasks = $dueTasks->filter(static fn (TaskInterface $task): bool => TaskInterface::PAUSED !== $task->getState());
             if (0 === $nonPausedTasks->count()) {
                 $symfonyStyle->warning([
                     'Each tasks has already been executed for the current minute',

src/Command/DebugConfigurationCommand.php

@@ -5,14 +5,16 @@
 namespace SchedulerBundle\Command;
 
 use SchedulerBundle\Transport\Configuration\ConfigurationInterface;
+
+use function sprintf;
+
 use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\Table;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
 
-use function sprintf;
+use Symfony\Component\Console\Style\SymfonyStyle;
 
 /**
  * @author Guillaume Loulier <[email protected]>

src/Command/DebugMiddlewareCommand.php

@@ -4,6 +4,9 @@
 
 namespace SchedulerBundle\Command;
 
+use function array_map;
+use function count;
+
 use ReflectionClass;
 use SchedulerBundle\Middleware\OrderedMiddlewareInterface;
 use SchedulerBundle\Middleware\PostExecutionMiddlewareInterface;
@@ -13,17 +16,17 @@
 use SchedulerBundle\Middleware\RequiredMiddlewareInterface;
 use SchedulerBundle\Middleware\SchedulerMiddlewareStackInterface;
 use SchedulerBundle\Middleware\WorkerMiddlewareStackInterface;
+
+use function sprintf;
+
 use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\Table;
+
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
-use function array_map;
-use function count;
-use function sprintf;
-
 /**
  * @author Guillaume Loulier <[email protected]>
  */