spiral / framework

src/Attributes/tests/Concerns/InteractWithReflection.php

Spacing +9 added lines, -9 removed lines

@@ -43,13 +43,13 @@ 
             ->getReflectionConstants()
         ;
 
-        foreach ($constants as $reflection) {
-            if ($reflection->getName() === $name) {
+        foreach ($constants as $reflection){
+            if ($reflection->getName() === $name){
                 return $reflection;
             }
         }
 
-        throw new \ReflectionException('Constant ' . $name . ' not found');
+        throw new \ReflectionException('Constant '.$name.' not found');
     }
 
     protected function getReflectionFunctionParameter(string $function, string $name): \ReflectionParameter
@@ -58,13 +58,13 @@ 
             ->getParameters()
         ;
 
-        foreach ($parameters as $reflection) {
-            if ($reflection->getName() === $name) {
+        foreach ($parameters as $reflection){
+            if ($reflection->getName() === $name){
                 return $reflection;
             }
         }
 
-        throw new \ReflectionException('Parameter ' . $name . ' not found');
+        throw new \ReflectionException('Parameter '.$name.' not found');
     }
 
     protected function getReflectionMethodParameter(string $class, string $method, string $name): \ReflectionParameter
@@ -73,12 +73,12 @@ 
             ->getParameters()
         ;
 
-        foreach ($parameters as $reflection) {
-            if ($reflection->getName() === $name) {
+        foreach ($parameters as $reflection){
+            if ($reflection->getName() === $name){
                 return $reflection;
             }
         }
 
-        throw new \ReflectionException('Parameter ' . $name . ' not found');
+        throw new \ReflectionException('Parameter '.$name.' not found');
     }
 }

Braces +12 added lines, -6 removed lines

@@ -43,8 +43,10 @@ 
             ->getReflectionConstants()
         ;
 
-        foreach ($constants as $reflection) {
-            if ($reflection->getName() === $name) {
+        foreach ($constants as $reflection)
+        {
+            if ($reflection->getName() === $name)
+            {
                 return $reflection;
             }
         }
@@ -58,8 +60,10 @@ 
             ->getParameters()
         ;
 
-        foreach ($parameters as $reflection) {
-            if ($reflection->getName() === $name) {
+        foreach ($parameters as $reflection)
+        {
+            if ($reflection->getName() === $name)
+            {
                 return $reflection;
             }
         }
@@ -73,8 +77,10 @@ 
             ->getParameters()
         ;
 
-        foreach ($parameters as $reflection) {
-            if ($reflection->getName() === $name) {
+        foreach ($parameters as $reflection)
+        {
+            if ($reflection->getName() === $name)
+            {
                 return $reflection;
             }
         }

src/DataGrid/src/helpers.php

Spacing +8 added lines, -8 removed lines

@@ -21,8 +21,8 @@ 
  */
 function hasKey(array $array, $search): bool
 {
-    foreach ($array as $key => $_) {
-        if (equals($key, $search)) {
+    foreach ($array as $key => $_){
+        if (equals($key, $search)){
             return true;
         }
     }
@@ -37,8 +37,8 @@ 
  */
 function hasValue(array $array, $search): bool
 {
-    foreach ($array as $value) {
-        if (equals($value, $search)) {
+    foreach ($array as $value){
+        if (equals($value, $search)){
             return true;
         }
     }
@@ -56,8 +56,8 @@ 
  */
 function getValue(array $array, string $search)
 {
-    foreach ($array as $key => $value) {
-        if (equals($key, $search)) {
+    foreach ($array as $key => $value){
+        if (equals($key, $search)){
             return $value;
         }
     }
@@ -73,11 +73,11 @@ 
  */
 function equals($value1, $value2): bool
 {
-    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0) {
+    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0){
         return true;
     }
 
-    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0) {
+    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0){
         return true;
     }
 

Braces +16 added lines, -8 removed lines

@@ -21,8 +21,10 @@ 
  */
 function hasKey(array $array, $search): bool
 {
-    foreach ($array as $key => $_) {
-        if (equals($key, $search)) {
+    foreach ($array as $key => $_)
+    {
+        if (equals($key, $search))
+        {
             return true;
         }
     }
@@ -37,8 +39,10 @@ 
  */
 function hasValue(array $array, $search): bool
 {
-    foreach ($array as $value) {
-        if (equals($value, $search)) {
+    foreach ($array as $value)
+    {
+        if (equals($value, $search))
+        {
             return true;
         }
     }
@@ -56,8 +60,10 @@ 
  */
 function getValue(array $array, string $search)
 {
-    foreach ($array as $key => $value) {
-        if (equals($key, $search)) {
+    foreach ($array as $key => $value)
+    {
+        if (equals($key, $search))
+        {
             return $value;
         }
     }
@@ -73,11 +79,13 @@ 
  */
 function equals($value1, $value2): bool
 {
-    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0) {
+    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0)
+    {
         return true;
     }
 
-    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0) {
+    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0)
+    {
         return true;
     }
 

src/Models/src/AbstractEntity.php

Spacing +28 added lines, -28 removed lines

@@ -79,7 +79,7 @@ 
      */
     public function hasField(string $name): bool
     {
-        if (!array_key_exists($name, $this->fields)) {
+        if (!array_key_exists($name, $this->fields)){
             return false;
         }
 
@@ -95,14 +95,14 @@ 
      */
     public function setField(string $name, $value, bool $filter = true): void
     {
-        if ($value instanceof ValueInterface) {
+        if ($value instanceof ValueInterface){
             //In case of non scalar values filters must be bypassed (check accessor compatibility?)
             $this->fields[$name] = clone $value;
 
             return;
         }
 
-        if (!$filter || (is_null($value) && $this->isNullable($name))) {
+        if (!$filter || (is_null($value) && $this->isNullable($name))){
             //Bypassing all filters
             $this->fields[$name] = $value;
 
@@ -112,10 +112,10 @@ 
         //Checking if field have accessor
         $accessor = $this->getMutator($name, ModelSchema::MUTATOR_ACCESSOR);
 
-        if ($accessor !== null) {
+        if ($accessor !== null){
             //Setting value thought associated accessor
             $this->thoughValue($accessor, $name, $value);
-        } else {
+        }else{
             //Setting value thought setter filter (if any)
             $this->setMutated($name, $value);
         }
@@ -132,7 +132,7 @@ 
     {
         $value = $this->hasField($name) ? $this->fields[$name] : $default;
 
-        if ($value instanceof ValueInterface || (is_null($value) && $this->isNullable($name))) {
+        if ($value instanceof ValueInterface || (is_null($value) && $this->isNullable($name))){
             //Direct access to value when value is accessor or null and declared as nullable
             return $value;
         }
@@ -140,7 +140,7 @@ 
         //Checking if field have accessor (decorator)
         $accessor = $this->getMutator($name, ModelSchema::MUTATOR_ACCESSOR);
 
-        if (!empty($accessor)) {
+        if (!empty($accessor)){
             return $this->fields[$name] = $this->createValue($accessor, $name, $value);
         }
 
@@ -163,15 +163,15 @@ 
      */
     public function setFields($fields = [], bool $all = false)
     {
-        if (!is_array($fields) && !$fields instanceof \Traversable) {
+        if (!is_array($fields) && !$fields instanceof \Traversable){
             return $this;
         }
 
-        foreach ($fields as $name => $value) {
-            if ($all || $this->isFillable($name)) {
-                try {
+        foreach ($fields as $name => $value){
+            if ($all || $this->isFillable($name)){
+                try{
                     $this->setField($name, $value, true);
-                } catch (AccessExceptionInterface $e) {
+                }catch (AccessExceptionInterface $e){
                     //We are suppressing field setting exceptions
                 }
             }
@@ -192,7 +192,7 @@ 
     public function getFields(bool $filter = true): array
     {
         $result = [];
-        foreach ($this->fields as $name => $_) {
+        foreach ($this->fields as $name => $_){
             $result[$name] = $this->getField($name, null, $filter);
         }
 
@@ -259,10 +259,10 @@ 
     public function getValue(): array
     {
         $result = [];
-        foreach ($this->fields as $field => $value) {
-            if ($value instanceof ValueInterface) {
+        foreach ($this->fields as $field => $value){
+            if ($value instanceof ValueInterface){
                 $result[$field] = $value->getValue();
-            } else {
+            }else{
                 $result[$field] = $value;
             }
         }
@@ -355,9 +355,9 @@ 
         $value,
         array $context = []
     ): ValueInterface {
-        if (!is_string($type) || !class_exists($type)) {
+        if (!is_string($type) || !class_exists($type)){
             throw new EntityException(
-                "Unable to create accessor for field `{$name}` in " . static::class
+                "Unable to create accessor for field `{$name}` in ".static::class
             );
         }
 
@@ -377,10 +377,10 @@ 
     {
         $getter = $this->getMutator($name, ModelSchema::MUTATOR_GETTER);
 
-        if ($filter && !empty($getter)) {
-            try {
+        if ($filter && !empty($getter)){
+            try{
                 return call_user_func($getter, $value);
-            } catch (\Exception $e) {
+            }catch (\Exception $e){
                 //Trying to filter null value, every filter must support it
                 return call_user_func($getter, null);
             }
@@ -399,13 +399,13 @@ 
     {
         $setter = $this->getMutator($name, ModelSchema::MUTATOR_SETTER);
 
-        if (!empty($setter)) {
-            try {
+        if (!empty($setter)){
+            try{
                 $this->fields[$name] = call_user_func($setter, $value);
-            } catch (\Exception $e) {
+            }catch (\Exception $e){
                 //Exceptional situation, we are choosing to keep original field value
             }
-        } else {
+        }else{
             $this->fields[$name] = $value;
         }
     }
@@ -419,13 +419,13 @@ 
      */
     private function thoughValue($type, string $name, $value): void
     {
-        if (array_key_exists($name, $this->fields)) {
+        if (array_key_exists($name, $this->fields)){
             $field = $this->fields[$name];
-        } else {
+        }else{
             $field = null;
         }
 
-        if (empty($field) || !($field instanceof ValueInterface)) {
+        if (empty($field) || !($field instanceof ValueInterface)){
             //New field representation
             $field = $this->createValue($type, $name, $value);
 

Braces +61 added lines, -27 removed lines

@@ -79,7 +79,8 @@ 
      */
     public function hasField(string $name): bool
     {
-        if (!array_key_exists($name, $this->fields)) {
+        if (!array_key_exists($name, $this->fields))
+        {
             return false;
         }
 
@@ -95,14 +96,16 @@ 
      */
     public function setField(string $name, $value, bool $filter = true): void
     {
-        if ($value instanceof ValueInterface) {
+        if ($value instanceof ValueInterface)
+        {
             //In case of non scalar values filters must be bypassed (check accessor compatibility?)
             $this->fields[$name] = clone $value;
 
             return;
         }
 
-        if (!$filter || (is_null($value) && $this->isNullable($name))) {
+        if (!$filter || (is_null($value) && $this->isNullable($name)))
+        {
             //Bypassing all filters
             $this->fields[$name] = $value;
 
@@ -112,10 +115,13 @@ 
         //Checking if field have accessor
         $accessor = $this->getMutator($name, ModelSchema::MUTATOR_ACCESSOR);
 
-        if ($accessor !== null) {
+        if ($accessor !== null)
+        {
             //Setting value thought associated accessor
             $this->thoughValue($accessor, $name, $value);
-        } else {
+        }
+        else
+        {
             //Setting value thought setter filter (if any)
             $this->setMutated($name, $value);
         }
@@ -132,7 +138,8 @@ 
     {
         $value = $this->hasField($name) ? $this->fields[$name] : $default;
 
-        if ($value instanceof ValueInterface || (is_null($value) && $this->isNullable($name))) {
+        if ($value instanceof ValueInterface || (is_null($value) && $this->isNullable($name)))
+        {
             //Direct access to value when value is accessor or null and declared as nullable
             return $value;
         }
@@ -140,7 +147,8 @@ 
         //Checking if field have accessor (decorator)
         $accessor = $this->getMutator($name, ModelSchema::MUTATOR_ACCESSOR);
 
-        if (!empty($accessor)) {
+        if (!empty($accessor))
+        {
             return $this->fields[$name] = $this->createValue($accessor, $name, $value);
         }
 
@@ -163,15 +171,21 @@ 
      */
     public function setFields($fields = [], bool $all = false)
     {
-        if (!is_array($fields) && !$fields instanceof \Traversable) {
+        if (!is_array($fields) && !$fields instanceof \Traversable)
+        {
             return $this;
         }
 
-        foreach ($fields as $name => $value) {
-            if ($all || $this->isFillable($name)) {
-                try {
+        foreach ($fields as $name => $value)
+        {
+            if ($all || $this->isFillable($name))
+            {
+                try
+                {
                     $this->setField($name, $value, true);
-                } catch (AccessExceptionInterface $e) {
+                }
+                catch (AccessExceptionInterface $e)
+                {
                     //We are suppressing field setting exceptions
                 }
             }
@@ -192,7 +206,8 @@ 
     public function getFields(bool $filter = true): array
     {
         $result = [];
-        foreach ($this->fields as $name => $_) {
+        foreach ($this->fields as $name => $_)
+        {
             $result[$name] = $this->getField($name, null, $filter);
         }
 
@@ -259,10 +274,14 @@ 
     public function getValue(): array
     {
         $result = [];
-        foreach ($this->fields as $field => $value) {
-            if ($value instanceof ValueInterface) {
+        foreach ($this->fields as $field => $value)
+        {
+            if ($value instanceof ValueInterface)
+            {
                 $result[$field] = $value->getValue();
-            } else {
+            }
+            else
+            {
                 $result[$field] = $value;
             }
         }
@@ -355,7 +374,8 @@ 
         $value,
         array $context = []
     ): ValueInterface {
-        if (!is_string($type) || !class_exists($type)) {
+        if (!is_string($type) || !class_exists($type))
+        {
             throw new EntityException(
                 "Unable to create accessor for field `{$name}` in " . static::class
             );
@@ -377,10 +397,14 @@ 
     {
         $getter = $this->getMutator($name, ModelSchema::MUTATOR_GETTER);
 
-        if ($filter && !empty($getter)) {
-            try {
+        if ($filter && !empty($getter))
+        {
+            try
+            {
                 return call_user_func($getter, $value);
-            } catch (\Exception $e) {
+            }
+            catch (\Exception $e)
+            {
                 //Trying to filter null value, every filter must support it
                 return call_user_func($getter, null);
             }
@@ -399,13 +423,19 @@ 
     {
         $setter = $this->getMutator($name, ModelSchema::MUTATOR_SETTER);
 
-        if (!empty($setter)) {
-            try {
+        if (!empty($setter))
+        {
+            try
+            {
                 $this->fields[$name] = call_user_func($setter, $value);
-            } catch (\Exception $e) {
+            }
+            catch (\Exception $e)
+            {
                 //Exceptional situation, we are choosing to keep original field value
             }
-        } else {
+        }
+        else
+        {
             $this->fields[$name] = $value;
         }
     }
@@ -419,13 +449,17 @@ 
      */
     private function thoughValue($type, string $name, $value): void
     {
-        if (array_key_exists($name, $this->fields)) {
+        if (array_key_exists($name, $this->fields))
+        {
             $field = $this->fields[$name];
-        } else {
+        }
+        else
+        {
             $field = null;
         }
 
-        if (empty($field) || !($field instanceof ValueInterface)) {
+        if (empty($field) || !($field instanceof ValueInterface))
+        {
             //New field representation
             $field = $this->createValue($type, $name, $value);
 

src/Scaffolder/src/helpers.php

Spacing +6 added lines, -6 removed lines

@@ -11,7 +11,7 @@ 
 
 namespace Spiral\Scaffolder;
 
-if (!function_exists('trimPostfix')) {
+if (!function_exists('trimPostfix')){
     /**
      * @param string $name
      * @param string $postfix
@@ -26,7 +26,7 @@ 
     }
 }
 
-if (!function_exists('isAssociativeArray')) {
+if (!function_exists('isAssociativeArray')){
     /**
      * @param array $array
      * @return bool
@@ -35,12 +35,12 @@ 
     function isAssociativeArray(array $array): bool
     {
         $keys = [];
-        foreach ($array as $key => $_) {
-            if (!is_int($key)) {
+        foreach ($array as $key => $_){
+            if (!is_int($key)){
                 return true;
             }
 
-            if ($key !== count($keys)) {
+            if ($key !== count($keys)){
                 return true;
             }
 
@@ -51,7 +51,7 @@ 
     }
 }
 
-if (!function_exists('defineArrayType')) {
+if (!function_exists('defineArrayType')){
     /**
      * @param array  $array
      * @param string|null $failureType

Braces +12 added lines, -6 removed lines

@@ -11,7 +11,8 @@ 
 
 namespace Spiral\Scaffolder;
 
-if (!function_exists('trimPostfix')) {
+if (!function_exists('trimPostfix'))
+{
     /**
      * @param string $name
      * @param string $postfix
@@ -26,7 +27,8 @@ 
     }
 }
 
-if (!function_exists('isAssociativeArray')) {
+if (!function_exists('isAssociativeArray'))
+{
     /**
      * @param array $array
      * @return bool
@@ -35,12 +37,15 @@ 
     function isAssociativeArray(array $array): bool
     {
         $keys = [];
-        foreach ($array as $key => $_) {
-            if (!is_int($key)) {
+        foreach ($array as $key => $_)
+        {
+            if (!is_int($key))
+            {
                 return true;
             }
 
-            if ($key !== count($keys)) {
+            if ($key !== count($keys))
+            {
                 return true;
             }
 
@@ -51,7 +56,8 @@ 
     }
 }
 
-if (!function_exists('defineArrayType')) {
+if (!function_exists('defineArrayType'))
+{
     /**
      * @param array  $array
      * @param string|null $failureType

src/Attributes/src/Composite/MergeReader.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
      */
     protected function each(callable $resolver): iterable
     {
-        foreach ($this->readers as $reader) {
+        foreach ($this->readers as $reader){
             yield from $resolver($reader);
         }
     }

Braces +2 added lines, -1 removed lines

@@ -18,7 +18,8 @@
      */
     protected function each(callable $resolver): iterable
     {
-        foreach ($this->readers as $reader) {
+        foreach ($this->readers as $reader)
+        {
             yield from $resolver($reader);
         }
     }

src/Attributes/src/Composite/SelectiveReader.php

Spacing +2 added lines, -2 removed lines

@@ -18,10 +18,10 @@
      */
     protected function each(callable $resolver): iterable
     {
-        foreach ($this->readers as $reader) {
+        foreach ($this->readers as $reader){
             $result = $this->iterableToArray($resolver($reader));
 
-            if (\count($result) > 0) {
+            if (\count($result) > 0){
                 return $result;
             }
         }

Braces +4 added lines, -2 removed lines

@@ -18,10 +18,12 @@
      */
     protected function each(callable $resolver): iterable
     {
-        foreach ($this->readers as $reader) {
+        foreach ($this->readers as $reader)
+        {
             $result = $this->iterableToArray($resolver($reader));
 
-            if (\count($result) > 0) {
+            if (\count($result) > 0)
+            {
                 return $result;
             }
         }

src/Framework/Bootloader/Cycle/SchemaBootloader.php

Spacing +5 added lines, -5 removed lines

@@ -85,11 +85,11 @@ 
     public function getGenerators(): array
     {
         $result = [];
-        foreach ($this->generators as $group) {
-            foreach ($group as $generator) {
-                if (is_object($generator) && !$generator instanceof Container\Autowire) {
+        foreach ($this->generators as $group){
+            foreach ($group as $generator){
+                if (is_object($generator) && !$generator instanceof Container\Autowire){
                     $result[] = $generator;
-                } else {
+                }else{
                     $result[] = $this->container->get($generator);
                 }
             }
@@ -107,7 +107,7 @@ 
     protected function schema(MemoryInterface $memory): SchemaInterface
     {
         $schemaCompiler = SchemaCompiler::fromMemory($memory);
-        if ($schemaCompiler->isEmpty()) {
+        if ($schemaCompiler->isEmpty()){
             $schemaCompiler = SchemaCompiler::compile(
                 $this->container->get(Registry::class),
                 $this->getGenerators()

Braces +11 added lines, -5 removed lines

@@ -85,11 +85,16 @@ 
     public function getGenerators(): array
     {
         $result = [];
-        foreach ($this->generators as $group) {
-            foreach ($group as $generator) {
-                if (is_object($generator) && !$generator instanceof Container\Autowire) {
+        foreach ($this->generators as $group)
+        {
+            foreach ($group as $generator)
+            {
+                if (is_object($generator) && !$generator instanceof Container\Autowire)
+                {
                     $result[] = $generator;
-                } else {
+                }
+                else
+                {
                     $result[] = $this->container->get($generator);
                 }
             }
@@ -107,7 +112,8 @@ 
     protected function schema(MemoryInterface $memory): SchemaInterface
     {
         $schemaCompiler = SchemaCompiler::fromMemory($memory);
-        if ($schemaCompiler->isEmpty()) {
+        if ($schemaCompiler->isEmpty())
+        {
             $schemaCompiler = SchemaCompiler::compile(
                 $this->container->get(Registry::class),
                 $this->getGenerators()

src/Framework/Command/Cycle/SyncCommand.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         );
         $schemaCompiler->toMemory($memory);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges()){
             $this->writeln("\n<info>ORM Schema has been synchronized</info>");
         }
     }

Braces +2 added lines, -1 removed lines

@@ -44,7 +44,8 @@
         );
         $schemaCompiler->toMemory($memory);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges())
+        {
             $this->writeln("\n<info>ORM Schema has been synchronized</info>");
         }
     }

src/Framework/Command/Cycle/MigrateCommand.php

Spacing +5 added lines, -5 removed lines

@@ -50,12 +50,12 @@ 
         Migrator $migrator,
         Console $console
     ): void {
-        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 @@ 
         );
         $schemaCompiler->toMemory($memory);
 
-        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

@@ -50,12 +50,15 @@ 
         Migrator $migrator,
         Console $console
     ): void {
-        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 @@ 
         );
         $schemaCompiler->toMemory($memory);
 
-        if ($show->hasChanges()) {
+        if ($show->hasChanges())
+        {
             (new Compiler())->compile($registry, [$migrations]);
 
-            if ($this->option('run')) {
+            if ($this->option('run'))
+            {
                 $console->run('migrate', [], $this->output);
             }
         }