platine-php / framework

src/Console/BaseMakeActionCommand.php

Braces +32 added lines, -33 removed lines

@@ -59,8 +59,7 @@ 
  * @class BaseMakeActionCommand

  * @package Platine\Framework\Console

  */

-abstract class BaseMakeActionCommand extends MakeCommand

-{

+abstract class BaseMakeActionCommand extends MakeCommand {
     /**

      * The form parameter class name

      * @var class-string

@@ -93,7 +92,7 @@ 
     public function __construct(

         Application $application,

         Filesystem $filesystem

-    ) {

+    ) {
         parent::__construct($application, $filesystem);

 

         $this->addOption(

@@ -213,14 +212,14 @@ 
 

         $writer->boldYellow('Enter the properties list (empty value to finish):', true);

         $value = '';

-        while ($value !== null) {

+        while ($value !== null) {
             $value = $io->prompt('Property full class name', null, null, false);

 

-            if (!empty($value)) {

+            if (!empty($value)) {
                 $value = trim($value);

-                if (!class_exists($value) && !interface_exists($value)) {

+                if (!class_exists($value) && !interface_exists($value)) {
                     $writer->boldWhiteBgRed(sprintf('The class [%s] does not exists', $value), true);

-                } else {

+                } else {
                     $shortClass = $this->getClassBaseName($value);

                     $name = Str::camel($shortClass, true);

                     //replace"interface", "abstract"

@@ -244,14 +243,14 @@ 
         $io = $this->io();

 

         $paramClass = $io->prompt('Enter the form parameter full class name', null);

-        while (!class_exists($paramClass)) {

+        while (!class_exists($paramClass)) {
             $paramClass = $io->prompt('Class does not exists, please enter the form parameter full class name', null);

         }

 

         $this->paramClass = $paramClass;

 

         $validatorClass = $io->prompt('Enter the form validator full class name', null);

-        while (!class_exists($validatorClass)) {

+        while (!class_exists($validatorClass)) {
             $validatorClass = $io->prompt(

                 'Class does not exists, please enter the form validator full class name',

                 null

@@ -261,14 +260,14 @@ 
         $this->validatorClass = $validatorClass;

 

         $entityClass = $io->prompt('Enter the entity full class name', null);

-        while (!class_exists($entityClass)) {

+        while (!class_exists($entityClass)) {
             $entityClass = $io->prompt('Class does not exists, please enter the entity full class name', null);

         }

 

         $this->entityClass = $entityClass;

 

         $repositoryClass = $io->prompt('Enter the repository full class name', null);

-        while (!class_exists($repositoryClass)) {

+        while (!class_exists($repositoryClass)) {
             $repositoryClass = $io->prompt('Class does not exists, please enter the repository full class name', null);

         }

 

@@ -284,7 +283,7 @@ 
     protected function addProperty(string $value, ?string $name = null): self

     {

         $shortClass = $this->getClassBaseName($value);

-        if ($name === null) {

+        if ($name === null) {
             $name = Str::camel($shortClass, true);

         }

 

@@ -306,7 +305,7 @@ 
      */

     protected function getPropertyName(string $value): string

     {

-        if (!isset($this->properties[$value])) {

+        if (!isset($this->properties[$value])) {
             return '';

         }

 

@@ -321,7 +320,7 @@ 
     protected function getTemplatePrefix(): string

     {

         $templatePrefix = $this->getOptionValue('templatePrefix');

-        if ($templatePrefix === null) {

+        if ($templatePrefix === null) {
             $actionName = $this->getShortClassName($this->className);

             $templatePrefix = Str::snake(str_ireplace('action', '', $actionName));

         }

@@ -337,10 +336,10 @@ 
     protected function getEntityContextKey(bool $isKey = true): string

     {

         $key = (string) $this->getOptionValue('entityContextKey');

-        if (!empty($key)) {

-            if ($isKey) {

+        if (!empty($key)) {
+            if ($isKey) {
                 $key = Str::snake($key, '_');

-            } else {

+            } else {
                 $key = Str::camel($key, true);

             }

         }

@@ -355,7 +354,7 @@ 
     protected function getRoutePrefix(): string

     {

         $routePrefix = $this->getOptionValue('routePrefix');

-        if ($routePrefix === null) {

+        if ($routePrefix === null) {
             $actionName = $this->getShortClassName($this->className);

             $routePrefix = Str::snake(str_ireplace('action', '', $actionName));

         }

@@ -392,7 +391,7 @@ 
     protected function getMessage(string $option): ?string

     {

         $message = (string) $this->getOptionValue($option);

-        if (!empty($message)) {

+        if (!empty($message)) {
             $message = addslashes($message);

         }

 

@@ -424,15 +423,15 @@ 
     {

         $result = [];

         $fields = (array) explode(',', $values);

-        foreach ($fields as $field) {

+        foreach ($fields as $field) {
             $column = $field;

             $param = $field;

             $value = (array) explode(':', $field);

-            if (isset($value[0])) {

+            if (isset($value[0])) {
                 $column = $value[0];

             }

 

-            if (isset($value[1])) {

+            if (isset($value[1])) {
                 $param = $value[1];

             }

 

@@ -451,23 +450,23 @@ 
     protected function formatFieldStr(array $fields, bool $orderField = false): string

     {

         $result = '';

-        foreach ($fields as $field => $param) {

-            if ($orderField) {

+        foreach ($fields as $field => $param) {
+            if ($orderField) {
                 $param = Str::upper($param);

                 $order = 'ASC';

-                if ($param === 'DESC') {

+                if ($param === 'DESC') {
                     $order = 'DESC';

                 }

 

-                if ($order === 'ASC') {

+                if ($order === 'ASC') {
                     $result .= sprintf('\'%s\', ', $field);

-                } else {

+                } else {
                     $result .= sprintf('\'%s\' => \'DESC\', ', $field);

                 }

-            } else {

-                if ($field === $param) {

+            } else {
+                if ($field === $param) {
                     $result .= sprintf('\'%s\', ', $field);

-                } else {

+                } else {
                     $result .= sprintf('\'%s\' => \'%s\', ', $field, $param);

                 }

             }

@@ -483,13 +482,13 @@ 
     protected function loadConfig(): void

     {

         $filename = $this->getOptionValue('config');

-        if (!empty($filename)) {

+        if (!empty($filename)) {
             $file = $this->filesystem->file($filename);

-            if ($file->exists() && $file->isReadable()) {

+            if ($file->exists() && $file->isReadable()) {
                 $content = $file->read();

                 /** @var array<string, string> $config */

                 $config = Json::decode($content, true);

-                foreach ($config as $option => $value) {

+                foreach ($config as $option => $value) {
                     $optionKey = Str::camel($option, true);

                     $this->values[$optionKey] = $value;

                 }

src/Console/PasswordGenerateCommand.php

Braces +4 added lines, -6 removed lines

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

  * @package Platine\Framework\Console

  */

-class PasswordGenerateCommand extends Command

-{

+class PasswordGenerateCommand extends Command {
     /**

      * The Hash instance

      * @var HashInterface

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

     public function __construct(

         HashInterface $hash

-    ) {

+    ) {
         parent::__construct('password:generate', 'Command to generate password');

         $this->hash = $hash;

         $this->addArgument('password', 'The password to hash', null, false);

@@ -85,8 +84,7 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

+    public function execute() {
         $io = $this->io();

         $writer = $io->writer();

         $password = $this->password;

@@ -109,7 +107,7 @@ 
     {

         $writer->boldYellow('GENERATION OF PASSWORD', true)->eol();

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

-        if (empty($password)) {

+        if (empty($password)) {
             $io = $this->io();

             $password = $io->prompt('Enter the plain password to generate', '');

         }

src/Pagination/ServerRequestUrlGenerator.php

Braces +8 added lines, -10 removed lines

@@ -54,8 +54,7 @@ 
  * @class ServerRequestUrlGenerator

  * @package Platine\Framework\Pagination

  */

-class ServerRequestUrlGenerator implements UrlGeneratorInterface

-{

+class ServerRequestUrlGenerator implements UrlGeneratorInterface {
     /**

      * The server request instance

      * @var ServerRequestInterface

@@ -79,8 +78,7 @@ 
      * @param ServerRequestInterface $request

      * @param string $queryName

      */

-    public function __construct(ServerRequestInterface $request, string $queryName)

-    {

+    public function __construct(ServerRequestInterface $request, string $queryName) {
         $this->request = $request;

         $this->queryVarName = $queryName;

 

@@ -114,18 +112,18 @@ 
         $page = $this->queryVarName;

 

         $query = '';

-        if (empty($queryString)) {

+        if (empty($queryString)) {
             $query = sprintf('?%s={num}', $page);

-        } else {

+        } else {
             $parts = explode($page . '=', $queryString);

             $count = count($parts);

 

-            if ($count === 1) {

+            if ($count === 1) {
                 $query = sprintf('?%s&%s={num}', $queryString, $page);

-            } else {

-                if (empty($parts[0])) {

+            } else {
+                if (empty($parts[0])) {
                     $query = sprintf('?%s={num}', $page);

-                } else {

+                } else {
                     $query = sprintf('?%s%s={num}', $parts[0], $page);

                 }

             }

src/Pagination/BootstrapRenderer.php

Indentation +2 added lines, -2 removed lines

@@ -79,8 +79,8 @@
         foreach ($pages as $page) {

             if ($page->getUrl() !== null) {

                 $html .= '<li class = "page-item' . ($page->isCurrent() ? '  active"' : '"')

-                      . '><a href = "' . $page->getUrl() . '" class="page-link">'

-                      . $page->getNumber() . '</a></li>';

+                        . '><a href = "' . $page->getUrl() . '" class="page-link">'

+                        . $page->getNumber() . '</a></li>';

             } else {

                 $html .= '<li class = "page-item disabled"><a href="#" class="page-link">'

                         . $page->getNumber() . '</a></li>';

Braces +7 added lines, -8 removed lines

@@ -54,20 +54,19 @@ 
  * @class BootstrapRenderer

  * @package Platine\Framework\Pagination

  */

-class BootstrapRenderer implements RendererInterface

-{

+class BootstrapRenderer implements RendererInterface {
     /**

      * {@inheritdoc}

      */

     public function render(Pagination $pagination): string

     {

-        if ($pagination->getTotalPages() <= 1) {

+        if ($pagination->getTotalPages() <= 1) {
             return '';

         }

 

         $html = '<ul class = "pagination">';

 

-        if ($pagination->hasPreviousPage()) {

+        if ($pagination->hasPreviousPage()) {
             $html .= '<li class = "page-item"><a href = "'

                     . $pagination->getPreviousUrl() . '" class="page-link">&laquo; '

                     . $pagination->getPreviousText() . '</a></li>';

@@ -76,18 +75,18 @@ 
         /** @var array<\Platine\Pagination\Page> $pages */

         $pages = $pagination->getPages();

 

-        foreach ($pages as $page) {

-            if ($page->getUrl() !== null) {

+        foreach ($pages as $page) {
+            if ($page->getUrl() !== null) {
                 $html .= '<li class = "page-item' . ($page->isCurrent() ? '  active"' : '"')

                       . '><a href = "' . $page->getUrl() . '" class="page-link">'

                       . $page->getNumber() . '</a></li>';

-            } else {

+            } else {
                 $html .= '<li class = "page-item disabled"><a href="#" class="page-link">'

                         . $page->getNumber() . '</a></li>';

             }

         }

 

-        if ($pagination->hasNextPage()) {

+        if ($pagination->hasNextPage()) {
             $html .= '<li class = "page-item"><a href = "'

                     . $pagination->getNextUrl() . '" class="page-link">'

                     . $pagination->getNextText() . ' &raquo;</a></li>';

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(),