spiral / framework

src/SendIt/src/MailJob.php

Spacing +6 added lines, -6 removed lines

@@ -45,11 +45,11 @@ 
      */
     public function handle(string $name, string $id, $payload): void
     {
-        if (\is_string($payload)) {
+        if (\is_string($payload)){
             $payload = json_decode($payload, true, 512, JSON_THROW_ON_ERROR);
         }
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload)){
             throw new InvalidArgumentException('Mail job payload should be an array.');
         }
 
@@ -57,15 +57,15 @@ 
 
         $email = $this->renderer->render($message);
 
-        if ($email->getFrom() === []) {
+        if ($email->getFrom() === []){
             $email->from(Address::create($this->config->getFromAddress()));
         }
 
         $recipients = $this->getRecipients($email);
 
-        try {
+        try{
             $this->mailer->send($email);
-        } catch (TransportExceptionInterface $e) {
+        }catch (TransportExceptionInterface $e){
             $this->getLogger()->error(
                 sprintf(
                     'Failed to send `%s` to "%s": %s',
@@ -95,7 +95,7 @@ 
 
         $addresses = array_merge($message->getTo(), $message->getCc(), $message->getBcc());
 
-        foreach ($addresses as $address) {
+        foreach ($addresses as $address){
             $emails[] = $address->toString();
         }
 

Braces +13 added lines, -6 removed lines

@@ -45,11 +45,13 @@ 
      */
     public function handle(string $name, string $id, $payload): void
     {
-        if (\is_string($payload)) {
+        if (\is_string($payload))
+        {
             $payload = json_decode($payload, true, 512, JSON_THROW_ON_ERROR);
         }
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload))
+        {
             throw new InvalidArgumentException('Mail job payload should be an array.');
         }
 
@@ -57,15 +59,19 @@ 
 
         $email = $this->renderer->render($message);
 
-        if ($email->getFrom() === []) {
+        if ($email->getFrom() === [])
+        {
             $email->from(Address::create($this->config->getFromAddress()));
         }
 
         $recipients = $this->getRecipients($email);
 
-        try {
+        try
+        {
             $this->mailer->send($email);
-        } catch (TransportExceptionInterface $e) {
+        }
+        catch (TransportExceptionInterface $e)
+        {
             $this->getLogger()->error(
                 sprintf(
                     'Failed to send `%s` to "%s": %s',
@@ -95,7 +101,8 @@ 
 
         $addresses = array_merge($message->getTo(), $message->getCc(), $message->getBcc());
 
-        foreach ($addresses as $address) {
+        foreach ($addresses as $address)
+        {
             $emails[] = $address->toString();
         }
 

src/AnnotatedRoutes/src/RouteLocator.php

Spacing +5 added lines, -5 removed lines

@@ -45,11 +45,11 @@ 
     public function findDeclarations(): array
     {
         $result = [];
-        foreach ($this->locator->getScopedClasses('routes') as $class) {
-            foreach ($class->getMethods() as $method) {
+        foreach ($this->locator->getScopedClasses('routes') as $class){
+            foreach ($class->getMethods() as $method){
                 $route = $this->reader->firstFunctionMetadata($method, Route::class);
 
-                if ($route === null) {
+                if ($route === null){
                     continue;
                 }
 
@@ -59,9 +59,9 @@ 
                     'controller' => $class->getName(),
                     'action'     => $method->getName(),
                     'group'      => $route->group,
-                    'verbs'      => (array) $route->methods,
+                    'verbs'      => (array)$route->methods,
                     'defaults'   => $route->defaults,
-                    'middleware' => (array) $route->middleware,
+                    'middleware' => (array)$route->middleware,
                     'priority'   => $route->priority,
                 ];
             }

Braces +6 added lines, -3 removed lines

@@ -45,11 +45,14 @@
     public function findDeclarations(): array
     {
         $result = [];
-        foreach ($this->locator->getScopedClasses('routes') as $class) {
-            foreach ($class->getMethods() as $method) {
+        foreach ($this->locator->getScopedClasses('routes') as $class)
+        {
+            foreach ($class->getMethods() as $method)
+            {
                 $route = $this->reader->firstFunctionMetadata($method, Route::class);
 
-                if ($route === null) {
+                if ($route === null)
+                {
                     continue;
                 }
 

src/Queue/src/Bootloader/QueueBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
             $registry = $container->get(HandlerRegistryInterface::class);
             $config = $container->get(QueueConfig::class);
 
-            foreach ($config->getRegistryHandlers() as $jobType => $handler) {
+            foreach ($config->getRegistryHandlers() as $jobType => $handler){
                 $registry->setHandler($jobType, $handler);
             }
         });

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
             $registry = $container->get(HandlerRegistryInterface::class);
             $config = $container->get(QueueConfig::class);
 
-            foreach ($config->getRegistryHandlers() as $jobType => $handler) {
+            foreach ($config->getRegistryHandlers() as $jobType => $handler)
+            {
                 $registry->setHandler($jobType, $handler);
             }
         });

src/Queue/src/ContainerRegistry.php

Spacing +3 added lines, -3 removed lines

@@ -23,13 +23,13 @@
 
     public function getHandler(string $jobType): HandlerInterface
     {
-        try {
+        try{
             $handler = $this->container->get($this->className($jobType));
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             throw new JobException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (!$handler instanceof HandlerInterface) {
+        if (!$handler instanceof HandlerInterface){
             throw new JobException("Unable to resolve job handler for `{$jobType}`");
         }
 

Braces +7 added lines, -3 removed lines

@@ -23,13 +23,17 @@
 
     public function getHandler(string $jobType): HandlerInterface
     {
-        try {
+        try
+        {
             $handler = $this->container->get($this->className($jobType));
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             throw new JobException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (!$handler instanceof HandlerInterface) {
+        if (!$handler instanceof HandlerInterface)
+        {
             throw new JobException("Unable to resolve job handler for `{$jobType}`");
         }
 

src/Queue/src/QueueManager.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
         // Replaces alias with real pipeline name
         $name = $this->config->getAliases()[$name] ?? $name;
 
-        if (!isset($this->pipelines[$name])) {
+        if (!isset($this->pipelines[$name])){
             $this->pipelines[$name] = $this->resolveConnection($name);
         }
 
@@ -43,9 +43,9 @@ 
     {
         $config = $this->config->getConnection($name);
 
-        try {
+        try{
             return $this->factory->make($config['driver'], $config);
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             throw new NotSupportedDriverException(
                 \sprintf(
                     'Driver `%s` is not supported. Connection `%s` cannot be created. Reason: `%s`',

Braces +7 added lines, -3 removed lines

@@ -28,7 +28,8 @@ 
         // Replaces alias with real pipeline name
         $name = $this->config->getAliases()[$name] ?? $name;
 
-        if (!isset($this->pipelines[$name])) {
+        if (!isset($this->pipelines[$name]))
+        {
             $this->pipelines[$name] = $this->resolveConnection($name);
         }
 
@@ -43,9 +44,12 @@ 
     {
         $config = $this->config->getConnection($name);
 
-        try {
+        try
+        {
             return $this->factory->make($config['driver'], $config);
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             throw new NotSupportedDriverException(
                 \sprintf(
                     'Driver `%s` is not supported. Connection `%s` cannot be created. Reason: `%s`',

src/Tokenizer/src/Reflection/ReflectionArgument.php

Spacing +14 added lines, -14 removed lines

@@ -21,8 +21,8 @@ 
     /**
      * Argument types.
      */
-    public const CONSTANT   = 'constant';   //Scalar constant and not variable.
-    public const VARIABLE   = 'variable';   //PHP variable
+    public const CONSTANT   = 'constant'; //Scalar constant and not variable.
+    public const VARIABLE   = 'variable'; //PHP variable
     public const EXPRESSION = 'expression'; //PHP code (expression).
     public const STRING     = 'string';
     private string $type;
@@ -59,7 +59,7 @@ 
      */
     public function stringValue(): string
     {
-        if ($this->type != self::STRING) {
+        if ($this->type != self::STRING){
             throw new ReflectionException(
                 "Unable to represent value as string, value type is '{$this->type}'"
             );
@@ -80,19 +80,19 @@ 
         $level = 0;
 
         $result = [];
-        foreach ($tokens as $token) {
-            if ($token[ReflectionFile::TOKEN_TYPE] == T_WHITESPACE) {
+        foreach ($tokens as $token){
+            if ($token[ReflectionFile::TOKEN_TYPE] == T_WHITESPACE){
                 continue;
             }
 
-            if (empty($definition)) {
+            if (empty($definition)){
                 $definition = ['type' => self::EXPRESSION, 'value' => '', 'tokens' => []];
             }
 
             if (
                 $token[ReflectionFile::TOKEN_TYPE] == '('
                 || $token[ReflectionFile::TOKEN_TYPE] == '['
-            ) {
+            ){
                 ++$level;
                 $definition['value'] .= $token[ReflectionFile::TOKEN_CODE];
                 continue;
@@ -101,18 +101,18 @@ 
             if (
                 $token[ReflectionFile::TOKEN_TYPE] == ')'
                 || $token[ReflectionFile::TOKEN_TYPE] == ']'
-            ) {
+            ){
                 --$level;
                 $definition['value'] .= $token[ReflectionFile::TOKEN_CODE];
                 continue;
             }
 
-            if ($level) {
+            if ($level){
                 $definition['value'] .= $token[ReflectionFile::TOKEN_CODE];
                 continue;
             }
 
-            if ($token[ReflectionFile::TOKEN_TYPE] == ',') {
+            if ($token[ReflectionFile::TOKEN_TYPE] == ','){
                 $result[] = self::createArgument($definition);
                 $definition = null;
                 continue;
@@ -123,9 +123,9 @@ 
         }
 
         //Last argument
-        if (is_array($definition)) {
+        if (is_array($definition)){
             $definition = self::createArgument($definition);
-            if (!empty($definition->getType())) {
+            if (!empty($definition->getType())){
                 $result[] = $definition;
             }
         }
@@ -142,9 +142,9 @@ 
     {
         $result = new static(self::EXPRESSION, $definition['value']);
 
-        if (count($definition['tokens']) == 1) {
+        if (count($definition['tokens']) == 1){
             //If argument represent by one token we can try to resolve it's type more precisely
-            switch ($definition['tokens'][0][0]) {
+            switch ($definition['tokens'][0][0]){
                 case T_VARIABLE:
                     $result->type = self::VARIABLE;
                     break;

src/Security/src/Traits/GuardedTrait.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
     public function getGuard(): GuardInterface
     {
         $container = ContainerScope::getContainer();
-        if (empty($container) || !$container->has(GuardInterface::class)) {
+        if (empty($container) || !$container->has(GuardInterface::class)){
             throw new ScopeException(
                 'Unable to get `GuardInterface`, binding is missing or container scope is not set'
             );
@@ -51,9 +51,9 @@ 
      */
     protected function resolvePermission(string $permission): string
     {
-        if (defined('static::GUARD_NAMESPACE')) {
+        if (defined('static::GUARD_NAMESPACE')){
             // Yay! Isolation
-            $permission = constant(static::class . '::' . 'GUARD_NAMESPACE') . '.' . $permission;
+            $permission = constant(static::class.'::'.'GUARD_NAMESPACE').'.'.$permission;
         }
 
         return $permission;

Braces +4 added lines, -2 removed lines

@@ -27,7 +27,8 @@ 
     public function getGuard(): GuardInterface
     {
         $container = ContainerScope::getContainer();
-        if (empty($container) || !$container->has(GuardInterface::class)) {
+        if (empty($container) || !$container->has(GuardInterface::class))
+        {
             throw new ScopeException(
                 'Unable to get `GuardInterface`, binding is missing or container scope is not set'
             );
@@ -51,7 +52,8 @@ 
      */
     protected function resolvePermission(string $permission): string
     {
-        if (defined('static::GUARD_NAMESPACE')) {
+        if (defined('static::GUARD_NAMESPACE'))
+        {
             // Yay! Isolation
             $permission = constant(static::class . '::' . 'GUARD_NAMESPACE') . '.' . $permission;
         }

src/Attributes/src/Internal/FallbackAttributeReader/AttributeFinderVisitor.php

Spacing +18 added lines, -18 removed lines

@@ -140,23 +140,23 @@ 
     {
         $this->updateContext($node);
 
-        if ($node instanceof ClassLike) {
-            foreach ($this->parse($node->attrGroups) as $prototype) {
+        if ($node instanceof ClassLike){
+            foreach ($this->parse($node->attrGroups) as $prototype){
                 $this->classes[$node->namespacedName->toString()][] = $prototype;
             }
 
             return null;
         }
 
-        if ($node instanceof FunctionLike) {
+        if ($node instanceof FunctionLike){
             $line = $node->getEndLine();
 
-            foreach ($this->parse($node->getAttrGroups()) as $prototype) {
+            foreach ($this->parse($node->getAttrGroups()) as $prototype){
                 $this->functions[$line][] = $prototype;
             }
 
-            foreach ($node->getParams() as $param) {
-                foreach ($this->parse($param->attrGroups) as $prototype) {
+            foreach ($node->getParams() as $param){
+                foreach ($this->parse($param->attrGroups) as $prototype){
                     assert(\is_string($param->var->name), 'Function parameter name should be an identifier');
 
                     $this->parameters[$line][$param->var->name][] = $prototype;
@@ -166,11 +166,11 @@ 
             return NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN;
         }
 
-        if ($node instanceof ClassConst) {
+        if ($node instanceof ClassConst){
             $class = $this->fqn();
 
-            foreach ($this->parse($node->attrGroups) as $prototype) {
-                foreach ($node->consts as $const) {
+            foreach ($this->parse($node->attrGroups) as $prototype){
+                foreach ($node->consts as $const){
                     $this->constants[$class][$this->name($const->name)][] = $prototype;
                 }
             }
@@ -178,11 +178,11 @@ 
             return NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN;
         }
 
-        if ($node instanceof Property) {
+        if ($node instanceof Property){
             $class = $this->fqn();
 
-            foreach ($this->parse($node->attrGroups) as $prototype) {
-                foreach ($node->props as $property) {
+            foreach ($this->parse($node->attrGroups) as $prototype){
+                foreach ($node->props as $property){
                     $this->properties[$class][$this->name($property->name)][] = $prototype;
                 }
             }
@@ -195,27 +195,27 @@ 
 
     public function leaveNode(Node $node): void
     {
-        if ($node instanceof Namespace_) {
+        if ($node instanceof Namespace_){
             $this->context[AttributeParser::CTX_NAMESPACE] = '';
 
             return;
         }
 
-        if ($node instanceof ClassLike) {
+        if ($node instanceof ClassLike){
             $this->context[AttributeParser::CTX_CLASS] = '';
             $this->context[AttributeParser::CTX_TRAIT] = '';
 
             return;
         }
 
-        if ($node instanceof FunctionLike) {
+        if ($node instanceof FunctionLike){
             $this->context[AttributeParser::CTX_FUNCTION] = '';
         }
     }
 
     private function updateContext(Node $node): void
     {
-        switch (true) {
+        switch (true){
             case $node instanceof Namespace_:
                 $this->context[AttributeParser::CTX_NAMESPACE] = $this->name($node->name);
                 break;
@@ -240,7 +240,7 @@ 
      */
     private function name($name): string
     {
-        if ($name === null) {
+        if ($name === null){
             return '';
         }
 
@@ -262,6 +262,6 @@ 
         $namespace = $this->context[AttributeParser::CTX_NAMESPACE] ?? '';
         $class = $this->context[AttributeParser::CTX_CLASS] ?? '';
 
-        return \trim($namespace . '\\' . $class, '\\');
+        return \trim($namespace.'\\'.$class, '\\');
     }
 }

Braces +34 added lines, -17 removed lines

@@ -140,23 +140,29 @@ 
     {
         $this->updateContext($node);
 
-        if ($node instanceof ClassLike) {
-            foreach ($this->parse($node->attrGroups) as $prototype) {
+        if ($node instanceof ClassLike)
+        {
+            foreach ($this->parse($node->attrGroups) as $prototype)
+            {
                 $this->classes[$node->namespacedName->toString()][] = $prototype;
             }
 
             return null;
         }
 
-        if ($node instanceof FunctionLike) {
+        if ($node instanceof FunctionLike)
+        {
             $line = $node->getEndLine();
 
-            foreach ($this->parse($node->getAttrGroups()) as $prototype) {
+            foreach ($this->parse($node->getAttrGroups()) as $prototype)
+            {
                 $this->functions[$line][] = $prototype;
             }
 
-            foreach ($node->getParams() as $param) {
-                foreach ($this->parse($param->attrGroups) as $prototype) {
+            foreach ($node->getParams() as $param)
+            {
+                foreach ($this->parse($param->attrGroups) as $prototype)
+                {
                     assert(\is_string($param->var->name), 'Function parameter name should be an identifier');
 
                     $this->parameters[$line][$param->var->name][] = $prototype;
@@ -166,11 +172,14 @@ 
             return NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN;
         }
 
-        if ($node instanceof ClassConst) {
+        if ($node instanceof ClassConst)
+        {
             $class = $this->fqn();
 
-            foreach ($this->parse($node->attrGroups) as $prototype) {
-                foreach ($node->consts as $const) {
+            foreach ($this->parse($node->attrGroups) as $prototype)
+            {
+                foreach ($node->consts as $const)
+                {
                     $this->constants[$class][$this->name($const->name)][] = $prototype;
                 }
             }
@@ -178,11 +187,14 @@ 
             return NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN;
         }
 
-        if ($node instanceof Property) {
+        if ($node instanceof Property)
+        {
             $class = $this->fqn();
 
-            foreach ($this->parse($node->attrGroups) as $prototype) {
-                foreach ($node->props as $property) {
+            foreach ($this->parse($node->attrGroups) as $prototype)
+            {
+                foreach ($node->props as $property)
+                {
                     $this->properties[$class][$this->name($property->name)][] = $prototype;
                 }
             }
@@ -195,27 +207,31 @@ 
 
     public function leaveNode(Node $node): void
     {
-        if ($node instanceof Namespace_) {
+        if ($node instanceof Namespace_)
+        {
             $this->context[AttributeParser::CTX_NAMESPACE] = '';
 
             return;
         }
 
-        if ($node instanceof ClassLike) {
+        if ($node instanceof ClassLike)
+        {
             $this->context[AttributeParser::CTX_CLASS] = '';
             $this->context[AttributeParser::CTX_TRAIT] = '';
 
             return;
         }
 
-        if ($node instanceof FunctionLike) {
+        if ($node instanceof FunctionLike)
+        {
             $this->context[AttributeParser::CTX_FUNCTION] = '';
         }
     }
 
     private function updateContext(Node $node): void
     {
-        switch (true) {
+        switch (true)
+        {
             case $node instanceof Namespace_:
                 $this->context[AttributeParser::CTX_NAMESPACE] = $this->name($node->name);
                 break;
@@ -240,7 +256,8 @@ 
      */
     private function name($name): string
     {
-        if ($name === null) {
+        if ($name === null)
+        {
             return '';
         }
 

src/Dumper/src/helpers.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 use Spiral\Core\ContainerScope;
 use Spiral\Debug\Dumper;
 
-if (!function_exists('dump')) {
+if (!function_exists('dump')){
     /**
      * Dump value.
      *
@@ -23,12 +23,12 @@ 
      */
     function dump($value, int $output = Dumper::OUTPUT): ?string
     {
-        if (!class_exists(ContainerScope::class)) {
+        if (!class_exists(ContainerScope::class)){
             return (new Dumper())->dump($value, $output);
         }
 
         $container = ContainerScope::getContainer();
-        if (is_null($container) || !$container->has(Dumper::class)) {
+        if (is_null($container) || !$container->has(Dumper::class)){
             $dumper = new Dumper();
 
             return $dumper->dump($value, $output);

Braces +6 added lines, -3 removed lines

@@ -12,7 +12,8 @@ 
 use Spiral\Core\ContainerScope;
 use Spiral\Debug\Dumper;
 
-if (!function_exists('dump')) {
+if (!function_exists('dump'))
+{
     /**
      * Dump value.
      *
@@ -23,12 +24,14 @@ 
      */
     function dump($value, int $output = Dumper::OUTPUT): ?string
     {
-        if (!class_exists(ContainerScope::class)) {
+        if (!class_exists(ContainerScope::class))
+        {
             return (new Dumper())->dump($value, $output);
         }
 
         $container = ContainerScope::getContainer();
-        if (is_null($container) || !$container->has(Dumper::class)) {
+        if (is_null($container) || !$container->has(Dumper::class))
+        {
             $dumper = new Dumper();
 
             return $dumper->dump($value, $output);