spiral / framework

src/Framework/helpers.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 use Spiral\Core\Container\Autowire;
 use Spiral\Router\RouterInterface;
 
-if (!\function_exists('bind')) {
+if (!\function_exists('bind')){
     /**
      * Shortcut to container Autowire definition.
      *
@@ -21,7 +21,7 @@ 
     }
 }
 
-if (!\function_exists('route')) {
+if (!\function_exists('route')){
     /**
      * Generate the URL to a named route.
      *

Braces +4 added lines, -2 removed lines

@@ -6,7 +6,8 @@ 
 use Spiral\Core\Container\Autowire;
 use Spiral\Router\RouterInterface;
 
-if (!\function_exists('bind')) {
+if (!\function_exists('bind'))
+{
     /**
      * Shortcut to container Autowire definition.
      *
@@ -21,7 +22,8 @@ 
     }
 }
 
-if (!\function_exists('route')) {
+if (!\function_exists('route'))
+{
     /**
      * Generate the URL to a named route.
      *

src/Framework/Security/GuardScope.php

Braces +11 added lines, -5 removed lines

@@ -29,8 +29,10 @@ 
     public function allows(string $permission, array $context = []): bool
     {
         $allows = false;
-        foreach ($this->getRoles() as $role) {
-            if (!$this->permissions->hasRole($role)) {
+        foreach ($this->getRoles() as $role)
+        {
+            if (!$this->permissions->hasRole($role))
+            {
                 continue;
             }
 
@@ -70,13 +72,17 @@ 
      */
     public function getActor(): ActorInterface
     {
-        if (!\is_null($this->actor)) {
+        if (!\is_null($this->actor))
+        {
             return $this->actor;
         }
 
-        try {
+        try
+        {
             return $this->container->get(ActorInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }
+        catch (NotFoundExceptionInterface $e)
+        {
             throw new ScopeException('Unable to resolve Actor from the scope', $e->getCode(), $e);
         }
     }

Spacing +6 added lines, -6 removed lines

@@ -22,7 +22,7 @@ 
         private readonly PermissionsInterface $permissions,
         #[Proxy] private readonly ContainerInterface $container,
         private array $roles = []
-    ) {
+    ){
     }
 
     /**
@@ -31,8 +31,8 @@ 
     public function allows(string $permission, array $context = []): bool
     {
         $allows = false;
-        foreach ($this->getRoles() as $role) {
-            if (!$this->permissions->hasRole($role)) {
+        foreach ($this->getRoles() as $role){
+            if (!$this->permissions->hasRole($role)){
                 continue;
             }
 
@@ -72,13 +72,13 @@ 
      */
     public function getActor(): ActorInterface
     {
-        if (!\is_null($this->actor)) {
+        if (!\is_null($this->actor)){
             return $this->actor;
         }
 
-        try {
+        try{
             return $this->container->get(ActorInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }catch (NotFoundExceptionInterface $e){
             throw new ScopeException('Unable to resolve Actor from the scope', $e->getCode(), $e);
         }
     }

src/Framework/Debug/StateCollector/HttpCollector.php

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
 
     public function populate(StateInterface $state): void
     {
-        if ($this->request === null) {
+        if ($this->request === null){
             return;
         }
 
@@ -32,11 +32,11 @@ 
 
         $state->setVariable('headers', $this->request->getHeaders());
 
-        if ($this->request->getQueryParams() !== []) {
+        if ($this->request->getQueryParams() !== []){
             $state->setVariable('query', $this->request->getQueryParams());
         }
 
-        if ($this->request->getParsedBody() !== null) {
+        if ($this->request->getParsedBody() !== null){
             $state->setVariable('data', $this->request->getParsedBody());
         }
     }

Braces +6 added lines, -3 removed lines

@@ -23,7 +23,8 @@ 
 
     public function populate(StateInterface $state): void
     {
-        if ($this->request === null) {
+        if ($this->request === null)
+        {
             return;
         }
 
@@ -32,11 +33,13 @@ 
 
         $state->setVariable('headers', $this->request->getHeaders());
 
-        if ($this->request->getQueryParams() !== []) {
+        if ($this->request->getQueryParams() !== [])
+        {
             $state->setVariable('query', $this->request->getQueryParams());
         }
 
-        if ($this->request->getParsedBody() !== null) {
+        if ($this->request->getParsedBody() !== null)
+        {
             $state->setVariable('data', $this->request->getParsedBody());
         }
     }

src/Framework/Debug/StateCollector/EnvironmentCollector.php

Spacing +2 added lines, -2 removed lines

@@ -15,14 +15,14 @@
     public function __construct(
         private readonly ContainerInterface $container,
         private readonly EnvironmentInterface $env
-    ) {
+    ){
     }
 
     public function populate(StateInterface $state): void
     {
         $state->setTag('php', \phpversion());
 
-        if ($this->container->has(DispatcherInterface::class)) {
+        if ($this->container->has(DispatcherInterface::class)){
             $state->setTag('dispatcher', $this->container->get(DispatcherInterface::class)::class);
         }
 

Braces +2 added lines, -1 removed lines

@@ -22,7 +22,8 @@
     {
         $state->setTag('php', \phpversion());
 
-        if ($this->container->has(DispatcherInterface::class)) {
+        if ($this->container->has(DispatcherInterface::class))
+        {
             $state->setTag('dispatcher', $this->container->get(DispatcherInterface::class)::class);
         }
 

src/Framework/Exceptions/Reporter/FileReporter.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function __construct(
         private FileSnapshot $fileSnapshot,
-    ) {
+    ){
     }
 
     public function report(\Throwable $exception): void

src/Framework/Command/PublishCommand.php

Spacing +6 added lines, -6 removed lines

@@ -37,10 +37,10 @@ 
         FilesInterface $files,
         DirectoriesInterface $directories
     ): int {
-        switch ($this->argument('type')) {
+        switch ($this->argument('type')){
             case 'replace':
             case 'follow':
-                if ($this->isDirectory()) {
+                if ($this->isDirectory()){
                     $this->sprintf(
                         '<fg=cyan>•</fg=cyan> publish directory <comment>%s</comment> to <comment>%s</comment>',
                         $this->getSource($files),
@@ -53,7 +53,7 @@ 
                         $this->getMergeMode(),
                         $this->getFileMode()
                     );
-                } else {
+                }else{
                     $this->sprintf(
                         '<fg=cyan>•</fg=cyan> publish file <comment>%s</comment> to <comment>%s</comment>',
                         $this->getSource($files),
@@ -90,7 +90,7 @@ 
 
     private function getSource(FilesInterface $files): ?string
     {
-        if (!$this->isDirectory()) {
+        if (!$this->isDirectory()){
             return $files->normalizePath($this->argument('source'));
         }
 
@@ -100,7 +100,7 @@ 
     private function getTarget(FilesInterface $files, DirectoriesInterface $directories): ?string
     {
         $target = $this->argument('target');
-        foreach ($directories->getAll() as $alias => $value) {
+        foreach ($directories->getAll() as $alias => $value){
             $target = \str_replace(\sprintf('@%s', $alias), $value, $target);
         }
 
@@ -111,7 +111,7 @@ 
     {
         return match (true) {
             $this->argument('type') === 'ensure' => true,
-            \str_contains((string) $this->argument('source'), '*') => true,
+            \str_contains((string)$this->argument('source'), '*') => true,
             default => \is_dir($this->argument('source'))
         };
     }

Braces +11 added lines, -5 removed lines

@@ -37,10 +37,12 @@ 
         FilesInterface $files,
         DirectoriesInterface $directories
     ): int {
-        switch ($this->argument('type')) {
+        switch ($this->argument('type'))
+        {
             case 'replace':
             case 'follow':
-                if ($this->isDirectory()) {
+                if ($this->isDirectory())
+                {
                     $this->sprintf(
                         '<fg=cyan>•</fg=cyan> publish directory <comment>%s</comment> to <comment>%s</comment>',
                         $this->getSource($files),
@@ -53,7 +55,9 @@ 
                         $this->getMergeMode(),
                         $this->getFileMode()
                     );
-                } else {
+                }
+                else
+                {
                     $this->sprintf(
                         '<fg=cyan>•</fg=cyan> publish file <comment>%s</comment> to <comment>%s</comment>',
                         $this->getSource($files),
@@ -90,7 +94,8 @@ 
 
     private function getSource(FilesInterface $files): ?string
     {
-        if (!$this->isDirectory()) {
+        if (!$this->isDirectory())
+        {
             return $files->normalizePath($this->argument('source'));
         }
 
@@ -100,7 +105,8 @@ 
     private function getTarget(FilesInterface $files, DirectoriesInterface $directories): ?string
     {
         $target = $this->argument('target');
-        foreach ($directories->getAll() as $alias => $value) {
+        foreach ($directories->getAll() as $alias => $value)
+        {
             $target = \str_replace(\sprintf('@%s', $alias), $value, $target);
         }
 

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

@@ -128,7 +128,7 @@
     private function getFileMode(): int
     {
         return match ($this->argument('mode')) {
-            'readonly' => FilesInterface::READONLY,
+            'readonly' => FilesInterface::readonly,
             'runtime' => FilesInterface::RUNTIME,
             default => FilesInterface::RUNTIME,
         };

src/Framework/Command/Views/ResetCommand.php

Spacing +6 added lines, -6 removed lines

@@ -18,20 +18,20 @@ 
 
     public function perform(ViewsConfig $config, FilesInterface $files): int
     {
-        if (!$files->exists($config->getCacheDirectory())) {
+        if (!$files->exists($config->getCacheDirectory())){
             $this->writeln('Cache directory is missing, no cache to be cleaned.');
 
             return self::FAILURE;
         }
 
-        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",
@@ -43,7 +43,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

@@ -18,20 +18,26 @@ 
 
     public function perform(ViewsConfig $config, FilesInterface $files): int
     {
-        if (!$files->exists($config->getCacheDirectory())) {
+        if (!$files->exists($config->getCacheDirectory()))
+        {
             $this->writeln('Cache directory is missing, no cache to be cleaned.');
 
             return self::FAILURE;
         }
 
-        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",
@@ -43,7 +49,8 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($this->isVerbose()) {
+            if ($this->isVerbose())
+            {
                 $this->sprintf(
                     "<fg=green>[deleted]</fg=green> `%s`\n",
                     $files->relativePath($filename, $config->getCacheDirectory())

src/Framework/Command/Views/CompileCommand.php

Spacing +14 added lines, -14 removed lines

@@ -25,13 +25,13 @@ 
         $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) {
-                if ($engine instanceof NativeEngine) {
+        foreach ($contexts as $context){
+            foreach ($views->getEngines() as $engine){
+                if ($engine instanceof NativeEngine){
                     // no need to compile
                     continue;
                 }
@@ -53,19 +53,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);
             }
         }
@@ -75,7 +75,7 @@ 
 
     protected function renderError(string $path, \Throwable $e): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 
@@ -91,7 +91,7 @@ 
     {
         $values = [];
 
-        foreach ($context->getDependencies() as $dependency) {
+        foreach ($context->getDependencies() as $dependency){
             $values[] = \sprintf(
                 '%s%s%s:%s%s%s',
                 Color::LIGHT_WHITE,
@@ -113,11 +113,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');
         }
 
@@ -126,7 +126,7 @@ 
 
     private function renderElapsed(float $start): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose()){
             return;
         }
 

Braces +30 added lines, -14 removed lines

@@ -25,13 +25,17 @@ 
         $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) {
-                if ($engine instanceof NativeEngine) {
+        foreach ($contexts as $context)
+        {
+            foreach ($views->getEngines() as $engine)
+            {
+                if ($engine instanceof NativeEngine)
+                {
                     // no need to compile
                     continue;
                 }
@@ -53,19 +57,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);
             }
         }
@@ -75,7 +86,8 @@ 
 
     protected function renderError(string $path, \Throwable $e): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }
 
@@ -91,7 +103,8 @@ 
     {
         $values = [];
 
-        foreach ($context->getDependencies() as $dependency) {
+        foreach ($context->getDependencies() as $dependency)
+        {
             $values[] = \sprintf(
                 '%s%s%s:%s%s%s',
                 Color::LIGHT_WHITE,
@@ -113,11 +126,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');
         }
 
@@ -126,7 +141,8 @@ 
 
     private function renderElapsed(float $start): void
     {
-        if (!$this->isVerbose()) {
+        if (!$this->isVerbose())
+        {
             return;
         }
 

src/Framework/Command/Translator/ExportCommand.php

Spacing +9 added lines, -9 removed lines

@@ -21,14 +21,14 @@ 
         ['locale', InputArgument::REQUIRED, 'Locale to be dumped'],
         ['path', InputArgument::REQUIRED, 'Export path'],
     ];
-    protected const OPTIONS     = [
+    protected const OPTIONS = [
         ['dumper', 'd', InputOption::VALUE_OPTIONAL, 'Dumper name', 'php'],
         ['fallback', 'f', InputOption::VALUE_NONE, 'Merge messages from fallback catalogue'],
     ];
 
     public function perform(TranslatorConfig $config, CatalogueManager $manager): int
     {
-        if (!$config->hasDumper($this->option('dumper'))) {
+        if (!$config->hasDumper($this->option('dumper'))){
             $this->writeln("<fg=red>Undefined dumper '{$this->option('dumper')}'.</fg=red>");
 
             return self::FAILURE;
@@ -40,7 +40,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",
                 \implode(',', $mc->getDomains())
@@ -58,8 +58,8 @@ 
             ]
         );
 
-        $this->writeln('Export successfully completed using <info>' . $dumper::class . '</info>');
-        $this->writeln('Output: <comment>' . \realpath($this->argument('path')) . '</comment>');
+        $this->writeln('Export successfully completed using <info>'.$dumper::class.'</info>');
+        $this->writeln('Output: <comment>'.\realpath($this->argument('path')).'</comment>');
 
         return self::SUCCESS;
     }
@@ -74,10 +74,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

@@ -28,7 +28,8 @@ 
 
     public function perform(TranslatorConfig $config, CatalogueManager $manager): int
     {
-        if (!$config->hasDumper($this->option('dumper'))) {
+        if (!$config->hasDumper($this->option('dumper')))
+        {
             $this->writeln("<fg=red>Undefined dumper '{$this->option('dumper')}'.</fg=red>");
 
             return self::FAILURE;
@@ -40,7 +41,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",
                 \implode(',', $mc->getDomains())
@@ -74,10 +76,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);
                     }
                 }