platine-php / framework

src/Handler/Error/ErrorHandler.php

Braces +17 added lines, -19 removed lines

@@ -63,8 +63,7 @@ 
  * @class ErrorHandler

  * @package Platine\Framework\Handler\Error

  */

-class ErrorHandler implements ErrorHandlerInterface

-{

+class ErrorHandler implements ErrorHandlerInterface {
     /**

      * The content type

      * @var string

@@ -123,8 +122,7 @@ 
      * Create new instance

      * @param LoggerInterface $logger

      */

-    public function __construct(LoggerInterface $logger)

-    {

+    public function __construct(LoggerInterface $logger) {
         $this->logger = $logger;

 

         //Add default renderer

@@ -148,7 +146,7 @@ 
         $this->exception = $exception;

         $this->method = $request->getMethod();

         $this->statusCode = $this->determineStatusCode();

-        if (empty($this->contentType)) {

+        if (empty($this->contentType)) {
             $this->contentType = $this->getRequestContentType($request);

         }

 

@@ -206,11 +204,11 @@ 
      */

     protected function determineStatusCode(): int

     {

-        if ($this->method === 'OPTIONS') {

+        if ($this->method === 'OPTIONS') {
             return 200;

         }

 

-        if ($this->exception instanceof HttpException) {

+        if ($this->exception instanceof HttpException) {
             return $this->exception->getCode();

         }

 

@@ -226,9 +224,9 @@ 
         if (

             !empty($this->contentType)

             && array_key_exists($this->contentType, $this->renderers)

-        ) {

+        ) {
             $renderer = $this->renderers[$this->contentType];

-        } else {

+        } else {
             $renderer = $this->renderer;

         }

 

@@ -250,29 +248,29 @@ 
 

         $count = count($selected);

 

-        if ($count > 0) {

+        if ($count > 0) {
             $current = current($selected);

 

             /**

              * Ensure other supported content types take precedence over text/plain

              * when multiple content types are provided via Accept header.

              */

-            if ($current === 'text/plain' && $count > 1) {

+            if ($current === 'text/plain' && $count > 1) {
                 $next = next($selected);

-                if (is_string($next)) {

+                if (is_string($next)) {
                     return $next;

                 }

             }

 

-            if (is_string($current)) {

+            if (is_string($current)) {
                 return $current;

             }

         }

 

         $matches = [];

-        if (preg_match('/\+(json|xml)/', $header, $matches)) {

+        if (preg_match('/\+(json|xml)/', $header, $matches)) {
             $contentType = 'application/' . $matches[1];

-            if (array_key_exists($contentType, $this->renderers)) {

+            if (array_key_exists($contentType, $this->renderers)) {
                 return $contentType;

             }

         }

@@ -312,18 +310,18 @@ 
         if (

             !empty($this->contentType)

             && array_key_exists($this->contentType, $this->renderers)

-        ) {

+        ) {
             $response = $response->withHeader('Content-Type', $this->contentType);

         }

 

-        if ($this->exception instanceof HttpMethodNotAllowedException) {

+        if ($this->exception instanceof HttpMethodNotAllowedException) {
             $allowMethods = implode(', ', $this->exception->getAllowedMethods());

             $response = $response->withHeader('Allow', $allowMethods);

         }

 

-        if ($this->exception instanceof HttpException) {

+        if ($this->exception instanceof HttpException) {
             $headers = $this->exception->getHeaders();

-            foreach ($headers as $name => $value) {

+            foreach ($headers as $name => $value) {
                 $response = $response->withAddedHeader($name, $value);

             }

         }

src/Migration/MigrationEntity.php

Braces +1 added lines, -2 removed lines

@@ -56,8 +56,7 @@
  * @package Platine\Framework\Migration

  * @extends Entity<MigrationEntity>

  */

-class MigrationEntity extends Entity

-{

+class MigrationEntity extends Entity {
     /**

      *

      * @param EntityMapperInterface<MigrationEntity> $mapper

src/Migration/Seed/Command/SeedStatusCommand.php

Indentation +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
     ) {

         parent::__construct($app, $config, $filesystem);

         $this->setName('seed:status')

-             ->setDescription('Show your database seeds status');

+                ->setDescription('Show your database seeds status');

     }

 

     /**

Braces +4 added lines, -6 removed lines

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

  * @extends AbstractSeedCommand<T>

  */

-class SeedStatusCommand extends AbstractSeedCommand

-{

+class SeedStatusCommand extends AbstractSeedCommand {
     /**

      * Create new instance

      * {@inheritdoc}

@@ -67,7 +66,7 @@ 
         Application $app,

         Config $config,

         Filesystem $filesystem

-    ) {

+    ) {
         parent::__construct($app, $config, $filesystem);

         $this->setName('seed:status')

              ->setDescription('Show your database seeds status');

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

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

         $writer->boldYellow('SEED STATUS', true)->eol();

         $writer->bold('Seed path: ');

@@ -90,7 +88,7 @@ 
 

         $rows = [];

         $index = 1;

-        foreach ($seeds as $description) {

+        foreach ($seeds as $description) {
             $rows[] = [

                 'No.' => (string) $index,

                 'Seed' => $description

src/Migration/Seed/Command/AbstractSeedCommand.php

Braces +8 added lines, -9 removed lines

@@ -64,8 +64,7 @@ 
  * @package Platine\Framework\Migration\Seed\Command

  * @template T

  */

-abstract class AbstractSeedCommand extends Command

-{

+abstract class AbstractSeedCommand extends Command {
     /**

      * The configuration to use

      * @var Config<T>

@@ -100,7 +99,7 @@ 
         Application $app,

         Config $config,

         Filesystem $filesystem

-    ) {

+    ) {
         parent::__construct('seed', 'Command to manage database seed');

         $this->application = $app;

         $this->config = $config;

@@ -117,7 +116,7 @@ 
     {

         $directory = $this->filesystem->directory($this->seedPath);

 

-        if (!$directory->exists() || !$directory->isWritable()) {

+        if (!$directory->exists() || !$directory->isWritable()) {
             throw new RuntimeException(sprintf(

                 'Seed directory [%s] does not exist or is writable',

                 $this->seedPath

@@ -141,7 +140,7 @@ 
         $file = $this->filesystem->file($fullPath);

         $fullClassName = 'Platine\\Framework\\Migration\\Seed\\' . $className;

 

-        if (!$file->exists()) {

+        if (!$file->exists()) {
             throw new RuntimeException(sprintf(

                 'Seed file [%s] does not exist',

                 $fullPath

@@ -150,7 +149,7 @@ 
 

         require_once $fullPath;

 

-        if (!class_exists($fullClassName)) {

+        if (!class_exists($fullClassName)) {
             throw new RuntimeException(sprintf(

                 'Seed class [%s] does not exist',

                 $fullClassName

@@ -176,9 +175,9 @@ 
         $result = [];

         /** @var FileInterface[] $files */

         $files = $directory->read(DirectoryInterface::FILE);

-        foreach ($files as $file) {

+        foreach ($files as $file) {
             $matches = [];

-            if (preg_match('/^([a-z]+)Seed\.php$/i', $file->getName(), $matches)) {

+            if (preg_match('/^([a-z]+)Seed\.php$/i', $file->getName(), $matches)) {
                 $result[Str::camel($matches[1])] = str_replace('_', ' ', Str::snake($matches[1]));

             }

         }

@@ -196,7 +195,7 @@ 
     protected function getSeedClassName(string $description): string

     {

         $desc = Str::camel($description, false);

-        if (!Str::endsWith('Seed', $desc)) {

+        if (!Str::endsWith('Seed', $desc)) {
             $desc .= 'Seed';

         }

         return $desc;

src/Migration/Seed/Command/SeedCreateDbCommand.php

Indentation +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
         $this->queryBuilder = $queryBuilder;

 

         $this->setName('seed:createdb')

-             ->setDescription('Create a new seed using existing data');

+                ->setDescription('Create a new seed using existing data');

 

         $this->addArgument('table', 'name of the table', null, true, false);

         $this->addArgument('name', 'name of seed', null, false, true);

Braces +7 added lines, -9 removed lines

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

  * @extends AbstractSeedCommand<T>

  */

-class SeedCreateDbCommand extends AbstractSeedCommand

-{

+class SeedCreateDbCommand extends AbstractSeedCommand {
     /**

      * The seed name

      * @var string

@@ -99,7 +98,7 @@ 
         Filesystem $filesystem,

         Schema $schema,

         QueryBuilder $queryBuilder

-    ) {

+    ) {
         parent::__construct($app, $config, $filesystem);

         $this->schema = $schema;

         $this->queryBuilder = $queryBuilder;

@@ -114,8 +113,7 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

 

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

@@ -138,8 +136,8 @@ 
 

         $io = $this->io();

 

-        if ($io->confirm('Are you confirm the generation of new seed?', 'n')) {

-            if (!$this->schema->hasTable($this->table, true)) {

+        if ($io->confirm('Are you confirm the generation of new seed?', 'n')) {
+            if (!$this->schema->hasTable($this->table, true)) {
                 $writer->boldRed(sprintf(

                     'Database table [%s] does not exist',

                     $this->table

@@ -163,7 +161,7 @@ 
         $writer->boldYellow('SEED GENERATION USING EXISTING DATA', true)->eol();

 

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

-        if (!$name) {

+        if (!$name) {
             $io = $this->io();

             $name = $io->prompt('Enter the name of the seed', 'Seed description');

         }

@@ -225,7 +223,7 @@ 
                     ->fetchAssoc()

                     ->all();

 

-        if (empty($data)) {

+        if (empty($data)) {
             return sprintf('// No data found on table "%s"', $this->table);

         }

 

src/Migration/Seed/Command/SeedCreateCommand.php

Indentation +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
     ) {

         parent::__construct($app, $config, $filesystem);

         $this->setName('seed:create')

-             ->setDescription('Create a new seed');

+                ->setDescription('Create a new seed');

 

         $this->addArgument('name', 'name of seed', null, false, true);

     }

Braces +5 added lines, -7 removed lines

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

  * @extends AbstractSeedCommand<T>

  */

-class SeedCreateCommand extends AbstractSeedCommand

-{

+class SeedCreateCommand extends AbstractSeedCommand {
     /**

      * The seed name

      * @var string

@@ -77,7 +76,7 @@ 
         Application $app,

         Config $config,

         Filesystem $filesystem

-    ) {

+    ) {
         parent::__construct($app, $config, $filesystem);

         $this->setName('seed:create')

              ->setDescription('Create a new seed');

@@ -88,8 +87,7 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

 

         $className = $this->getSeedClassName($this->name);

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

         $io = $this->io();

 

-        if ($io->confirm('Are you confirm the generation of new seed?', 'n')) {

+        if ($io->confirm('Are you confirm the generation of new seed?', 'n')) {
             $this->checkSeedPath();

             $this->generateClass($fullPath, $className);

             $writer->boldGreen(sprintf(

@@ -127,7 +125,7 @@ 
         $writer->boldYellow('SEED GENERATION', true)->eol();

 

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

-        if (!$name) {

+        if (!$name) {
             $io = $this->io();

             $name = $io->prompt('Enter the name of the seed', 'Seed description');

         }

src/Migration/Seed/Command/SeedExecuteCommand.php

Indentation +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
     ) {

         parent::__construct($app, $config, $filesystem);

         $this->setName('seed:exec')

-             ->setDescription('Command to execute seed');

+                ->setDescription('Command to execute seed');

     }

 

     /**

Braces +7 added lines, -9 removed lines

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

  * @extends AbstractSeedCommand<T>

  */

-class SeedExecuteCommand extends AbstractSeedCommand

-{

+class SeedExecuteCommand extends AbstractSeedCommand {
     /**

      * Create new instance

      * {@inheritdoc}

@@ -67,7 +66,7 @@ 
         Application $app,

         Config $config,

         Filesystem $filesystem

-    ) {

+    ) {
         parent::__construct($app, $config, $filesystem);

         $this->setName('seed:exec')

              ->setDescription('Command to execute seed');

@@ -76,23 +75,22 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

         $writer = $io->writer();

         $writer->boldYellow('SEED EXECUTION', true)->eol();

 

         $seeds = array_values($this->getAvailableSeeds());

 

-        if (empty($seeds)) {

+        if (empty($seeds)) {
             $writer->boldGreen('No seed available for execution');

-        } else {

+        } else {
             $choices = [];

-            foreach ($seeds as $key => $seed) {

+            foreach ($seeds as $key => $seed) {
                 $choices[$key + 1] = $seed;

             }

             $indexes = $io->choices('Choose which seed to execute', $choices, []);

-            foreach ($indexes as $index) {

+            foreach ($indexes as $index) {
                 $this->executeSeed($seeds[$index - 1]);

             }

         }

src/Migration/Seed/AbstractSeed.php

Braces +1 added lines, -2 removed lines

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

  * @package Platine\Framework\Migration\Seed

  */

-abstract class AbstractSeed extends QueryBuilder

-{

+abstract class AbstractSeed extends QueryBuilder {
     /**

      * Run the seed

      * @return void

src/Migration/Command/MigrationMigrateCommand.php

Indentation +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
     ) {

         parent::__construct($app, $repository, $config, $filesystem);

         $this->setName('migration:migrate')

-             ->setDescription('Upgrade migration to latest');

+                ->setDescription('Upgrade migration to latest');

     }

 

     /**

Braces +7 added lines, -9 removed lines

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

  * @extends AbstractCommand<T>

  */

-class MigrationMigrateCommand extends AbstractCommand

-{

+class MigrationMigrateCommand extends AbstractCommand {
     /**

      * Create new instance

      * {@inheritdoc}

@@ -69,7 +68,7 @@ 
         MigrationRepository $repository,

         Config $config,

         Filesystem $filesystem

-    ) {

+    ) {
         parent::__construct($app, $repository, $config, $filesystem);

         $this->setName('migration:migrate')

              ->setDescription('Upgrade migration to latest');

@@ -78,8 +77,7 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

         $writer = $io->writer();

         $writer->boldYellow('MIGRATION UPGRADE TO LATEST', true)->eol();

@@ -88,23 +86,23 @@ 
         $executed = $this->getExecuted();

         $diff = array_diff_key($migrations, $executed);

 

-        if (empty($diff)) {

+        if (empty($diff)) {
             $writer->boldGreen('Migration already up to date');

             return;

         }

 

         $writer->bold('Migration list to be upgraded:', true);

-        foreach ($diff as $version => $value) {

+        foreach ($diff as $version => $value) {
             $cleanDescription = str_replace('_', ' ', $value);

             $writer->boldGreen(sprintf(' * %s - %s', $version, $cleanDescription), true);

         }

 

         $writer->write('', true);

 

-        if ($io->confirm('Are you confirm the migration upgrade to latest?', 'n')) {

+        if ($io->confirm('Are you confirm the migration upgrade to latest?', 'n')) {
             /** @var MigrationExecuteCommand<T> $migrationExecute */

             $migrationExecute = $this->application->get(MigrationExecuteCommand::class);

-            foreach ($diff as $version => $description) {

+            foreach ($diff as $version => $description) {
                 $cleanDescription = str_replace('_', ' ', $description);

                 $version = (string) $version;

                 $migrationExecute->executeMigrationUp($version, $cleanDescription);