spiral / framework

src/Bootloader/SnapshotsBootloader.php

Spacing +2 added lines, -2 removed lines

@@ -41,9 +41,9 @@
         EnvironmentInterface $env,
         DirectoriesInterface $dirs,
         FilesInterface $files
-    ) {
+    ){
         return new FileSnapshotter(
-            $dirs->get('runtime') . '/snapshots/',
+            $dirs->get('runtime').'/snapshots/',
             $env->get('SNAPSHOT_MAX_FILES', self::MAX_SNAPSHOTS),
             $env->get('SNAPSHOT_VERBOSITY', HandlerInterface::VERBOSITY_VERBOSE),
             new HtmlHandler(),

src/Bootloader/Http/SessionBootloader.php

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
         CookiesBootloader $cookies,
         HttpBootloader $http,
         DirectoriesInterface $directories
-    ) {
+    ){
         $config->setDefaults('session', [
             'lifetime' => 86400,
             'cookie'   => 'sid',
@@ -58,7 +58,7 @@ 
             'handler'  => new Autowire(
                 FileHandler::class,
                 [
-                    'directory' => $directories->get('runtime') . 'session',
+                    'directory' => $directories->get('runtime').'session',
                     'lifetime'  => 86400
                 ]
             )

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",
@@ -51,7 +51,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",
@@ -51,7 +57,8 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose())
+            {
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $cacheDirectory)

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->getCacheDirectory())) {
+        if (!$files->exists($config->getCacheDirectory())){
             $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->getCacheDirectory()) as $filename) {
-            try {
+        foreach ($files->getFiles($config->getCacheDirectory()) 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",
@@ -53,7 +53,7 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose()){
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $config->getCacheDirectory())

Braces +13 added lines, -6 removed lines

@@ -27,20 +27,26 @@ 
      */
     public function perform(ViewsConfig $config, FilesInterface $files)
     {
-        if (!$files->exists($config->getCacheDirectory())) {
+        if (!$files->exists($config->getCacheDirectory()))
+        {
             $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->getCacheDirectory()) as $filename) {
-            try {
+        foreach ($files->getFiles($config->getCacheDirectory()) 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",
@@ -53,7 +59,8 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose())
+            {
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $config->getCacheDirectory())

src/Command/Translator/ExportCommand.php

Spacing +8 added lines, -8 removed lines

@@ -39,7 +39,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;
@@ -51,7 +51,7 @@ 
             $manager->get($this->argument('locale'))
         );
 
-        if ($this->isVerbose() && !empty($mc->getDomains())) {
+        if ($this->isVerbose() && !empty($mc->getDomains())){
             $this->sprintf(
                 "<info>Exporting domain(s):</info> %s\n",
                 join(',', $mc->getDomains())
@@ -66,8 +66,8 @@ 
             '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>");
     }
 
     /**
@@ -86,10 +86,10 @@ 
             $catalogue->getData()
         );
 
-        if ($this->option('fallback')) {
-            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages) {
-                foreach ($messages as $id => $message) {
-                    if (!$messageCatalogue->defines($id, $domain)) {
+        if ($this->option('fallback')){
+            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages){
+                foreach ($messages as $id => $message){
+                    if (!$messageCatalogue->defines($id, $domain)){
                         $messageCatalogue->set($id, $message, $domain);
                     }
                 }

Braces +12 added lines, -6 removed lines

@@ -39,7 +39,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;
@@ -51,7 +52,8 @@ 
             $manager->get($this->argument('locale'))
         );
 
-        if ($this->isVerbose() && !empty($mc->getDomains())) {
+        if ($this->isVerbose() && !empty($mc->getDomains()))
+        {
             $this->sprintf(
                 "<info>Exporting domain(s):</info> %s\n",
                 join(',', $mc->getDomains())
@@ -86,10 +88,14 @@ 
             $catalogue->getData()
         );
 
-        if ($this->option('fallback')) {
-            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages) {
-                foreach ($messages as $id => $message) {
-                    if (!$messageCatalogue->defines($id, $domain)) {
+        if ($this->option('fallback'))
+        {
+            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages)
+            {
+                foreach ($messages as $id => $message)
+                {
+                    if (!$messageCatalogue->defines($id, $domain))
+                    {
                         $messageCatalogue->set($id, $message, $domain);
                     }
                 }

tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -9,4 +9,4 @@
 ini_set('display_errors', true);
 
 //Composer
-require dirname(__DIR__) . '/vendor/autoload.php';
+require dirname(__DIR__).'/vendor/autoload.php';

tests/Framework/Encrypter/KeyCommandTest.php

Spacing +9 added lines, -9 removed lines

@@ -23,7 +23,7 @@ 
     public function testMountFileNotFound()
     {
         $out = $this->runCommand('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
 
         $this->assertContains('Unable to find', $out);
@@ -31,18 +31,18 @@ 
 
     public function testReplace()
     {
-        file_put_contents(__DIR__ . '/.env', '{encrypt-key}');
+        file_put_contents(__DIR__.'/.env', '{encrypt-key}');
 
         $out = $this->runCommand('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
 
         $this->assertContains('key has been updated', $out);
 
-        $body = file_get_contents(__DIR__ . '/.env');
+        $body = file_get_contents(__DIR__.'/.env');
         $this->assertContains($body, $out);
 
-        unlink(__DIR__ . '/.env');
+        unlink(__DIR__.'/.env');
     }
 
     public function testReplaceCurrent()
@@ -53,18 +53,18 @@ 
             'ENCRYPTER_KEY' => $key
         ]);
 
-        file_put_contents(__DIR__ . '/.env', $key);
+        file_put_contents(__DIR__.'/.env', $key);
 
         $out = $app->console()->run('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
         $out = $out->getOutput()->fetch();
 
         $this->assertContains('key has been updated', $out);
 
-        $body = file_get_contents(__DIR__ . '/.env');
+        $body = file_get_contents(__DIR__.'/.env');
         $this->assertContains($body, $out);
 
-        unlink(__DIR__ . '/.env');
+        unlink(__DIR__.'/.env');
     }
 }

tests/Framework/GRPC/DispatcherTest.php

Spacing +13 added lines, -13 removed lines

@@ -21,14 +21,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();
-        $proto = $fs->normalizePath($this->app->dir('app') . 'proto/service.proto');
+        $proto = $fs->normalizePath($this->app->dir('app').'proto/service.proto');
 
         // protoc can't figure relative paths
         $proto = str_replace('Framework/../', '', $proto);
@@ -37,7 +37,7 @@ 
             'proto' => $proto
         ]);
 
-        file_put_contents($this->app->dir('app') . 'src/Service/EchoService.php', GenerateTest::SERVICE);
+        file_put_contents($this->app->dir('app').'src/Service/EchoService.php', GenerateTest::SERVICE);
     }
 
     public function tearDown()
@@ -46,12 +46,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');
         }
     }
 
@@ -77,7 +77,7 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping"               
@@ -88,7 +88,7 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('send')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out){
                 $msg = new Message();
                 $msg->mergeFromString($out);
                 $this->assertSame("hello", $msg->getMsg());
@@ -99,7 +99,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })
@@ -119,7 +119,7 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid"               
@@ -130,7 +130,7 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out){
                 $this->assertContains('Method `Invalid` not found', $out);
                 return true;
             })
@@ -138,7 +138,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })

Braces +18 added lines, -9 removed lines

@@ -21,7 +21,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');
         }
 
@@ -46,11 +47,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');
         }
     }
@@ -77,7 +80,8 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping"               
@@ -88,7 +92,8 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('send')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out)
+            {
                 $msg = new Message();
                 $msg->mergeFromString($out);
                 $this->assertSame("hello", $msg->getMsg());
@@ -99,7 +104,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })
@@ -119,7 +125,8 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid"               
@@ -130,7 +137,8 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out)
+            {
                 $this->assertContains('Method `Invalid` not found', $out);
                 return true;
             })
@@ -138,7 +146,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })

tests/Framework/BaseTest.php

Spacing +4 added lines, -4 removed lines

@@ -17,10 +17,10 @@
     public function makeApp(array $env = []): TestApp
     {
         return TestApp::init([
-            'root'    => __DIR__ . '/../..',
-            'app'     => __DIR__ . '/../app',
-            'runtime' => sys_get_temp_dir() . '/spiral',
-            'cache'   => sys_get_temp_dir() . '/spiral',
+            'root'    => __DIR__.'/../..',
+            'app'     => __DIR__.'/../app',
+            'runtime' => sys_get_temp_dir().'/spiral',
+            'cache'   => sys_get_temp_dir().'/spiral',
         ], new Environment($env), false);
     }
 }