spiral / framework

src/Command/Migrate/MigrateCommand.php

Spacing +3 added lines, -3 removed lines

@@ -23,14 +23,14 @@ 
      */
     public function perform()
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment()){
             return;
         }
 
         $found = false;
         $count = $this->option('one') ? 1 : PHP_INT_MAX;
 
-        while ($count > 0 && ($migration = $this->migrator->run())) {
+        while ($count > 0 && ($migration = $this->migrator->run())){
             $found = true;
             $count--;
 
@@ -40,7 +40,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln("<fg=red>No outstanding migrations were found.</fg=red>");
         }
     }

Braces +6 added lines, -3 removed lines

@@ -23,14 +23,16 @@ 
      */
     public function perform()
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment())
+        {
             return;
         }
 
         $found = false;
         $count = $this->option('one') ? 1 : PHP_INT_MAX;
 
-        while ($count > 0 && ($migration = $this->migrator->run())) {
+        while ($count > 0 && ($migration = $this->migrator->run()))
+        {
             $found = true;
             $count--;
 
@@ -40,7 +42,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln("<fg=red>No outstanding migrations were found.</fg=red>");
         }
     }

src/Command/Migrate/AbstractCommand.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
      */
     protected function verifyConfigured(): bool
     {
-        if (!$this->migrator->isConfigured()) {
+        if (!$this->migrator->isConfigured()){
             $this->writeln(
                 "<fg=red>Migrations are not configured yet, run '<info>migrate:init</info>' first.</fg=red>"
             );
@@ -58,14 +58,14 @@ 
      */
     protected function verifyEnvironment(): bool
     {
-        if ($this->option('force') || $this->config->isSafe()) {
+        if ($this->option('force') || $this->config->isSafe()){
             //Safe to run
             return true;
         }
 
         $this->writeln("<fg=red>Confirmation is required to run migrations!</fg=red>");
 
-        if (!$this->askConfirmation()) {
+        if (!$this->askConfirmation()){
             $this->writeln("<comment>Cancelling operation...</comment>");
 
             return false;

Braces +6 added lines, -3 removed lines

@@ -40,7 +40,8 @@ 
      */
     protected function verifyConfigured(): bool
     {
-        if (!$this->migrator->isConfigured()) {
+        if (!$this->migrator->isConfigured())
+        {
             $this->writeln(
                 "<fg=red>Migrations are not configured yet, run '<info>migrate:init</info>' first.</fg=red>"
             );
@@ -58,14 +59,16 @@ 
      */
     protected function verifyEnvironment(): bool
     {
-        if ($this->option('force') || $this->config->isSafe()) {
+        if ($this->option('force') || $this->config->isSafe())
+        {
             //Safe to run
             return true;
         }
 
         $this->writeln("<fg=red>Confirmation is required to run migrations!</fg=red>");
 
-        if (!$this->askConfirmation()) {
+        if (!$this->askConfirmation())
+        {
             $this->writeln("<comment>Cancelling operation...</comment>");
 
             return false;

src/Command/Migrate/StatusCommand.php

Spacing +4 added lines, -4 removed lines

@@ -27,18 +27,18 @@ 
      */
     public function perform(FilesInterface $files)
     {
-        if (!$this->verifyConfigured()) {
+        if (!$this->verifyConfigured()){
             return;
         }
 
-        if (empty($this->migrator->getMigrations())) {
+        if (empty($this->migrator->getMigrations())){
             $this->writeln("<comment>No migrations were found.</comment>");
 
             return;
         }
 
         $table = $this->table(['Migration', 'Filename', 'Created at', 'Executed at']);
-        foreach ($this->migrator->getMigrations() as $migration) {
+        foreach ($this->migrator->getMigrations() as $migration){
             $filename = (new \ReflectionClass($migration))->getFileName();
 
             $state = $migration->getState();
@@ -51,7 +51,7 @@ 
                 $state->getTimeCreated()->format('Y-m-d H:i:s'),
                 $state->getStatus() == State::STATUS_PENDING
                     ? self::PENDING
-                    : '<info>' . $state->getTimeExecuted()->format('Y-m-d H:i:s') . '</info>'
+                    : '<info>'.$state->getTimeExecuted()->format('Y-m-d H:i:s').'</info>'
             ]);
         }
 

Braces +6 added lines, -3 removed lines

@@ -27,18 +27,21 @@
      */
     public function perform(FilesInterface $files)
     {
-        if (!$this->verifyConfigured()) {
+        if (!$this->verifyConfigured())
+        {
             return;
         }
 
-        if (empty($this->migrator->getMigrations())) {
+        if (empty($this->migrator->getMigrations()))
+        {
             $this->writeln("<comment>No migrations were found.</comment>");
 
             return;
         }
 
         $table = $this->table(['Migration', 'Filename', 'Created at', 'Executed at']);
-        foreach ($this->migrator->getMigrations() as $migration) {
+        foreach ($this->migrator->getMigrations() as $migration)
+        {
             $filename = (new \ReflectionClass($migration))->getFileName();
 
             $state = $migration->getState();

src/Command/Migrate/RollbackCommand.php

Spacing +3 added lines, -3 removed lines

@@ -23,14 +23,14 @@ 
      */
     public function perform()
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment()){
             //Making sure we can safely migrate in this environment
             return;
         }
 
         $found = false;
         $count = !$this->option('all') ? 1 : PHP_INT_MAX;
-        while ($count > 0 && ($migration = $this->migrator->rollback())) {
+        while ($count > 0 && ($migration = $this->migrator->rollback())){
             $found = true;
             $count--;
             $this->sprintf(
@@ -39,7 +39,7 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found){
             $this->writeln("<fg=red>No executed migrations were found.</fg=red>");
         }
     }

Braces +6 added lines, -3 removed lines

@@ -23,14 +23,16 @@ 
      */
     public function perform()
     {
-        if (!$this->verifyConfigured() || !$this->verifyEnvironment()) {
+        if (!$this->verifyConfigured() || !$this->verifyEnvironment())
+        {
             //Making sure we can safely migrate in this environment
             return;
         }
 
         $found = false;
         $count = !$this->option('all') ? 1 : PHP_INT_MAX;
-        while ($count > 0 && ($migration = $this->migrator->rollback())) {
+        while ($count > 0 && ($migration = $this->migrator->rollback()))
+        {
             $found = true;
             $count--;
             $this->sprintf(
@@ -39,7 +41,8 @@ 
             );
         }
 
-        if (!$found) {
+        if (!$found)
+        {
             $this->writeln("<fg=red>No executed migrations were found.</fg=red>");
         }
     }

src/Command/Framework/CleanCommand.php

Spacing +6 added lines, -6 removed lines

@@ -25,20 +25,20 @@ 
     public function perform(FilesInterface $files, DirectoriesInterface $directories)
     {
         $cacheDirectory = $directories->get('cache');
-        if (!$files->exists($cacheDirectory)) {
+        if (!$files->exists($cacheDirectory)){
             $this->writeln("Cache directory is missing, no cache to be cleaned.");
 
             return;
         }
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose()){
             $this->writeln("<info>Cleaning application cache:</info>");
         }
 
-        foreach ($files->getFiles($cacheDirectory) as $filename) {
-            try {
+        foreach ($files->getFiles($cacheDirectory) as $filename){
+            try{
                 $files->delete($filename);
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $cacheDirectory),
                     $e->getMessage()
@@ -47,7 +47,7 @@ 
                 continue;
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose()){
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $cacheDirectory)

Braces +13 added lines, -6 removed lines

@@ -25,20 +25,26 @@ 
     public function perform(FilesInterface $files, DirectoriesInterface $directories)
     {
         $cacheDirectory = $directories->get('cache');
-        if (!$files->exists($cacheDirectory)) {
+        if (!$files->exists($cacheDirectory))
+        {
             $this->writeln("Cache directory is missing, no cache to be cleaned.");
 
             return;
         }
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose())
+        {
             $this->writeln("<info>Cleaning application cache:</info>");
         }
 
-        foreach ($files->getFiles($cacheDirectory) as $filename) {
-            try {
+        foreach ($files->getFiles($cacheDirectory) as $filename)
+        {
+            try
+            {
                 $files->delete($filename);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $cacheDirectory),
                     $e->getMessage()
@@ -47,7 +53,8 @@ 
                 continue;
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose())
+            {
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $cacheDirectory)

src/Command/Framework/ExtensionsCommand.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     public function perform()
     {
         $grid = $this->table(['Extension:', 'Version:']);
-        foreach (get_loaded_extensions() as $extension) {
+        foreach (get_loaded_extensions() as $extension){
             $grid->addRow([$extension, phpversion($extension)]);
         }
         $grid->render();

Braces +2 added lines, -1 removed lines

@@ -22,7 +22,8 @@
     public function perform()
     {
         $grid = $this->table(['Extension:', 'Version:']);
-        foreach (get_loaded_extensions() as $extension) {
+        foreach (get_loaded_extensions() as $extension)
+        {
             $grid->addRow([$extension, phpversion($extension)]);
         }
         $grid->render();

src/Command/Database/TableCommand.php

Spacing +12 added lines, -12 removed lines

@@ -41,7 +41,7 @@ 
         $database = $dbal->database($this->option('database'));
         $schema = $database->table($this->argument('table'))->getSchema();
 
-        if (!$schema->exists()) {
+        if (!$schema->exists()){
             throw new DBALException(
                 "Table {$database->getName()}.{$this->argument('table')} does not exists."
             );
@@ -55,11 +55,11 @@ 
 
         $this->describeColumns($schema);
 
-        if (!empty($indexes = $schema->getIndexes())) {
+        if (!empty($indexes = $schema->getIndexes())){
             $this->describeIndexes($database, $indexes);
         }
 
-        if (!empty($foreignKeys = $schema->getForeignKeys())) {
+        if (!empty($foreignKeys = $schema->getForeignKeys())){
             $this->describeForeignKeys($database, $foreignKeys);
         }
 
@@ -79,34 +79,34 @@ 
             'Default Value:'
         ]);
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column){
             $name = $column->getName();
             $type = $column->getType();
 
             $abstractType = $column->getAbstractType();
             $defaultValue = $column->getDefaultValue();
 
-            if ($column->getSize()) {
+            if ($column->getSize()){
                 $type .= " ({$column->getSize()})";
             }
 
-            if ($abstractType == 'decimal') {
+            if ($abstractType == 'decimal'){
                 $type .= " ({$column->getPrecision()}, {$column->getScale()})";
             }
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys())) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys())){
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
-            if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+            if (in_array($abstractType, ['primary', 'bigPrimary'])){
                 $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
             }
 
-            if ($defaultValue instanceof FragmentInterface) {
+            if ($defaultValue instanceof FragmentInterface){
                 $defaultValue = "<info>{$defaultValue}</info>";
             }
 
-            if ($defaultValue instanceof \DateTimeInterface) {
+            if ($defaultValue instanceof \DateTimeInterface){
                 $defaultValue = $defaultValue->format('c');
             }
 
@@ -135,7 +135,7 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index){
             $indexesTable->addRow([
                 $index->getName(),
                 $index->isUnique() ? 'UNIQUE INDEX' : 'INDEX',
@@ -166,7 +166,7 @@ 
             'On Update:'
         ]);
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference){
             $foreignTable->addRow([
                 $reference->getName(),
                 $reference->getColumn(),

Braces +24 added lines, -12 removed lines

@@ -41,7 +41,8 @@ 
         $database = $dbal->database($this->option('database'));
         $schema = $database->table($this->argument('table'))->getSchema();
 
-        if (!$schema->exists()) {
+        if (!$schema->exists())
+        {
             throw new DBALException(
                 "Table {$database->getName()}.{$this->argument('table')} does not exists."
             );
@@ -55,11 +56,13 @@ 
 
         $this->describeColumns($schema);
 
-        if (!empty($indexes = $schema->getIndexes())) {
+        if (!empty($indexes = $schema->getIndexes()))
+        {
             $this->describeIndexes($database, $indexes);
         }
 
-        if (!empty($foreignKeys = $schema->getForeignKeys())) {
+        if (!empty($foreignKeys = $schema->getForeignKeys()))
+        {
             $this->describeForeignKeys($database, $foreignKeys);
         }
 
@@ -79,34 +82,41 @@ 
             'Default Value:'
         ]);
 
-        foreach ($schema->getColumns() as $column) {
+        foreach ($schema->getColumns() as $column)
+        {
             $name = $column->getName();
             $type = $column->getType();
 
             $abstractType = $column->getAbstractType();
             $defaultValue = $column->getDefaultValue();
 
-            if ($column->getSize()) {
+            if ($column->getSize())
+            {
                 $type .= " ({$column->getSize()})";
             }
 
-            if ($abstractType == 'decimal') {
+            if ($abstractType == 'decimal')
+            {
                 $type .= " ({$column->getPrecision()}, {$column->getScale()})";
             }
 
-            if (in_array($column->getName(), $schema->getPrimaryKeys())) {
+            if (in_array($column->getName(), $schema->getPrimaryKeys()))
+            {
                 $name = "<fg=magenta>{$name}</fg=magenta>";
             }
 
-            if (in_array($abstractType, ['primary', 'bigPrimary'])) {
+            if (in_array($abstractType, ['primary', 'bigPrimary']))
+            {
                 $abstractType = "<fg=magenta>{$abstractType}</fg=magenta>";
             }
 
-            if ($defaultValue instanceof FragmentInterface) {
+            if ($defaultValue instanceof FragmentInterface)
+            {
                 $defaultValue = "<info>{$defaultValue}</info>";
             }
 
-            if ($defaultValue instanceof \DateTimeInterface) {
+            if ($defaultValue instanceof \DateTimeInterface)
+            {
                 $defaultValue = $defaultValue->format('c');
             }
 
@@ -135,7 +145,8 @@ 
         );
 
         $indexesTable = $this->table(['Name:', 'Type:', 'Columns:']);
-        foreach ($indexes as $index) {
+        foreach ($indexes as $index)
+        {
             $indexesTable->addRow([
                 $index->getName(),
                 $index->isUnique() ? 'UNIQUE INDEX' : 'INDEX',
@@ -166,7 +177,8 @@ 
             'On Update:'
         ]);
 
-        foreach ($foreignKeys as $reference) {
+        foreach ($foreignKeys as $reference)
+        {
             $foreignTable->addRow([
                 $reference->getName(),
                 $reference->getColumn(),

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 
         if (!$schema->exists()) {
             throw new DBALException(
-                "Table {$database->getName()}.{$this->argument('table')} does not exists."
+                "table {$database->getName()}.{$this->argument('table')} does not exists."
             );
         }
 

src/Command/Database/ListCommand.php

Spacing +9 added lines, -9 removed lines

@@ -29,13 +29,13 @@ 
      */
     public function perform(DatabaseConfig $config, DatabaseManager $dbal)
     {
-        if ($this->argument('db')) {
+        if ($this->argument('db')){
             $databases = [$this->argument('db')];
-        } else {
+        }else{
             $databases = array_keys($config->getDatabases());
         }
 
-        if (empty($databases)) {
+        if (empty($databases)){
             $this->writeln("<fg=red>No databases found.</fg=red>");
 
             return;
@@ -51,7 +51,7 @@ 
             'Count Records:'
         ]);
 
-        foreach ($databases as $database) {
+        foreach ($databases as $database){
             $database = $dbal->database($database);
 
             /** @var AbstractDriver $driver */
@@ -64,10 +64,10 @@ 
                 $database->getPrefix() ?: '<comment>---</comment>'
             ];
 
-            try {
+            try{
 
                 $driver->connect();
-            } catch (\Exception $exception) {
+            }catch (\Exception $exception){
                 $grid->addRow(array_merge(
                     $header,
                     [
@@ -77,7 +77,7 @@ 
                     ]
                 ));
 
-                if ($database->getName() != end($databases)) {
+                if ($database->getName() != end($databases)){
                     $grid->addRow(new TableSeparator());
                 }
 
@@ -85,7 +85,7 @@ 
             }
 
             $header[] = "<info>connected</info>";
-            foreach ($database->getTables() as $table) {
+            foreach ($database->getTables() as $table){
                 $grid->addRow(array_merge(
                     $header,
                     [$table->getName(), number_format($table->count())]
@@ -94,7 +94,7 @@ 
             }
 
             $header[1] && $grid->addRow(array_merge($header, ["no tables", "no records"]));
-            if ($database->getName() != end($databases)) {
+            if ($database->getName() != end($databases)){
                 $grid->addRow(new TableSeparator());
             }
         }

Braces +20 added lines, -9 removed lines

@@ -29,13 +29,17 @@ 
      */
     public function perform(DatabaseConfig $config, DatabaseManager $dbal)
     {
-        if ($this->argument('db')) {
+        if ($this->argument('db'))
+        {
             $databases = [$this->argument('db')];
-        } else {
+        }
+        else
+        {
             $databases = array_keys($config->getDatabases());
         }
 
-        if (empty($databases)) {
+        if (empty($databases))
+        {
             $this->writeln("<fg=red>No databases found.</fg=red>");
 
             return;
@@ -51,7 +55,8 @@ 
             'Count Records:'
         ]);
 
-        foreach ($databases as $database) {
+        foreach ($databases as $database)
+        {
             $database = $dbal->database($database);
 
             /** @var AbstractDriver $driver */
@@ -64,10 +69,13 @@ 
                 $database->getPrefix() ?: '<comment>---</comment>'
             ];
 
-            try {
+            try
+            {
 
                 $driver->connect();
-            } catch (\Exception $exception) {
+            }
+            catch (\Exception $exception)
+            {
                 $grid->addRow(array_merge(
                     $header,
                     [
@@ -77,7 +85,8 @@ 
                     ]
                 ));
 
-                if ($database->getName() != end($databases)) {
+                if ($database->getName() != end($databases))
+                {
                     $grid->addRow(new TableSeparator());
                 }
 
@@ -85,7 +94,8 @@ 
             }
 
             $header[] = "<info>connected</info>";
-            foreach ($database->getTables() as $table) {
+            foreach ($database->getTables() as $table)
+            {
                 $grid->addRow(array_merge(
                     $header,
                     [$table->getName(), number_format($table->count())]
@@ -94,7 +104,8 @@ 
             }
 
             $header[1] && $grid->addRow(array_merge($header, ["no tables", "no records"]));
-            if ($database->getName() != end($databases)) {
+            if ($database->getName() != end($databases))
+            {
                 $grid->addRow(new TableSeparator());
             }
         }

src/Command/Translator/ExportCommand.php

Spacing +8 added lines, -8 removed lines

@@ -37,7 +37,7 @@ 
      */
     public function perform(TranslatorConfig $config, CatalogueManager $manager)
     {
-        if (!$config->hasDumper($this->option('dumper'))) {
+        if (!$config->hasDumper($this->option('dumper'))){
             $this->writeln("<fg=red>Undefined dumper '{$this->option('dumper')}'.</fg=red>");
 
             return;
@@ -50,17 +50,17 @@ 
             $catalogue->getData()
         );
 
-        if ($this->option('fallback')) {
-            foreach ($manager->get($config->fallbackLocale())->getData() as $domain => $messages) {
-                foreach ($messages as $id => $message) {
-                    if (!$messageCatalogue->defines($id, $domain)) {
+        if ($this->option('fallback')){
+            foreach ($manager->get($config->fallbackLocale())->getData() as $domain => $messages){
+                foreach ($messages as $id => $message){
+                    if (!$messageCatalogue->defines($id, $domain)){
                         $messageCatalogue->set($id, $message, $domain);
                     }
                 }
             }
         }
 
-        if ($this->isVerbose() && !empty($messageCatalogue->getDomains())) {
+        if ($this->isVerbose() && !empty($messageCatalogue->getDomains())){
             $this->sprintf("<info>Exporting domain(s):</info> %s\n",
                 join(',', $messageCatalogue->getDomains())
             );
@@ -74,7 +74,7 @@ 
             'xliff_version'  => '2.0' // forcing default version for xliff dumper only
         ]);
 
-        $this->writeln("Export successfully completed using <info>" . get_class($dumper) . "</info>");
-        $this->writeln("Output: <comment>" . realpath($this->argument('path')) . "</comment>");
+        $this->writeln("Export successfully completed using <info>".get_class($dumper)."</info>");
+        $this->writeln("Output: <comment>".realpath($this->argument('path'))."</comment>");
     }
 }
\ No newline at end of file

Braces +12 added lines, -6 removed lines

@@ -37,7 +37,8 @@ 
      */
     public function perform(TranslatorConfig $config, CatalogueManager $manager)
     {
-        if (!$config->hasDumper($this->option('dumper'))) {
+        if (!$config->hasDumper($this->option('dumper')))
+        {
             $this->writeln("<fg=red>Undefined dumper '{$this->option('dumper')}'.</fg=red>");
 
             return;
@@ -50,17 +51,22 @@ 
             $catalogue->getData()
         );
 
-        if ($this->option('fallback')) {
-            foreach ($manager->get($config->fallbackLocale())->getData() as $domain => $messages) {
-                foreach ($messages as $id => $message) {
-                    if (!$messageCatalogue->defines($id, $domain)) {
+        if ($this->option('fallback'))
+        {
+            foreach ($manager->get($config->fallbackLocale())->getData() as $domain => $messages)
+            {
+                foreach ($messages as $id => $message)
+                {
+                    if (!$messageCatalogue->defines($id, $domain))
+                    {
                         $messageCatalogue->set($id, $message, $domain);
                     }
                 }
             }
         }
 
-        if ($this->isVerbose() && !empty($messageCatalogue->getDomains())) {
+        if ($this->isVerbose() && !empty($messageCatalogue->getDomains()))
+        {
             $this->sprintf("<info>Exporting domain(s):</info> %s\n",
                 join(',', $messageCatalogue->getDomains())
             );