fix cs styles

shapecode · Oct 26, 2024 · 6f4e5f7 · 6f4e5f7
1 parent 2d46e9d
commit 6f4e5f7
Show file tree

Hide file tree

Showing 10 changed files with 55 additions and 23 deletions.
diff --git a/phpcs.xml b/phpcs.xml
@@ -31,4 +31,10 @@
             <property name="lineLimit" value="160"/>
         </properties>
     </rule>
+
+    <rule ref="SlevomatCodingStandard.Classes.RequireMultiLineMethodSignature">
+        <properties>
+            <property name="minParametersCount" value="2"/>
+        </properties>
+    </rule>
 </ruleset>
diff --git a/src/Command/CronJobDisableCommand.php b/src/Command/CronJobDisableCommand.php
@@ -36,8 +36,10 @@ protected function configure(): void
             ->addArgument('job', InputArgument::REQUIRED, 'Name or id of the job to disable');
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
+    protected function execute(
+        InputInterface $input,
+        OutputInterface $output,
+    ): int {
         $io = new CronStyle($input, $output);
 
         $nameOrId = $input->getArgument('job');

diff --git a/src/Command/CronJobEnableCommand.php b/src/Command/CronJobEnableCommand.php
@@ -36,8 +36,10 @@ protected function configure(): void
             ->addArgument('job', InputArgument::REQUIRED, 'Name or id of the job to disable');
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
+    protected function execute(
+        InputInterface $input,
+        OutputInterface $output,
+    ): int {
         $io = new CronStyle($input, $output);
 
         $nameOrId = $input->getArgument('job');

diff --git a/src/Command/CronProcessCommand.php b/src/Command/CronProcessCommand.php
@@ -48,8 +48,10 @@ protected function configure(): void
         $this->addArgument('cron', InputArgument::REQUIRED);
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
+    protected function execute(
+        InputInterface $input,
+        OutputInterface $output,
+    ): int {
         $io = new CronStyle($input, $output);
 
         $job = $this->cronJobRepository->find($input->getArgument('cron'));
@@ -121,8 +123,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         return $statusCode;
     }
 
-    private function recordJobResult(CronJob $job, float $timeTaken, BufferedOutput $output, int $statusCode): void
-    {
+    private function recordJobResult(
+        CronJob $job,
+        float $timeTaken,
+        BufferedOutput $output,
+        int $statusCode,
+    ): void {
         $buffer = $output->isQuiet() ? null : $output->fetch();
 
         $result = new CronJobResult(

diff --git a/src/Command/CronRunCommand.php b/src/Command/CronRunCommand.php
@@ -41,8 +41,10 @@ public function __construct(
         parent::__construct();
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
+    protected function execute(
+        InputInterface $input,
+        OutputInterface $output,
+    ): int {
         $style = new CronStyle($input, $output);
         $now   = DateTime::createFromImmutable($this->clock->now());
 
@@ -123,7 +125,7 @@ private function waitProcesses(CronJobRunningCollection $processes): void
                 $this->entityManager->refresh($job);
                 $job->decreaseRunningInstances();
 
-                if ($job->getRunningInstances() == 0) {
+                if ($job->getRunningInstances() === 0) {
                     $job->calculateNextRun();
                 }
 

diff --git a/src/Command/CronScanCommand.php b/src/Command/CronScanCommand.php
@@ -42,8 +42,10 @@ protected function configure(): void
             ->addOption('default-disabled', 'd', InputOption::VALUE_NONE, 'If set, new jobs will be disabled by default');
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
+    protected function execute(
+        InputInterface $input,
+        OutputInterface $output,
+    ): int {
         $io = new CronStyle($input, $output);
         $io->comment(sprintf('Scan for cron jobs started at %s', $this->clock->now()->format('r')));
         $io->title('scanning ...');
@@ -123,8 +125,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         return Command::SUCCESS;
     }
 
-    private function newJobFound(CronStyle $io, CronJobMetadata $metadata, bool $defaultDisabled, int $counter): void
-    {
+    private function newJobFound(
+        CronStyle $io,
+        CronJobMetadata $metadata,
+        bool $defaultDisabled,
+        int $counter,
+    ): void {
         $newJob =
             CronJob::create(
                 $metadata->command,

diff --git a/src/Command/CronStatusCommand.php b/src/Command/CronStatusCommand.php
@@ -26,8 +26,10 @@ public function __construct(
         parent::__construct();
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
+    protected function execute(
+        InputInterface $input,
+        OutputInterface $output,
+    ): int {
         $io = new CronStyle($input, $output);
 
         $io->title('Cron job status');

diff --git a/src/DependencyInjection/ShapecodeCronExtension.php b/src/DependencyInjection/ShapecodeCronExtension.php
@@ -17,8 +17,10 @@
 final class ShapecodeCronExtension extends ConfigurableExtension
 {
     /** @param array<mixed> $mergedConfig */
-    protected function loadInternal(array $mergedConfig, ContainerBuilder $container): void
-    {
+    protected function loadInternal(
+        array $mergedConfig,
+        ContainerBuilder $container,
+    ): void {
         $locator = new FileLocator(__DIR__ . '/../Resources/config');
         $loader  = new Loader\YamlFileLoader($container, $locator);
         $loader->load('services.yml');

diff --git a/src/Entity/CronJob.php b/src/Entity/CronJob.php
@@ -60,8 +60,10 @@ public function __construct(
         $this->calculateNextRun();
     }
 
-    public static function create(string $command, string $period): self
-    {
+    public static function create(
+        string $command,
+        string $period,
+    ): self {
         return new self($command, $period);
     }
 

diff --git a/src/Repository/CronJobRepository.php b/src/Repository/CronJobRepository.php
@@ -18,8 +18,10 @@ public function __construct(ManagerRegistry $registry)
         parent::__construct($registry, CronJob::class);
     }
 
-    public function findOneByCommand(string $command, int $number = 1): CronJob|null
-    {
+    public function findOneByCommand(
+        string $command,
+        int $number = 1,
+    ): CronJob|null {
         return $this->findOneBy([
             'command' => $command,
             'number'  => $number,