spiral / framework

src/Cycle/SelectInjector.php

Spacing +2 added lines, -2 removed lines

@@ -35,11 +35,11 @@
      */
     public function createInjection(\ReflectionClass $class, string $context = null)
     {
-        if (!$this->orm->getSchema()->defines($context)) {
+        if (!$this->orm->getSchema()->defines($context)){
             $context = Inflector::singularize($context);
         }
 
-        if (!$this->orm->getSchema()->defines($context)) {
+        if (!$this->orm->getSchema()->defines($context)){
             throw new ORMException("Cycle schema is not initiated or role missing, run `cycle`");
         }
 

Braces +4 added lines, -2 removed lines

@@ -35,11 +35,13 @@
      */
     public function createInjection(\ReflectionClass $class, string $context = null)
     {
-        if (!$this->orm->getSchema()->defines($context)) {
+        if (!$this->orm->getSchema()->defines($context))
+        {
             $context = Inflector::singularize($context);
         }
 
-        if (!$this->orm->getSchema()->defines($context)) {
+        if (!$this->orm->getSchema()->defines($context))
+        {
             throw new ORMException("Cycle schema is not initiated or role missing, run `cycle`");
         }
 

src/Command/CleanCommand.php

Spacing +6 added lines, -6 removed lines

@@ -26,20 +26,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){
                 // @codeCoverageIgnoreStart
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $cacheDirectory),
@@ -50,7 +50,7 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            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

@@ -26,20 +26,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)
+            {
                 // @codeCoverageIgnoreStart
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $cacheDirectory),
@@ -50,7 +56,8 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose())
+            {
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $cacheDirectory)

src/Command/GRPC/ListCommand.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
     {
         $services = $locator->getServices();
 
-        if ($services === []) {
+        if ($services === []){
             $this->writeln("<comment>No GRPC services were found.</comment>");
             return;
         }
@@ -35,7 +35,7 @@ 
             'File:',
         ]);
 
-        foreach ($services as $interface => $instance) {
+        foreach ($services as $interface => $instance){
             $grid->addRow([
                 $interface::NAME,
                 get_class($instance),

Braces +4 added lines, -2 removed lines

@@ -24,7 +24,8 @@ 
     {
         $services = $locator->getServices();
 
-        if ($services === []) {
+        if ($services === [])
+        {
             $this->writeln("<comment>No GRPC services were found.</comment>");
             return;
         }
@@ -35,7 +36,8 @@ 
             'File:',
         ]);
 
-        foreach ($services as $interface => $instance) {
+        foreach ($services as $interface => $instance)
+        {
             $grid->addRow([
                 $interface::NAME,
                 get_class($instance),

src/Command/GRPC/GenerateCommand.php

Spacing +7 added lines, -7 removed lines

@@ -39,7 +39,7 @@ 
     public function perform(KernelInterface $kernel, FilesInterface $files, DirectoriesInterface $dirs)
     {
         $protoFile = $this->argument('proto');
-        if (!file_exists($protoFile)) {
+        if (!file_exists($protoFile)){
             $this->sprintf("<error>File `%s` not found.</error>", $protoFile);
             return;
         }
@@ -52,19 +52,19 @@ 
 
         $this->sprintf("<info>Compiling <fg=cyan>`%s`</fg=cyan>:</info>\n", $protoFile);
 
-        try {
+        try{
             $result = $compiler->compile($protoFile);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $this->sprintf("<error>Error:</error> <fg=red>%s</fg=red>\n", $e->getMessage());
             return;
         }
 
-        if ($result === []) {
+        if ($result === []){
             $this->sprintf("<info>No files were generated.</info>\n", $protoFile);
             return;
         }
 
-        foreach ($result as $file) {
+        foreach ($result as $file){
             $this->sprintf(
                 "<fg=green>•</fg=green> %s%s%s\n",
                 Color::LIGHT_WHITE,
@@ -83,7 +83,7 @@ 
     protected function getPath(KernelInterface $kernel): string
     {
         $path = $this->argument('namespace');
-        if ($path !== 'auto') {
+        if ($path !== 'auto'){
             return $path;
         }
 
@@ -101,7 +101,7 @@ 
     protected function getNamespace(KernelInterface $kernel): string
     {
         $namespace = $this->argument('namespace');
-        if ($namespace !== 'auto') {
+        if ($namespace !== 'auto'){
             return $namespace;
         }
 

Braces +15 added lines, -7 removed lines

@@ -39,7 +39,8 @@ 
     public function perform(KernelInterface $kernel, FilesInterface $files, DirectoriesInterface $dirs)
     {
         $protoFile = $this->argument('proto');
-        if (!file_exists($protoFile)) {
+        if (!file_exists($protoFile))
+        {
             $this->sprintf("<error>File `%s` not found.</error>", $protoFile);
             return;
         }
@@ -52,19 +53,24 @@ 
 
         $this->sprintf("<info>Compiling <fg=cyan>`%s`</fg=cyan>:</info>\n", $protoFile);
 
-        try {
+        try
+        {
             $result = $compiler->compile($protoFile);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $this->sprintf("<error>Error:</error> <fg=red>%s</fg=red>\n", $e->getMessage());
             return;
         }
 
-        if ($result === []) {
+        if ($result === [])
+        {
             $this->sprintf("<info>No files were generated.</info>\n", $protoFile);
             return;
         }
 
-        foreach ($result as $file) {
+        foreach ($result as $file)
+        {
             $this->sprintf(
                 "<fg=green>•</fg=green> %s%s%s\n",
                 Color::LIGHT_WHITE,
@@ -83,7 +89,8 @@ 
     protected function getPath(KernelInterface $kernel): string
     {
         $path = $this->argument('namespace');
-        if ($path !== 'auto') {
+        if ($path !== 'auto')
+        {
             return $path;
         }
 
@@ -101,7 +108,8 @@ 
     protected function getNamespace(KernelInterface $kernel): string
     {
         $namespace = $this->argument('namespace');
-        if ($namespace !== 'auto') {
+        if ($namespace !== 'auto')
+        {
             return $namespace;
         }
 

src/Command/Cycle/MigrateCommand.php

Spacing +6 added lines, -6 removed lines

@@ -48,13 +48,13 @@ 
         GenerateMigrations $migrations,
         Migrator $migrator,
         Console $console
-    ) {
-        if (!$this->verifyConfigured()) {
+    ){
+        if (!$this->verifyConfigured()){
             return;
         }
 
-        foreach ($migrator->getMigrations() as $migration) {
-            if ($migration->getState()->getStatus() !== State::STATUS_EXECUTED) {
+        foreach ($migrator->getMigrations() as $migration){
+            if ($migration->getState()->getStatus() !== State::STATUS_EXECUTED){
                 $this->writeln("<fg=red>Outstanding migrations found, run `migrate` first.</fg=red>");
                 return;
             }
@@ -69,10 +69,10 @@ 
 
         $memory->saveData('cycle', $schema);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges()){
             (new Compiler())->compile($registry, [$migrations]);
 
-            if ($this->option('run')) {
+            if ($this->option('run')){
                 $console->run('migrate', [], $this->output);
             }
         }

Braces +10 added lines, -5 removed lines

@@ -49,12 +49,15 @@ 
         Migrator $migrator,
         Console $console
     ) {
-        if (!$this->verifyConfigured()) {
+        if (!$this->verifyConfigured())
+        {
             return;
         }
 
-        foreach ($migrator->getMigrations() as $migration) {
-            if ($migration->getState()->getStatus() !== State::STATUS_EXECUTED) {
+        foreach ($migrator->getMigrations() as $migration)
+        {
+            if ($migration->getState()->getStatus() !== State::STATUS_EXECUTED)
+            {
                 $this->writeln("<fg=red>Outstanding migrations found, run `migrate` first.</fg=red>");
                 return;
             }
@@ -69,10 +72,12 @@ 
 
         $memory->saveData('cycle', $schema);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges())
+        {
             (new Compiler())->compile($registry, [$migrations]);
 
-            if ($this->option('run')) {
+            if ($this->option('run'))
+            {
                 $console->run('migrate', [], $this->output);
             }
         }

src/Command/Views/ResetCommand.php

Spacing +6 added lines, -6 removed lines

@@ -27,20 +27,20 @@ 
      */
     public function perform(ViewsConfig $config, FilesInterface $files)
     {
-        if (!$files->exists($config->cacheDirectory())) {
+        if (!$files->exists($config->cacheDirectory())){
             $this->writeln("Cache directory is missing, no cache to be cleaned.");
 
             return;
         }
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose()){
             $this->writeln("<info>Cleaning view cache:</info>");
         }
 
-        foreach ($files->getFiles($config->cacheDirectory()) as $filename) {
-            try {
+        foreach ($files->getFiles($config->cacheDirectory()) as $filename){
+            try{
                 $files->delete($filename);
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 // @codeCoverageIgnoreStart
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $config->cacheDirectory()),
@@ -52,7 +52,7 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose()){
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $config->cacheDirectory())

Braces +13 added lines, -6 removed lines

@@ -27,20 +27,26 @@ 
      */
     public function perform(ViewsConfig $config, FilesInterface $files)
     {
-        if (!$files->exists($config->cacheDirectory())) {
+        if (!$files->exists($config->cacheDirectory()))
+        {
             $this->writeln("Cache directory is missing, no cache to be cleaned.");
 
             return;
         }
 
-        if ($this->isVerbose()) {
+        if ($this->isVerbose())
+        {
             $this->writeln("<info>Cleaning view cache:</info>");
         }
 
-        foreach ($files->getFiles($config->cacheDirectory()) as $filename) {
-            try {
+        foreach ($files->getFiles($config->cacheDirectory()) as $filename)
+        {
+            try
+            {
                 $files->delete($filename);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 // @codeCoverageIgnoreStart
                 $this->sprintf("<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>\n",
                     $files->relativePath($filename, $config->cacheDirectory()),
@@ -52,7 +58,8 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose())
+            {
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $config->cacheDirectory())

src/Command/Views/CompileCommand.php

Spacing +13 added lines, -13 removed lines

@@ -32,12 +32,12 @@ 
         $generator = new ContextGenerator($views->getContext());
 
         $contexts = $generator->generate();
-        if (empty($contexts)) {
+        if (empty($contexts)){
             $contexts[] = $views->getContext();
         }
 
-        foreach ($contexts as $context) {
-            foreach ($views->getEngines() as $engine) {
+        foreach ($contexts as $context){
+            foreach ($views->getEngines() as $engine){
                 $this->compile($engine, $context);
             }
         }
@@ -57,19 +57,19 @@ 
             $this->describeContext($context) ?? "default"
         );
 
-        foreach ($engine->getLoader()->list() as $path) {
+        foreach ($engine->getLoader()->list() as $path){
             $start = microtime(true);
-            try {
+            try{
                 $engine->reset($path, $context);
                 $engine->compile($path, $context);
 
-                if ($this->isVerbose()) {
+                if ($this->isVerbose()){
                     $this->sprintf("<info>•</info> %s", $path);
                 }
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 $this->renderError($path, $e);
                 continue;
-            } finally {
+            }finally{
                 $this->renderElapsed($start);
             }
         }
@@ -85,7 +85,7 @@ 
     {
         $values = [];
 
-        foreach ($context->getDependencies() as $dependency) {
+        foreach ($context->getDependencies() as $dependency){
             $values[] = sprintf(
                 "%s%s%s:%s%s%s",
                 Color::LIGHT_WHITE,
@@ -117,7 +117,7 @@ 
      */
     protected function renderError(string $path, \Throwable $e): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 
@@ -134,11 +134,11 @@ 
      */
     private function renderSuccess(string $lastPath = null): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 
-        if ($lastPath === null) {
+        if ($lastPath === null){
             $this->writeln("• no views found");
         }
 
@@ -150,7 +150,7 @@ 
      */
     private function renderElapsed(float $start): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 

Braces +28 added lines, -13 removed lines

@@ -32,12 +32,15 @@ 
         $generator = new ContextGenerator($views->getContext());
 
         $contexts = $generator->generate();
-        if (empty($contexts)) {
+        if (empty($contexts))
+        {
             $contexts[] = $views->getContext();
         }
 
-        foreach ($contexts as $context) {
-            foreach ($views->getEngines() as $engine) {
+        foreach ($contexts as $context)
+        {
+            foreach ($views->getEngines() as $engine)
+            {
                 $this->compile($engine, $context);
             }
         }
@@ -57,19 +60,26 @@ 
             $this->describeContext($context) ?? "default"
         );
 
-        foreach ($engine->getLoader()->list() as $path) {
+        foreach ($engine->getLoader()->list() as $path)
+        {
             $start = microtime(true);
-            try {
+            try
+            {
                 $engine->reset($path, $context);
                 $engine->compile($path, $context);
 
-                if ($this->isVerbose()) {
+                if ($this->isVerbose())
+                {
                     $this->sprintf("<info>•</info> %s", $path);
                 }
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 $this->renderError($path, $e);
                 continue;
-            } finally {
+            }
+            finally
+            {
                 $this->renderElapsed($start);
             }
         }
@@ -85,7 +95,8 @@ 
     {
         $values = [];
 
-        foreach ($context->getDependencies() as $dependency) {
+        foreach ($context->getDependencies() as $dependency)
+        {
             $values[] = sprintf(
                 "%s%s%s:%s%s%s",
                 Color::LIGHT_WHITE,
@@ -117,7 +128,8 @@ 
      */
     protected function renderError(string $path, \Throwable $e): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }
 
@@ -134,11 +146,13 @@ 
      */
     private function renderSuccess(string $lastPath = null): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }
 
-        if ($lastPath === null) {
+        if ($lastPath === null)
+        {
             $this->writeln("• no views found");
         }
 
@@ -150,7 +164,8 @@ 
      */
     private function renderElapsed(float $start): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }
 

tests/Framework/Http/SapiTest.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
         $e = new BufferEmitter();
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -48,7 +48,7 @@ 
         $this->app->get(SapiDispatcher::class)->serve($e);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);
@@ -65,7 +65,7 @@ 
         ]);
 
         $files = $app->get(FilesInterface::class)->getFiles(
-            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -74,7 +74,7 @@ 
         $app->get(SapiDispatcher::class)->serve($e);
 
         $files = $app->get(FilesInterface::class)->getFiles(
-            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);

tests/Framework/Http/RrTest.php

Spacing +11 added lines, -11 removed lines

@@ -21,7 +21,7 @@ 
     public function setUp()
     {
         parent::setUp();
-        ini_set('error_log', __DIR__ . '/err.log');
+        ini_set('error_log', __DIR__.'/err.log');
     }
 
     public function tearDown()
@@ -29,8 +29,8 @@ 
         parent::tearDown();
         ini_set('error_log', 'stderr');
 
-        if (file_exists(__DIR__ . '/err.log')) {
-            unlink(__DIR__ . '/err.log');
+        if (file_exists(__DIR__.'/err.log')){
+            unlink(__DIR__.'/err.log');
         }
     }
 
@@ -56,7 +56,7 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r){
                 $this->assertSame('Hello, dave.', (string)$r->getBody());
                 return true;
             })
@@ -78,7 +78,7 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r){
                 $this->assertContains('500', (string)$r->getBody());
                 return true;
             })
@@ -87,7 +87,7 @@ 
         $psr->shouldReceive('acceptRequest')->once()->andReturn(null);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -95,7 +95,7 @@ 
         $this->app->get(RrDispacher::class)->serve($psr);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);
@@ -116,7 +116,7 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r){
                 $this->assertContains('undefined', (string)$r->getBody());
                 return true;
             })
@@ -125,7 +125,7 @@ 
         $psr->shouldReceive('acceptRequest')->once()->andReturn(null);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -133,11 +133,11 @@ 
         $this->app->get(RrDispacher::class)->serve($psr);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);
 
-        $this->assertContains('undefined', file_get_contents(__DIR__ . '/err.log'));
+        $this->assertContains('undefined', file_get_contents(__DIR__.'/err.log'));
     }
 }
\ No newline at end of file

Braces +8 added lines, -4 removed lines

@@ -29,7 +29,8 @@ 
         parent::tearDown();
         ini_set('error_log', 'stderr');
 
-        if (file_exists(__DIR__ . '/err.log')) {
+        if (file_exists(__DIR__ . '/err.log'))
+        {
             unlink(__DIR__ . '/err.log');
         }
     }
@@ -56,7 +57,8 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r)
+            {
                 $this->assertSame('Hello, dave.', (string)$r->getBody());
                 return true;
             })
@@ -78,7 +80,8 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r)
+            {
                 $this->assertContains('500', (string)$r->getBody());
                 return true;
             })
@@ -116,7 +119,8 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r)
+            {
                 $this->assertContains('undefined', (string)$r->getBody());
                 return true;
             })