platine-php / framework

src/Migration/Command/MigrationCreateCommand.php

Indentation +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     ) {

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

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

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

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

 

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

     }

Braces +5 added lines, -7 removed lines

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

  * @extends AbstractCommand<T>

  */

-class MigrationCreateCommand extends AbstractCommand

-{

+class MigrationCreateCommand extends AbstractCommand {
     /**

      * The migration name

      * @var string

@@ -78,7 +77,7 @@ 
         MigrationRepository $repository,

         Config $config,

         Filesystem $filesystem

-    ) {

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

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

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

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

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

 

         $version = date('Ymd_His');

@@ -113,7 +111,7 @@ 
 

         $io = $this->io();

 

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

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

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

             $writer->boldGreen(sprintf(

@@ -131,7 +129,7 @@ 
         $writer->boldYellow('MIGRATION GENERATION', true)->eol();

 

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

-        if (!$name) {

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

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

         }

src/Migration/Command/MigrationResetCommand.php

Indentation +1 added lines, -1 removed lines

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

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

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

-             ->setDescription('Rollback all migration done before');

+                ->setDescription('Rollback all migration done before');

     }

 

     /**

Braces +7 added lines, -9 removed lines

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

  * @extends AbstractCommand<T>

  */

-class MigrationResetCommand extends AbstractCommand

-{

+class MigrationResetCommand 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:reset')

              ->setDescription('Rollback all migration done before');

@@ -78,21 +77,20 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

         $writer = $io->writer();

         $writer->boldYellow('ALL MIGRATION ROLLBACK', true)->eol();

 

         $executed = $this->getExecuted('DESC');

 

-        if (empty($executed)) {

+        if (empty($executed)) {
             $writer->boldGreen('No migration done before');

             return;

         }

 

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

-        foreach ($executed as $version => $entity) {

+        foreach ($executed as $version => $entity) {
             $writer->boldGreen(

                 sprintf(' * %s - %s', $version, $entity->description),

                 true

@@ -101,10 +99,10 @@ 
 

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

 

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

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

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

-            foreach ($executed as $version => $entity) {

+            foreach ($executed as $version => $entity) {
                 $migrationExecute->executeMigrationDown($version, $entity->description);

             }

 

src/Migration/Command/MigrationStatusCommand.php

Indentation +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
     ) {

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

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

-             ->setDescription('Show current status of your migrations');

+                ->setDescription('Show current status of your migrations');

     }

 

     /**

Braces +5 added lines, -7 removed lines

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

  * @extends AbstractCommand<T>

  */

-class MigrationStatusCommand extends AbstractCommand

-{

+class MigrationStatusCommand extends AbstractCommand {
     /**

      * Create new instance

      * {@inheritdoc}

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

         Config $config,

         Filesystem $filesystem

-    ) {

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

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

              ->setDescription('Show current status of your migrations');

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

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

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

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

@@ -99,7 +97,7 @@ 
         $writer->boldYellow('MIGRATION LIST', true);

 

         $rows = [];

-        foreach ($executed as $version => $entity) {

+        foreach ($executed as $version => $entity) {
             $rows[] = [

                 'version' => (string) $version,

                 'description' => $entity->description,

@@ -108,7 +106,7 @@ 
             ];

         }

 

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

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

             $version = (string) $version;

             $rows[] = [

src/Migration/Command/MigrationInitCommand.php

Indentation +4 added lines, -4 removed lines

@@ -81,7 +81,7 @@ 
     ) {

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

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

-             ->setDescription('Initialize the migration by creating migration table');

+                ->setDescription('Initialize the migration by creating migration table');

 

         $this->schema = $schema;

     }

@@ -119,10 +119,10 @@ 
         $tableName = $this->table;

         $this->schema->create($tableName, function (CreateTable $table) {

             $table->string('version', 20)

-                   ->description('The migration version')

-                   ->primary();

+                    ->description('The migration version')

+                    ->primary();

             $table->string('description')

-                   ->description('The migration description');

+                    ->description('The migration description');

             $table->datetime('created_at')

                     ->description('Migration run time');

 

Spacing +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
     protected function createMigrationTable(): void

     {

         $tableName = $this->table;

-        $this->schema->create($tableName, function (CreateTable $table) {

+        $this->schema->create($tableName, function(CreateTable $table) {

             $table->string('version', 20)

                    ->description('The migration version')

                    ->primary();

Braces +5 added lines, -7 removed lines

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

  * @extends AbstractCommand<T>

  */

-class MigrationInitCommand extends AbstractCommand

-{

+class MigrationInitCommand extends AbstractCommand {
     /**

      * The schema to use

      * @var Schema

@@ -78,7 +77,7 @@ 
         Schema $schema,

         Config $config,

         Filesystem $filesystem

-    ) {

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

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

              ->setDescription('Initialize the migration by creating migration table');

@@ -89,13 +88,12 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

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

         $writer = $io->writer();

         $writer->boldYellow('MIGRATION INITIALIZATION', true);

 

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

+        if ($this->schema->hasTable($this->table, true)) {
             $writer->boldRed(sprintf(

                 'Migration table [%s] already created',

                 $this->table

@@ -117,7 +115,7 @@ 
     protected function createMigrationTable(): void

     {

         $tableName = $this->table;

-        $this->schema->create($tableName, function (CreateTable $table) {

+        $this->schema->create($tableName, function (CreateTable $table) {
             $table->string('version', 20)

                    ->description('The migration version')

                    ->primary();

src/Migration/Command/MigrationExecuteCommand.php

Indentation +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
     ) {

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

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

-             ->setDescription('Execute the migration up/down for one version');

+                ->setDescription('Execute the migration up/down for one version');

 

         $this->addArgument('type', 'type of migration [up|down]', 'up', true, true, false, function ($val) {

             if (!in_array($val, ['up', 'down'])) {

@@ -83,7 +83,7 @@ 
                 ));

             }

 

-             return $val;

+                return $val;

         });

 

         $this->addOption('-i|--id', 'the migration version', null, false, true);

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         $this->setName('migration:exec')

              ->setDescription('Execute the migration up/down for one version');

 

-        $this->addArgument('type', 'type of migration [up|down]', 'up', true, true, false, function ($val) {

+        $this->addArgument('type', 'type of migration [up|down]', 'up', true, true, false, function($val) {

             if (!in_array($val, ['up', 'down'])) {

                 throw new RuntimeException(sprintf(

                     'Invalid argument type [%s], must be one of [up, down]',

Braces +19 added lines, -21 removed lines

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

  * @extends AbstractCommand<T>

  */

-class MigrationExecuteCommand extends AbstractCommand

-{

+class MigrationExecuteCommand extends AbstractCommand {
     /**

      * Create new instance

      * {@inheritdoc}

@@ -70,13 +69,13 @@ 
         MigrationRepository $repository,

         Config $config,

         Filesystem $filesystem

-    ) {

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

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

              ->setDescription('Execute the migration up/down for one version');

 

-        $this->addArgument('type', 'type of migration [up|down]', 'up', true, true, false, function ($val) {

-            if (!in_array($val, ['up', 'down'])) {

+        $this->addArgument('type', 'type of migration [up|down]', 'up', true, true, false, function ($val) {
+            if (!in_array($val, ['up', 'down'])) {
                 throw new RuntimeException(sprintf(

                     'Invalid argument type [%s], must be one of [up, down]',

                     $val

@@ -92,8 +91,7 @@ 
     /**

      * {@inheritdoc}

      */

-    public function execute()

-    {

+    public function execute() {
         $type = $this->getArgumentValue('type');

 

         $io = $this->io();

@@ -104,43 +102,43 @@ 
         $executed = $this->getExecuted('DESC');

         $version = $this->getOptionValue('id');

 

-        if ($type === 'up') {

+        if ($type === 'up') {
             $diff = array_diff_key($migrations, $executed);

-            if (empty($diff)) {

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

-            } else {

-                if (empty($version)) {

+            } else {
+                if (empty($version)) {
                     $version = $io->choice('Choose which version to migrate up', $diff);

                 }

 

-                if (!isset($diff[$version])) {

+                if (!isset($diff[$version])) {
                     $writer->boldRed(sprintf(

                         'Invalid migration version [%s] or already executed',

                         $version

                     ));

-                } else {

+                } else {
                     $description = str_replace('_', ' ', $migrations[$version]);

                     $this->executeMigrationUp($version, $description);

                 }

             }

-        } else {

-            if (empty($executed)) {

+        } else {
+            if (empty($executed)) {
                 $writer->boldGreen('No migration to rollback');

-            } else {

+            } else {
                 $data = [];

-                foreach ($executed as $ver => $entity) {

+                foreach ($executed as $ver => $entity) {
                     $data[$ver] = $entity->description;

                 }

-                if (empty($version)) {

+                if (empty($version)) {
                     $version = $io->choice('Choose which version to rollback', $data);

                 }

 

-                if (!isset($data[$version])) {

+                if (!isset($data[$version])) {
                     $writer->boldRed(sprintf(

                         'Invalid migration version [%s] or not yet executed',

                         $version

                     ));

-                } else {

+                } else {
                     $description = str_replace('_', ' ', $data[$version]);

                     $this->executeMigrationDown($version, $description);

                 }

@@ -197,7 +195,7 @@ 
             'version' => $version

         ]);

 

-        if ($entity) {

+        if ($entity) {
             $this->repository->delete($entity);

         }

     }

src/Migration/AbstractMigration.php

Braces +1 added lines, -2 removed lines

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

  * @package Platine\Framework\Migration

  */

-abstract class AbstractMigration extends Schema

-{

+abstract class AbstractMigration extends Schema {
     /**

      * Execute when do migrate

      * @return void

src/Migration/MigrationRepository.php

Braces +2 added lines, -4 removed lines

@@ -55,14 +55,12 @@
  * @package Platine\Framework\Migration

  * @extends Repository<MigrationEntity>

  */

-class MigrationRepository extends Repository

-{

+class MigrationRepository extends Repository {
     /**

      * Create new instance

      * @param EntityManager<MigrationEntity> $manager

      */

-    public function __construct(EntityManager $manager)

-    {

+    public function __construct(EntityManager $manager) {
         parent::__construct($manager, MigrationEntity::class);

     }

 }

src/Security/JWT/Encoder/Base64UrlSafeEncoder.php

Braces +1 added lines, -2 removed lines

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

  * @package Platine\Framework\Security\JWT\Encoder

  */

-class Base64UrlSafeEncoder implements EncoderInterface

-{

+class Base64UrlSafeEncoder implements EncoderInterface {
     /**

      * {@inheritdoc}

      */

src/Security/JWT/EncoderInterface.php

Braces +1 added lines, -2 removed lines

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

  * @package Platine\Framework\Security\JWT

  */

-interface EncoderInterface

-{

+interface EncoderInterface {
     /**

      * Encode the given data

      * @param string $data