platine-php / framework

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);

src/Migration/Command/AbstractCommand.php

Indentation +4 added lines, -4 removed lines

@@ -219,9 +219,9 @@ 
     {

         /** @var MigrationEntity[] $migrations */

         $migrations = $this->repository

-                           ->query()

-                           ->orderBy('version', $orderDir)

-                           ->all();

+                            ->query()

+                            ->orderBy('version', $orderDir)

+                            ->all();

         $result = [];

 

         foreach ($migrations as $entity) {

@@ -243,7 +243,7 @@ 
     protected function getMigrationClassName(string $description, string $version): string

     {

         return Str::camel($description, false)

-               . Str::replaceFirst('_', '', $version);

+                . Str::replaceFirst('_', '', $version);

     }

 

     /**

Braces +8 added lines, -9 removed lines

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

  * @template T

  */

-abstract class AbstractCommand extends Command

-{

+abstract class AbstractCommand extends Command {
     /**

      * The migration repository

      * @var MigrationRepository

@@ -116,7 +115,7 @@ 
         MigrationRepository $repository,

         Config $config,

         Filesystem $filesystem

-    ) {

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

         $this->application = $app;

         $this->repository = $repository;

@@ -135,7 +134,7 @@ 
     {

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

 

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

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

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

                 $this->migrationPath

@@ -162,7 +161,7 @@ 
         $file = $this->filesystem->file($fullPath);

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

 

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

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

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

                 $fullPath

@@ -171,7 +170,7 @@ 
 

         require_once $fullPath;

 

-        if (!class_exists($fullClassName)) {

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

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

                 $fullClassName

@@ -198,9 +197,9 @@ 
         $result = [];

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

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

-        foreach ($files as $file) {

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

-            if (preg_match('/^([0-9_]+)_([a-z0-9_]+)\.php$/i', $file->getName(), $matches)) {

+            if (preg_match('/^([0-9_]+)_([a-z0-9_]+)\.php$/i', $file->getName(), $matches)) {
                 $result[$matches[1]] = $matches[2];

             }

         }

@@ -224,7 +223,7 @@ 
                            ->all();

         $result = [];

 

-        foreach ($migrations as $entity) {

+        foreach ($migrations as $entity) {
             /** @var string $version */

             $version = $entity->version;

 

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);

         }

     }