spiral / framework

tests/Framework/GRPC/GenerateTest.php

Braces +6 added lines, -3 removed lines

@@ -32,7 +32,8 @@ 
     public function setUp()
     {
         exec('protoc 2>&1', $out);
-        if (strpos(join("\n", $out), '--php_out') === false) {
+        if (strpos(join("\n", $out), '--php_out') === false)
+        {
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -51,11 +52,13 @@ 
 
         $fs = new Files();
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/Service')) {
+        if ($fs->isDirectory($this->app->dir('app') . 'src/Service'))
+        {
             $fs->deleteDirectory($this->app->dir('app') . 'src/Service');
         }
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata')) {
+        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata'))
+        {
             $fs->deleteDirectory($this->app->dir('app') . 'src/GPBMetadata');
         }
     }

Spacing +9 added lines, -9 removed lines

@@ -33,14 +33,14 @@ 
     public function setUp()
     {
         exec('protoc 2>&1', $out);
-        if (strpos(join("\n", $out), '--php_out') === false) {
+        if (strpos(join("\n", $out), '--php_out') === false){
             $this->markTestSkipped('Protoc binary is missing');
         }
 
         parent::setUp();
 
         $fs = new Files();
-        $this->proto = $fs->normalizePath($this->app->dir('app') . 'proto/service.proto');
+        $this->proto = $fs->normalizePath($this->app->dir('app').'proto/service.proto');
 
         // protoc can't figure relative paths
         $this->proto = str_replace('Framework/../', '', $this->proto);
@@ -52,12 +52,12 @@ 
 
         $fs = new Files();
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/Service')) {
-            $fs->deleteDirectory($this->app->dir('app') . 'src/Service');
+        if ($fs->isDirectory($this->app->dir('app').'src/Service')){
+            $fs->deleteDirectory($this->app->dir('app').'src/Service');
         }
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata')) {
-            $fs->deleteDirectory($this->app->dir('app') . 'src/GPBMetadata');
+        if ($fs->isDirectory($this->app->dir('app').'src/GPBMetadata')){
+            $fs->deleteDirectory($this->app->dir('app').'src/GPBMetadata');
         }
     }
 
@@ -85,8 +85,8 @@ 
             'proto' => $this->proto
         ]);
 
-        $this->assertFileExists($this->app->dir('app') . 'src/Service/EchoInterface.php');
-        $this->assertFileExists($this->app->dir('app') . 'src/Service/Sub/Message.php');
-        $this->assertFileExists($this->app->dir('app') . 'src/GPBMetadata/Service.php');
+        $this->assertFileExists($this->app->dir('app').'src/Service/EchoInterface.php');
+        $this->assertFileExists($this->app->dir('app').'src/Service/Sub/Message.php');
+        $this->assertFileExists($this->app->dir('app').'src/GPBMetadata/Service.php');
     }
 }

src/GRPC/ProtoCompiler.php

Spacing +6 added lines, -6 removed lines

@@ -58,14 +58,14 @@ 
 
         $output = trim(join("\n", $output), "\n ,");
 
