platine-php / framework

src/Task/Scheduler.php

Indentation +2 added lines, -2 removed lines

@@ -150,8 +150,8 @@
             $task->run();

         } catch (Throwable $e) {

             $this->logger->error('Error occurred when execute task {task}, error: {error}', [

-               'task' => $task->name(),

-               'error' => $e->getMessage()

+                'task' => $task->name(),

+                'error' => $e->getMessage()

             ]);

         }

     }

Braces +7 added lines, -8 removed lines

@@ -56,8 +56,7 @@ 
  * @class Scheduler

  * @package Platine\Framework\Task

  */

-class Scheduler implements SchedulerInterface

-{

+class Scheduler implements SchedulerInterface {
     /**

      * The logger instance

      * @var LoggerInterface

@@ -76,7 +75,7 @@ 
      */

     public function __construct(

         LoggerInterface $logger

-    ) {

+    ) {
         $this->logger = $logger;

     }

 

@@ -87,8 +86,8 @@ 
     {

         $datetime = new DateTime();

         $tasks = $this->tasks;

-        foreach ($tasks as $task) {

-            if ($this->isDue($task, $datetime)) {

+        foreach ($tasks as $task) {
+            if ($this->isDue($task, $datetime)) {
                 $this->execute($task);

             }

         }

@@ -100,7 +99,7 @@ 
     public function add(TaskInterface $task): void

     {

         $key = $task->name();

-        if (array_key_exists($key, $this->tasks)) {

+        if (array_key_exists($key, $this->tasks)) {
             throw new RuntimeException(sprintf('Task [%s] already exist', $key));

         }

         $this->tasks[$key] = $task;

@@ -146,9 +145,9 @@ 
     public function execute(TaskInterface $task): void

     {

         $this->logger->info('Execute task {task}', ['task' => $task->name()]);

-        try {

+        try {
             $task->run();

-        } catch (Throwable $e) {

+        } catch (Throwable $e) {
             $this->logger->error('Error occurred when execute task {task}, error: {error}', [

                'task' => $task->name(),

                'error' => $e->getMessage()

src/Task/SchedulerInterface.php

Braces +1 added lines, -2 removed lines

@@ -51,8 +51,7 @@
  * @class SchedulerInterface

  * @package Platine\Framework\Task

  */

-interface SchedulerInterface

-{

+interface SchedulerInterface {
     /**

      * Execute the scheduler

      * @return void

src/Task/Command/SchedulerRunCommand.php

Indentation +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
         parent::__construct($scheduler, $application, $config);

 

         $this->setName('scheduler:run')

-             ->setDescription('Execute the scheduled tasks');

+                ->setDescription('Execute the scheduled tasks');

 

         $this->addArgument('name', 'the name of task to execute instead of all', null, false, true);

     }

Braces +5 added lines, -7 removed lines

@@ -57,8 +57,7 @@ 
  * @template T

  * @extends AbstractCommand<T>

  */

-class SchedulerRunCommand extends AbstractCommand

-{

+class SchedulerRunCommand extends AbstractCommand {
     /**

      * Create new instance

      * @param SchedulerInterface $scheduler

@@ -69,7 +68,7 @@ 
         SchedulerInterface $scheduler,

         Application $application,

         Config $config

-    ) {

+    ) {
         parent::__construct($scheduler, $application, $config);

 

         $this->setName('scheduler:run')

@@ -81,15 +80,14 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

+    public function execute() {
         //Load the task

         $this->loadTasks();

 

         $name = $this->getArgumentValue('name');

-        if (!empty($name)) {

+        if (!empty($name)) {
             $task = $this->scheduler->get($name);

-            if ($task !== null) {

+            if ($task !== null) {
                 $this->scheduler->execute($task);

                 return;

             }

src/Task/Command/AbstractCommand.php

Braces +7 added lines, -8 removed lines

@@ -59,8 +59,7 @@ 
  * @package Platine\Framework\Task\Command

  * @template T

  */

-abstract class AbstractCommand extends Command

-{

+abstract class AbstractCommand extends Command {
     /**

      * The scheduler instance

      * @var SchedulerInterface

@@ -89,7 +88,7 @@ 
         SchedulerInterface $scheduler,

         Application $application,

         Config $config

-    ) {

+    ) {
         parent::__construct('scheduler', 'Command to manage scheduler tasks');

         $this->scheduler = $scheduler;

         $this->application = $application;

@@ -103,7 +102,7 @@ 
      */

     protected function addTask($task): void

     {

-        if (is_string($task)) {

+        if (is_string($task)) {
             $task = $this->createTask($task);

         }

 

@@ -118,7 +117,7 @@ 
     {

         /** @var string[] $tasks */

         $tasks = $this->config->get('tasks', []);

-        foreach ($tasks as $task) {

+        foreach ($tasks as $task) {
             $this->addTask($task);

         }

     }

@@ -130,11 +129,11 @@ 
      */

     protected function createTask(string $task): TaskInterface

     {

-        if ($this->application->has($task)) {

+        if ($this->application->has($task)) {
             return $this->application->get($task);

         }

 

-        if (class_exists($task)) {

+        if (class_exists($task)) {
             /** @var TaskInterface $o */

             $o = new $task();

             return $o;

@@ -157,7 +156,7 @@ 
         //Load providers tasks

         /** @var class-string[] $tasks */

         $tasks = $this->application->getProvidersTasks();

-        foreach ($tasks as $task) {

+        foreach ($tasks as $task) {
             $this->addTask($task);

         }

     }

src/Task/Command/SchedulerListCommand.php

Indentation +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         parent::__construct($scheduler, $application, $config);

 

         $this->setName('scheduler:list')

-             ->setDescription('Show the list of scheduled tasks');

+                ->setDescription('Show the list of scheduled tasks');

     }

 

     /**

Braces +4 added lines, -6 removed lines

@@ -59,8 +59,7 @@ 
  * @template T

  * @extends AbstractCommand<T>

  */

-class SchedulerListCommand extends AbstractCommand

-{

+class SchedulerListCommand extends AbstractCommand {
     /**

      * Create new instance

      * @param SchedulerInterface $scheduler

@@ -71,7 +70,7 @@ 
         SchedulerInterface $scheduler,

         Application $application,

         Config $config

-    ) {

+    ) {
         parent::__construct($scheduler, $application, $config);

 

         $this->setName('scheduler:list')

@@ -81,8 +80,7 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

+    public function execute() {
         //Load the task list

         $this->loadTasks();

 

@@ -102,7 +100,7 @@ 
         $tasks = $this->scheduler->all();

 

         $rows = [];

-        foreach ($tasks as $task) {

+        foreach ($tasks as $task) {
             $nextExecution = date('Y-m-d H:i', Cron::parse($task->expression()));

             $rows[] = [

                 'name' => $task->name(),

src/Task/TaskInterface.php

Braces +1 added lines, -2 removed lines

@@ -51,8 +51,7 @@
  * @class TaskInterface

  * @package Platine\Framework\Task

  */

-interface TaskInterface

-{

+interface TaskInterface {
     /**

      * Execute the task

      * @return void

src/Task/Cron.php

Braces +8 added lines, -9 removed lines

@@ -53,8 +53,7 @@ 
  * @class Cron

  * @package Platine\Framework\Task

  */

-class Cron

-{

+class Cron {
     /**

      * Parse the given cron expression and finds next execution time(stamp)

      * @param string $expression

@@ -80,7 +79,7 @@ 
                 . '((\*(\/[0-9]+)?)|[0-9\-\,\/]+)\s+'

                 . '((\*(\/[0-9]+)?)|[0-9\-\,\/]+)\s+'

                 . '((\*(\/[0-9]+)?)|[0-9\-\,\/]+)$/i';

-        if (!preg_match($cronRegex, $cronExpression)) {

+        if (!preg_match($cronRegex, $cronExpression)) {
             throw new InvalidArgumentException(sprintf(

                 'Invalid cron expression [%s]',

                 $expression

@@ -88,12 +87,12 @@ 
         }

 

         $crons = preg_split('/[\s]+/i', $cronExpression);

-        if ($crons === false) {

+        if ($crons === false) {
             return 0;

         }

 

         $start = time();

-        if ($timestamp !== null) {

+        if ($timestamp !== null) {
             $start = $timestamp;

         }

 

@@ -109,7 +108,7 @@ 
         // to check more than 1 year ahead

         $total = 60 * 60 * 24 * 366;

 

-        for ($i = 0; $i <= $total; $i += 60) {

+        for ($i = 0; $i <= $total; $i += 60) {
             $current = $start + $i;

             if (

                 in_array((int) date('j', $current), $dates['dom']) &&

@@ -117,7 +116,7 @@ 
                 in_array((int) date('w', $current), $dates['dow']) &&

                 in_array((int) date('G', $current), $dates['hours']) &&

                 in_array((int) date('i', $current), $dates['minutes'])

-            ) {

+            ) {
                 return $current;

             }

         }

@@ -137,7 +136,7 @@ 
     {

         $result = [];

         $values = explode(',', $expression);

-        foreach ($values as $value) {

+        foreach ($values as $value) {
             $slashValues = explode('/', $value);

             $step = $slashValues[1] ?? 1;

             $minusValues = explode('-', $slashValues[0]);

@@ -147,7 +146,7 @@ 
             $maximum = count($minusValues) === 2 ? $minusValues[1]

                         : ($slashValues[0] === '*' ? $max : $slashValues[0]);

 

-            for ($i = $minimum; $i <= $maximum; $i += $step) {

+            for ($i = $minimum; $i <= $maximum; $i += $step) {
                 $result[$i] = intval($i);

             }

         }

src/Service/Provider/FilesystemServiceProvider.php

Braces +1 added lines, -2 removed lines

@@ -56,8 +56,7 @@
  * @class FilesystemServiceProvider

  * @package Platine\Framework\Service\Provider

  */

-class FilesystemServiceProvider extends ServiceProvider

-{

+class FilesystemServiceProvider extends ServiceProvider {
     /**

      * {@inheritdoc}

      */

src/Service/Provider/AuditServiceProvider.php

Braces +1 added lines, -2 removed lines

@@ -56,8 +56,7 @@
  * @class AuditServiceProvider

  * @package Platine\Framework\Service\Provider

  */

-class AuditServiceProvider extends ServiceProvider

-{

+class AuditServiceProvider extends ServiceProvider {
     /**

      * {@inheritdoc}

      */