-        if ($output !== '') {
+        if ($output !== ''){
             $this->files->deleteDirectory($tmpDir);
             throw new CompileException($output);
         }
 
         // copying files (using relative path and namespace)
         $result = [];
-        foreach ($this->files->getFiles($tmpDir) as $file) {
+        foreach ($this->files->getFiles($tmpDir) as $file){
             $result[] = $this->copy($tmpDir, $file);
         }
 
@@ -82,11 +82,11 @@ 
     private function copy(string $tmpDir, string $file): string
     {
         $source = ltrim($this->files->relativePath($file, $tmpDir), '\\/');
-        if (strpos($source, $this->baseNamespace) === 0) {
+        if (strpos($source, $this->baseNamespace) === 0){
             $source = ltrim(substr($source, strlen($this->baseNamespace)), '\\/');
         }
 
-        $target = $this->files->normalizePath($this->basePath . '/' . $source);
+        $target = $this->files->normalizePath($this->basePath.'/'.$source);
 
         $this->files->ensureDirectory(dirname($target));
         $this->files->copy($file, $target);
@@ -99,7 +99,7 @@ 
      */
     private function tmpDir(): string
     {
-        $directory = sys_get_temp_dir() . '/' . spl_object_hash($this);
+        $directory = sys_get_temp_dir().'/'.spl_object_hash($this);
         $this->files->ensureDirectory($directory);
 
         return $this->files->normalizePath($directory, true);
@@ -113,7 +113,7 @@ 
      */
     private function getProtoFiles(string $protoFile): array
     {
-        return array_filter($this->files->getFiles(dirname($protoFile)), function ($file) {
+        return array_filter($this->files->getFiles(dirname($protoFile)), function ($file){
             return strpos($file, '.proto') !== false;
         });
     }

Braces +8 added lines, -4 removed lines

@@ -58,14 +58,16 @@ 
 
         $output = trim(join("\n", $output), "\n ,");
 
-        if ($output !== '') {
+        if ($output !== '')
+        {
             $this->files->deleteDirectory($tmpDir);
             throw new CompileException($output);
         }
 
         // copying files (using relative path and namespace)
         $result = [];
-        foreach ($this->files->getFiles($tmpDir) as $file) {
+        foreach ($this->files->getFiles($tmpDir) as $file)
+        {
             $result[] = $this->copy($tmpDir, $file);
         }
 
@@ -82,7 +84,8 @@ 
     private function copy(string $tmpDir, string $file): string
     {
         $source = ltrim($this->files->relativePath($file, $tmpDir), '\\/');
-        if (strpos($source, $this->baseNamespace) === 0) {
+        if (strpos($source, $this->baseNamespace) === 0)
+        {
             $source = ltrim(substr($source, strlen($this->baseNamespace)), '\\/');
         }
 
@@ -113,7 +116,8 @@ 
      */
     private function getProtoFiles(string $protoFile): array
     {
-        return array_filter($this->files->getFiles(dirname($protoFile)), function ($file) {
+        return array_filter($this->files->getFiles(dirname($protoFile)), function ($file)
+        {
             return strpos($file, '.proto') !== false;
         });
     }

functions.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 
 use Spiral\Debug\Dumper;
 
-if (!function_exists('bind')) {
+if (!function_exists('bind')){
     /**
      * Shortcut to container Autowire definition.
      *
@@ -27,7 +27,7 @@ 
     }
 }
 
-if (!function_exists('dumprr')) {
+if (!function_exists('dumprr')){
     /**
      * Dumprr is similar to Dump function but always redirect output to STDERR.
      *

Braces +4 added lines, -2 removed lines

@@ -9,7 +9,8 @@ 
 
 use Spiral\Debug\Dumper;
 
-if (!function_exists('bind')) {
+if (!function_exists('bind'))
+{
     /**
      * Shortcut to container Autowire definition.
      *
@@ -27,7 +28,8 @@ 
     }
 }
 
-if (!function_exists('dumprr')) {
+if (!function_exists('dumprr'))
+{
     /**
      * Dumprr is similar to Dump function but always redirect output to STDERR.
      *

src/Command/Cycle/SyncCommand.php

Braces +2 added lines, -1 removed lines

@@ -41,7 +41,8 @@
 
         $memory->saveData('cycle', $schema);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges())
+        {
             $this->writeln("\n<info>ORM Schema has been synchronized</info>");
         }
 

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
         CycleBootloader $cycleBootloader,
         Registry $registry,
         MemoryInterface $memory
-    ) {
+    ){
         $show = new ShowChanges($this->output);
 
         $schema = (new Compiler())->compile(
@@ -47,7 +47,7 @@ 
 
         $memory->saveData('cycle', $schema);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges()){
             $this->writeln("\n<info>ORM Schema has been synchronized</info>");
         }
 

src/Http/Diactoros/UploadedFileFactory.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
         string $clientFilename = null,
         string $clientMediaType = null
     ): UploadedFileInterface {
-        if ($size === null) {
+        if ($size === null){
             $size = $stream->getSize();
         }
 

Braces +2 added lines, -1 removed lines

@@ -26,7 +26,8 @@
         string $clientFilename = null,
         string $clientMediaType = null
     ): UploadedFileInterface {
-        if ($size === null) {
+        if ($size === null)
+        {
             $size = $stream->getSize();
         }
 

src/Http/Diactoros/ResponseFactory.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $response = new Response('php://memory', $code, []);
         $response = $response->withStatus($code, $reasonPhrase);
 
-        foreach ($this->config->getBaseHeaders() as $header => $value) {
+        foreach ($this->config->getBaseHeaders() as $header => $value){
             $response = $response->withAddedHeader($header, $value);
         }
 

Braces +2 added lines, -1 removed lines

@@ -37,7 +37,8 @@
         $response = new Response('php://memory', $code, []);
         $response = $response->withStatus($code, $reasonPhrase);
 
-        foreach ($this->config->getBaseHeaders() as $header => $value) {
+        foreach ($this->config->getBaseHeaders() as $header => $value)
+        {
             $response = $response->withAddedHeader($header, $value);
         }
 

src/Bootloader/Http/HttpBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 
         $kernel->addDispatcher($factory->make(SapiDispatcher::class));
 
-        if (class_exists(PSR7Client::class)) {
+        if (class_exists(PSR7Client::class)){
             $kernel->addDispatcher($factory->make(RrDispacher::class));
         }
     }

Braces +2 added lines, -1 removed lines

@@ -66,7 +66,8 @@
 
         $kernel->addDispatcher($factory->make(SapiDispatcher::class));
 
-        if (class_exists(PSR7Client::class)) {
+        if (class_exists(PSR7Client::class))
+        {
             $kernel->addDispatcher($factory->make(RrDispacher::class));
         }
     }

src/Console/ConsoleDispatcher.php

Spacing +8 added lines, -8 removed lines

@@ -44,7 +44,7 @@ 
         EnvironmentInterface $env,
         FinalizerInterface $finalizer,
         ContainerInterface $container
-    ) {
+    ){
         $this->env = $env;
         $this->finalizer = $finalizer;
         $this->container = $container;
@@ -73,11 +73,11 @@ 
         /** @var Console $console */
         $console = $this->container->get(Console::class);
 
-        try {
+        try{
             $console->start($input ?? new ArgvInput(), $output);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $this->handleException($e, $output);
-        } finally {
+        }finally{
             $listener->disable();
             $this->finalizer->finalize(false);
         }
@@ -89,9 +89,9 @@ 
      */
     protected function handleException(\Throwable $e, OutputInterface $output)
     {
-        try {
+        try{
             $this->container->get(SnapshotterInterface::class)->register($e);
-        } catch (\Throwable|ContainerExceptionInterface $se) {
+        }catch (\Throwable | ContainerExceptionInterface $se){
             // no need to notify when unable to register an exception
         }
 
@@ -106,11 +106,11 @@ 
      */
     private function mapVerbosity(OutputInterface $output): int
     {
-        if ($output->isDebug()) {
+        if ($output->isDebug()){
             return ConsoleHandler::VERBOSITY_DEBUG;
         }
 
-        if ($output->isVeryVerbose()) {
+        if ($output->isVeryVerbose()){
             return ConsoleHandler::VERBOSITY_VERBOSE;
         }
 

Braces +17 added lines, -7 removed lines

@@ -73,11 +73,16 @@ 
         /** @var Console $console */
         $console = $this->container->get(Console::class);
 
-        try {
+        try
+        {
             $console->start($input ?? new ArgvInput(), $output);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $this->handleException($e, $output);
-        } finally {
+        }
+        finally
+        {
             $listener->disable();
             $this->finalizer->finalize(false);
         }
@@ -89,9 +94,12 @@ 
      */
     protected function handleException(\Throwable $e, OutputInterface $output)
     {
-        try {
+        try
+        {
             $this->container->get(SnapshotterInterface::class)->register($e);
-        } catch (\Throwable|ContainerExceptionInterface $se) {
+        }
+        catch (\Throwable|ContainerExceptionInterface $se)
+        {
             // no need to notify when unable to register an exception
         }
 
@@ -106,11 +114,13 @@ 
      */
     private function mapVerbosity(OutputInterface $output): int
     {
-        if ($output->isDebug()) {
+        if ($output->isDebug())
+        {
             return ConsoleHandler::VERBOSITY_DEBUG;
         }
 
-        if ($output->isVeryVerbose()) {
+        if ($output->isVeryVerbose())
+        {
             return ConsoleHandler::VERBOSITY_VERBOSE;
         }
 

src/Command/Cycle/Generator/ShowChanges.php

Spacing +18 added lines, -18 removed lines

@@ -40,11 +40,11 @@ 
         $this->output->writeln("<info>Detecting schema changes:</info>");
 
         $this->changes = [];
-        foreach ($registry->getIterator() as $e) {
-            if ($registry->hasTable($e)) {
+        foreach ($registry->getIterator() as $e){
+            if ($registry->hasTable($e)){
                 $table = $registry->getTableSchema($e);
 
-                if ($table->getComparator()->hasChanges()) {
+                if ($table->getComparator()->hasChanges()){
                     $this->changes[] = [
                         'database' => $registry->getDatabase($e),
                         'table'    => $registry->getTable($e),
@@ -54,13 +54,13 @@ 
             }
         }
 
-        if ($this->changes === []) {
+        if ($this->changes === []){
             $this->output->writeln("<fg=yellow>no database changes has been detected</fg=yellow>");
 
             return $registry;
         }
 
-        foreach ($this->changes as $change) {
+        foreach ($this->changes as $change){
             $this->output->write(sprintf("• <fg=cyan>%s.%s</fg=cyan>", $change['database'], $change['table']));
             $this->describeChanges($change['schema']);
         }
@@ -73,22 +73,22 @@ 
      */
     protected function describeChanges(AbstractTable $table)
     {
-        if (!$this->output->isVerbose()) {
+        if (!$this->output->isVerbose()){
             $this->output->writeln(sprintf(
                 ": <fg=green>%s</fg=green> change(s) detected",
                 $this->numChanges($table)
             ));
 
             return;
-        } else {
+        }else{
             $this->output->write("\n");
         }
 
-        if (!$table->exists()) {
+        if (!$table->exists()){
             $this->output->writeln("    - create table");
         }
 
-        if ($table->getStatus() === AbstractTable::STATUS_DECLARED_DROPPED) {
+        if ($table->getStatus() === AbstractTable::STATUS_DECLARED_DROPPED){
             $this->output->writeln("    - drop table");
             return;
         }
@@ -105,15 +105,15 @@ 
      */
     protected function describeColumns(Comparator $cmp)
     {
-        foreach ($cmp->addedColumns() as $column) {
+        foreach ($cmp->addedColumns() as $column){
             $this->output->writeln("    - add column <fg=yellow>{$column->getName()}</fg=yellow>");
         }
 
-        foreach ($cmp->droppedColumns() as $column) {
+        foreach ($cmp->droppedColumns() as $column){
             $this->output->writeln("    - drop column <fg=yellow>{$column->getName()}</fg=yellow>");
         }
 
-        foreach ($cmp->alteredColumns() as $column) {
+        foreach ($cmp->alteredColumns() as $column){
             $column = $column[0];
             $this->output->writeln("    - alter column <fg=yellow>{$column->getName()}</fg=yellow>");
         }
@@ -124,17 +124,17 @@ 
      */
     protected function describeIndexes(Comparator $cmp)
     {
-        foreach ($cmp->addedIndexes() as $index) {
+        foreach ($cmp->addedIndexes() as $index){
             $index = join(', ', $index->getColumns());
             $this->output->writeln("    - add index on <fg=yellow>[{$index}]</fg=yellow>");
         }
 
-        foreach ($cmp->droppedIndexes() as $index) {
+        foreach ($cmp->droppedIndexes() as $index){
             $index = join(', ', $index->getColumns());
             $this->output->writeln("    - drop index on <fg=yellow>[{$index}]</fg=yellow>");
         }
 
-        foreach ($cmp->alteredIndexes() as $index) {
+        foreach ($cmp->alteredIndexes() as $index){
             $index = $index[0];
             $index = join(', ', $index->getColumns());
             $this->output->writeln("    - alter index on <fg=yellow>[{$index}]</fg=yellow>");
@@ -146,17 +146,17 @@ 
      */
     protected function describeFKs(Comparator $cmp)
     {
-        foreach ($cmp->addedForeignKeys() as $fk) {
+        foreach ($cmp->addedForeignKeys() as $fk){
             $fkColumns = join(', ', $fk->getColumns());
             $this->output->writeln("    - add foreign key on <fg=yellow>{$fkColumns}</fg=yellow>");
         }
 
-        foreach ($cmp->droppedForeignKeys() as $fk) {
+        foreach ($cmp->droppedForeignKeys() as $fk){
             $fkColumns = join(', ', $fk->getColumns());
             $this->output->writeln("    - drop foreign key <fg=yellow>{$fkColumns}</fg=yellow>");
         }
 
-        foreach ($cmp->alteredForeignKeys() as $fk) {
+        foreach ($cmp->alteredForeignKeys() as $fk){
             $fk = $fk[0];
             $fkColumns = join(', ', $fk->getColumns());
             $this->output->writeln("    - alter foreign key <fg=yellow>{$fkColumns}</fg=yellow>");

Braces +37 added lines, -18 removed lines

@@ -40,11 +40,14 @@ 
         $this->output->writeln("<info>Detecting schema changes:</info>");
 
         $this->changes = [];
-        foreach ($registry->getIterator() as $e) {
-            if ($registry->hasTable($e)) {
+        foreach ($registry->getIterator() as $e)
+        {
+            if ($registry->hasTable($e))
+            {
                 $table = $registry->getTableSchema($e);
 
-                if ($table->getComparator()->hasChanges()) {
+                if ($table->getComparator()->hasChanges())
+                {
                     $this->changes[] = [
                         'database' => $registry->getDatabase($e),
                         'table'    => $registry->getTable($e),
@@ -54,13 +57,15 @@ 
             }
         }
 
-        if ($this->changes === []) {
+        if ($this->changes === [])
+        {
             $this->output->writeln("<fg=yellow>no database changes has been detected</fg=yellow>");
 
             return $registry;
         }
 
-        foreach ($this->changes as $change) {
+        foreach ($this->changes as $change)
+        {
             $this->output->write(sprintf("• <fg=cyan>%s.%s</fg=cyan>", $change['database'], $change['table']));
             $this->describeChanges($change['schema']);
         }
@@ -73,22 +78,27 @@ 
      */
     protected function describeChanges(AbstractTable $table)
     {
-        if (!$this->output->isVerbose()) {
+        if (!$this->output->isVerbose())
+        {
             $this->output->writeln(sprintf(
                 ": <fg=green>%s</fg=green> change(s) detected",
                 $this->numChanges($table)
             ));
 
             return;
-        } else {
+        }
+        else
+        {
             $this->output->write("\n");
         }
 
-        if (!$table->exists()) {
+        if (!$table->exists())
+        {
             $this->output->writeln("    - create table");
         }
 
-        if ($table->getStatus() === AbstractTable::STATUS_DECLARED_DROPPED) {
+        if ($table->getStatus() === AbstractTable::STATUS_DECLARED_DROPPED)
+        {
             $this->output->writeln("    - drop table");
             return;
         }
@@ -105,15 +115,18 @@ 
      */
     protected function describeColumns(Comparator $cmp)
     {
-        foreach ($cmp->addedColumns() as $column) {
+        foreach ($cmp->addedColumns() as $column)
+        {
             $this->output->writeln("    - add column <fg=yellow>{$column->getName()}</fg=yellow>");
         }
 
-        foreach ($cmp->droppedColumns() as $column) {
+        foreach ($cmp->droppedColumns() as $column)
+        {
             $this->output->writeln("    - drop column <fg=yellow>{$column->getName()}</fg=yellow>");
         }
 
-        foreach ($cmp->alteredColumns() as $column) {
+        foreach ($cmp->alteredColumns() as $column)
+        {
             $column = $column[0];
             $this->output->writeln("    - alter column <fg=yellow>{$column->getName()}</fg=yellow>");
         }
@@ -124,17 +137,20 @@ 
      */
     protected function describeIndexes(Comparator $cmp)
     {
-        foreach ($cmp->addedIndexes() as $index) {
+        foreach ($cmp->addedIndexes() as $index)
+        {
             $index = join(', ', $index->getColumns());
             $this->output->writeln("    - add index on <fg=yellow>[{$index}]</fg=yellow>");
         }
 
-        foreach ($cmp->droppedIndexes() as $index) {
+        foreach ($cmp->droppedIndexes() as $index)
+        {
             $index = join(', ', $index->getColumns());
             $this->output->writeln("    - drop index on <fg=yellow>[{$index}]</fg=yellow>");
         }
 
-        foreach ($cmp->alteredIndexes() as $index) {
+        foreach ($cmp->alteredIndexes() as $index)
+        {
             $index = $index[0];
             $index = join(', ', $index->getColumns());
             $this->output->writeln("    - alter index on <fg=yellow>[{$index}]</fg=yellow>");
@@ -146,17 +162,20 @@ 
      */
     protected function describeFKs(Comparator $cmp)
     {
-        foreach ($cmp->addedForeignKeys() as $fk) {
+        foreach ($cmp->addedForeignKeys() as $fk)
+        {
             $fkColumns = join(', ', $fk->getColumns());
             $this->output->writeln("    - add foreign key on <fg=yellow>{$fkColumns}</fg=yellow>");
         }
 
-        foreach ($cmp->droppedForeignKeys() as $fk) {
+        foreach ($cmp->droppedForeignKeys() as $fk)
+        {
             $fkColumns = join(', ', $fk->getColumns());
             $this->output->writeln("    - drop foreign key <fg=yellow>{$fkColumns}</fg=yellow>");
         }
 
-        foreach ($cmp->alteredForeignKeys() as $fk) {
+        foreach ($cmp->alteredForeignKeys() as $fk)
+        {
             $fk = $fk[0];
             $fkColumns = join(', ', $fk->getColumns());
             $this->output->writeln("    - alter foreign key <fg=yellow>{$fkColumns}</fg=yellow>");