Passed
Pull Request — master (#737)
by Vadim
08:14
created
src/Attributes/tests/TestCase.php 2 patches
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -31,7 +31,8 @@  discard block
 block discarded – undo
31 31
      */
32 32
     protected function iterableToArray(iterable $iterable): array
33 33
     {
34
-        if ($iterable instanceof \Traversable) {
34
+        if ($iterable instanceof \Traversable)
35
+        {
35 36
             return \iterator_to_array($iterable, false);
36 37
         }
37 38
 
@@ -47,7 +48,8 @@  discard block
 block discarded – undo
47 48
     {
48 49
         $instance = new $class();
49 50
 
50
-        foreach ($fields as $field => $value) {
51
+        foreach ($fields as $field => $value)
52
+        {
51 53
             $instance->$field = $value;
52 54
         }
53 55
 
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
      */
25 25
     protected function iterableToArray(iterable $iterable): array
26 26
     {
27
-        if ($iterable instanceof \Traversable) {
27
+        if ($iterable instanceof \Traversable){
28 28
             return \iterator_to_array($iterable, false);
29 29
         }
30 30
 
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
     {
50 50
         $instance = new $class();
51 51
 
52
-        foreach ($fields as $field => $value) {
52
+        foreach ($fields as $field => $value){
53 53
             $instance->$field = $value;
54 54
         }
55 55
 
Please login to merge, or discard this patch.
src/Attributes/tests/Concerns/InteractWithReflection.php 2 patches
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -43,13 +43,13 @@  discard block
 block discarded – undo
43 43
             ->getReflectionConstants()
44 44
         ;
45 45
 
46
-        foreach ($constants as $reflection) {
47
-            if ($reflection->getName() === $name) {
46
+        foreach ($constants as $reflection){
47
+            if ($reflection->getName() === $name){
48 48
                 return $reflection;
49 49
             }
50 50
         }
51 51
 
52
-        throw new \ReflectionException('Constant ' . $name . ' not found');
52
+        throw new \ReflectionException('Constant '.$name.' not found');
53 53
     }
54 54
 
55 55
     protected function getReflectionFunctionParameter(string $function, string $name): \ReflectionParameter
@@ -58,13 +58,13 @@  discard block
 block discarded – undo
58 58
             ->getParameters()
59 59
         ;
60 60
 
61
-        foreach ($parameters as $reflection) {
62
-            if ($reflection->getName() === $name) {
61
+        foreach ($parameters as $reflection){
62
+            if ($reflection->getName() === $name){
63 63
                 return $reflection;
64 64
             }
65 65
         }
66 66
 
67
-        throw new \ReflectionException('Parameter ' . $name . ' not found');
67
+        throw new \ReflectionException('Parameter '.$name.' not found');
68 68
     }
69 69
 
70 70
     protected function getReflectionMethodParameter(string $class, string $method, string $name): \ReflectionParameter
@@ -73,12 +73,12 @@  discard block
 block discarded – undo
73 73
             ->getParameters()
74 74
         ;
75 75
 
76
-        foreach ($parameters as $reflection) {
77
-            if ($reflection->getName() === $name) {
76
+        foreach ($parameters as $reflection){
77
+            if ($reflection->getName() === $name){
78 78
                 return $reflection;
79 79
             }
80 80
         }
81 81
 
82
-        throw new \ReflectionException('Parameter ' . $name . ' not found');
82
+        throw new \ReflectionException('Parameter '.$name.' not found');
83 83
     }
84 84
 }
Please login to merge, or discard this patch.
Braces   +12 added lines, -6 removed lines patch added patch discarded remove patch
@@ -43,8 +43,10 @@  discard block
 block discarded – undo
43 43
             ->getReflectionConstants()
44 44
         ;
45 45
 
46
-        foreach ($constants as $reflection) {
47
-            if ($reflection->getName() === $name) {
46
+        foreach ($constants as $reflection)
47
+        {
48
+            if ($reflection->getName() === $name)
49
+            {
48 50
                 return $reflection;
49 51
             }
50 52
         }
@@ -58,8 +60,10 @@  discard block
 block discarded – undo
58 60
             ->getParameters()
59 61
         ;
60 62
 
61
-        foreach ($parameters as $reflection) {
62
-            if ($reflection->getName() === $name) {
63
+        foreach ($parameters as $reflection)
64
+        {
65
+            if ($reflection->getName() === $name)
66
+            {
63 67
                 return $reflection;
64 68
             }
65 69
         }
@@ -73,8 +77,10 @@  discard block
 block discarded – undo
73 77
             ->getParameters()
74 78
         ;
75 79
 
76
-        foreach ($parameters as $reflection) {
77
-            if ($reflection->getName() === $name) {
80
+        foreach ($parameters as $reflection)
81
+        {
82
+            if ($reflection->getName() === $name)
83
+            {
78 84
                 return $reflection;
79 85
             }
80 86
         }
Please login to merge, or discard this patch.
src/DataGrid/src/helpers.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -21,8 +21,8 @@  discard block
 block discarded – undo
21 21
  */
22 22
 function hasKey(array $array, $search): bool
23 23
 {
24
-    foreach ($array as $key => $_) {
25
-        if (equals($key, $search)) {
24
+    foreach ($array as $key => $_){
25
+        if (equals($key, $search)){
26 26
             return true;
27 27
         }
28 28
     }
@@ -37,8 +37,8 @@  discard block
 block discarded – undo
37 37
  */
38 38
 function hasValue(array $array, $search): bool
39 39
 {
40
-    foreach ($array as $value) {
41
-        if (equals($value, $search)) {
40
+    foreach ($array as $value){
41
+        if (equals($value, $search)){
42 42
             return true;
43 43
         }
44 44
     }
@@ -56,8 +56,8 @@  discard block
 block discarded – undo
56 56
  */
57 57
 function getValue(array $array, string $search)
58 58
 {
59
-    foreach ($array as $key => $value) {
60
-        if (equals($key, $search)) {
59
+    foreach ($array as $key => $value){
60
+        if (equals($key, $search)){
61 61
             return $value;
62 62
         }
63 63
     }
@@ -73,11 +73,11 @@  discard block
 block discarded – undo
73 73
  */
74 74
 function equals($value1, $value2): bool
75 75
 {
76
-    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0) {
76
+    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0){
77 77
         return true;
78 78
     }
79 79
 
80
-    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0) {
80
+    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0){
81 81
         return true;
82 82
     }
83 83
 
Please login to merge, or discard this patch.
Braces   +16 added lines, -8 removed lines patch added patch discarded remove patch
@@ -21,8 +21,10 @@  discard block
 block discarded – undo
21 21
  */
22 22
 function hasKey(array $array, $search): bool
23 23
 {
24
-    foreach ($array as $key => $_) {
25
-        if (equals($key, $search)) {
24
+    foreach ($array as $key => $_)
25
+    {
26
+        if (equals($key, $search))
27
+        {
26 28
             return true;
27 29
         }
28 30
     }
@@ -37,8 +39,10 @@  discard block
 block discarded – undo
37 39
  */
38 40
 function hasValue(array $array, $search): bool
39 41
 {
40
-    foreach ($array as $value) {
41
-        if (equals($value, $search)) {
42
+    foreach ($array as $value)
43
+    {
44
+        if (equals($value, $search))
45
+        {
42 46
             return true;
43 47
         }
44 48
     }
@@ -56,8 +60,10 @@  discard block
 block discarded – undo
56 60
  */
57 61
 function getValue(array $array, string $search)
58 62
 {
59
-    foreach ($array as $key => $value) {
60
-        if (equals($key, $search)) {
63
+    foreach ($array as $key => $value)
64
+    {
65
+        if (equals($key, $search))
66
+        {
61 67
             return $value;
62 68
         }
63 69
     }
@@ -73,11 +79,13 @@  discard block
 block discarded – undo
73 79
  */
74 80
 function equals($value1, $value2): bool
75 81
 {
76
-    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0) {
82
+    if (is_string($value1) && is_string($value2) && strcasecmp($value1, $value2) === 0)
83
+    {
77 84
         return true;
78 85
     }
79 86
 
80
-    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0) {
87
+    if (is_numeric($value1) && is_numeric($value2) && strcasecmp((string)$value1, (string)$value2) === 0)
88
+    {
81 89
         return true;
82 90
     }
83 91
 
Please login to merge, or discard this patch.
src/Scaffolder/src/helpers.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@  discard block
 block discarded – undo
11 11
 
12 12
 namespace Spiral\Scaffolder;
13 13
 
14
-if (!function_exists('trimPostfix')) {
14
+if (!function_exists('trimPostfix')){
15 15
     /**
16 16
      * @param string $name
17 17
      * @param string $postfix
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
     }
27 27
 }
28 28
 
29
-if (!function_exists('isAssociativeArray')) {
29
+if (!function_exists('isAssociativeArray')){
30 30
     /**
31 31
      * @param array $array
32 32
      * @return bool
@@ -35,12 +35,12 @@  discard block
 block discarded – undo
35 35
     function isAssociativeArray(array $array): bool
36 36
     {
37 37
         $keys = [];
38
-        foreach ($array as $key => $_) {
39
-            if (!is_int($key)) {
38
+        foreach ($array as $key => $_){
39
+            if (!is_int($key)){
40 40
                 return true;
41 41
             }
42 42
 
43
-            if ($key !== count($keys)) {
43
+            if ($key !== count($keys)){
44 44
                 return true;
45 45
             }
46 46
 
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
     }
52 52
 }
53 53
 
54
-if (!function_exists('defineArrayType')) {
54
+if (!function_exists('defineArrayType')){
55 55
     /**
56 56
      * @param array  $array
57 57
      * @param string|null $failureType
Please login to merge, or discard this patch.
Braces   +12 added lines, -6 removed lines patch added patch discarded remove patch
@@ -11,7 +11,8 @@  discard block
 block discarded – undo
11 11
 
12 12
 namespace Spiral\Scaffolder;
13 13
 
14
-if (!function_exists('trimPostfix')) {
14
+if (!function_exists('trimPostfix'))
15
+{
15 16
     /**
16 17
      * @param string $name
17 18
      * @param string $postfix
@@ -26,7 +27,8 @@  discard block
 block discarded – undo
26 27
     }
27 28
 }
28 29
 
29
-if (!function_exists('isAssociativeArray')) {
30
+if (!function_exists('isAssociativeArray'))
31
+{
30 32
     /**
31 33
      * @param array $array
32 34
      * @return bool
@@ -35,12 +37,15 @@  discard block
 block discarded – undo
35 37
     function isAssociativeArray(array $array): bool
36 38
     {
37 39
         $keys = [];
38
-        foreach ($array as $key => $_) {
39
-            if (!is_int($key)) {
40
+        foreach ($array as $key => $_)
41
+        {
42
+            if (!is_int($key))
43
+            {
40 44
                 return true;
41 45
             }
42 46
 
43
-            if ($key !== count($keys)) {
47
+            if ($key !== count($keys))
48
+            {
44 49
                 return true;
45 50
             }
46 51
 
@@ -51,7 +56,8 @@  discard block
 block discarded – undo
51 56
     }
52 57
 }
53 58
 
54
-if (!function_exists('defineArrayType')) {
59
+if (!function_exists('defineArrayType'))
60
+{
55 61
     /**
56 62
      * @param array  $array
57 63
      * @param string|null $failureType
Please login to merge, or discard this patch.
src/Attributes/src/Composite/MergeReader.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
      */
19 19
     protected function each(callable $resolver): iterable
20 20
     {
21
-        foreach ($this->readers as $reader) {
21
+        foreach ($this->readers as $reader){
22 22
             yield from $resolver($reader);
23 23
         }
24 24
     }
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,8 @@
 block discarded – undo
18 18
      */
19 19
     protected function each(callable $resolver): iterable
20 20
     {
21
-        foreach ($this->readers as $reader) {
21
+        foreach ($this->readers as $reader)
22
+        {
22 23
             yield from $resolver($reader);
23 24
         }
24 25
     }
Please login to merge, or discard this patch.
src/Attributes/src/Composite/SelectiveReader.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,10 +18,10 @@
 block discarded – undo
18 18
      */
19 19
     protected function each(callable $resolver): iterable
20 20
     {
21
-        foreach ($this->readers as $reader) {
21
+        foreach ($this->readers as $reader){
22 22
             $result = $this->iterableToArray($resolver($reader));
23 23
 
24
-            if (\count($result) > 0) {
24
+            if (\count($result) > 0){
25 25
                 return $result;
26 26
             }
27 27
         }
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,10 +18,12 @@
 block discarded – undo
18 18
      */
19 19
     protected function each(callable $resolver): iterable
20 20
     {
21
-        foreach ($this->readers as $reader) {
21
+        foreach ($this->readers as $reader)
22
+        {
22 23
             $result = $this->iterableToArray($resolver($reader));
23 24
 
24
-            if (\count($result) > 0) {
25
+            if (\count($result) > 0)
26
+            {
25 27
                 return $result;
26 28
             }
27 29
         }
Please login to merge, or discard this patch.
src/Framework/Bootloader/Cycle/SchemaBootloader.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -85,11 +85,11 @@  discard block
 block discarded – undo
85 85
     public function getGenerators(): array
86 86
     {
87 87
         $result = [];
88
-        foreach ($this->generators as $group) {
89
-            foreach ($group as $generator) {
90
-                if (is_object($generator) && !$generator instanceof Container\Autowire) {
88
+        foreach ($this->generators as $group){
89
+            foreach ($group as $generator){
90
+                if (is_object($generator) && !$generator instanceof Container\Autowire){
91 91
                     $result[] = $generator;
92
-                } else {
92
+                }else{
93 93
                     $result[] = $this->container->get($generator);
94 94
                 }
95 95
             }
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
     protected function schema(MemoryInterface $memory): SchemaInterface
108 108
     {
109 109
         $schemaCompiler = SchemaCompiler::fromMemory($memory);
110
-        if ($schemaCompiler->isEmpty()) {
110
+        if ($schemaCompiler->isEmpty()){
111 111
             $schemaCompiler = SchemaCompiler::compile(
112 112
                 $this->container->get(Registry::class),
113 113
                 $this->getGenerators()
Please login to merge, or discard this patch.
Braces   +11 added lines, -5 removed lines patch added patch discarded remove patch
@@ -85,11 +85,16 @@  discard block
 block discarded – undo
85 85
     public function getGenerators(): array
86 86
     {
87 87
         $result = [];
88
-        foreach ($this->generators as $group) {
89
-            foreach ($group as $generator) {
90
-                if (is_object($generator) && !$generator instanceof Container\Autowire) {
88
+        foreach ($this->generators as $group)
89
+        {
90
+            foreach ($group as $generator)
91
+            {
92
+                if (is_object($generator) && !$generator instanceof Container\Autowire)
93
+                {
91 94
                     $result[] = $generator;
92
-                } else {
95
+                }
96
+                else
97
+                {
93 98
                     $result[] = $this->container->get($generator);
94 99
                 }
95 100
             }
@@ -107,7 +112,8 @@  discard block
 block discarded – undo
107 112
     protected function schema(MemoryInterface $memory): SchemaInterface
108 113
     {
109 114
         $schemaCompiler = SchemaCompiler::fromMemory($memory);
110
-        if ($schemaCompiler->isEmpty()) {
115
+        if ($schemaCompiler->isEmpty())
116
+        {
111 117
             $schemaCompiler = SchemaCompiler::compile(
112 118
                 $this->container->get(Registry::class),
113 119
                 $this->getGenerators()
Please login to merge, or discard this patch.
src/Framework/Command/Cycle/SyncCommand.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
         );
45 45
         $schemaCompiler->toMemory($memory);
46 46
 
47
-        if ($show->hasChanges()) {
47
+        if ($show->hasChanges()){
48 48
             $this->writeln("\n<info>ORM Schema has been synchronized</info>");
49 49
         }
50 50
     }
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,8 @@
 block discarded – undo
44 44
         );
45 45
         $schemaCompiler->toMemory($memory);
46 46
 
47
-        if ($show->hasChanges()) {
47
+        if ($show->hasChanges())
48
+        {
48 49
             $this->writeln("\n<info>ORM Schema has been synchronized</info>");
49 50
         }
50 51
     }
Please login to merge, or discard this patch.
src/Prototype/tests/Commands/AbstractCommandsTest.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -39,13 +39,13 @@  discard block
 block discarded – undo
39 39
 
40 40
     public function __construct($name = null, array $data = [], $dataName = '')
41 41
     {
42
-        $this->storage = new Storage($this->dir() . '/Fixtures/');
42
+        $this->storage = new Storage($this->dir().'/Fixtures/');
43 43
         parent::__construct($name, $data, $dataName);
44 44
     }
45 45
 
46 46
     public function setUp(): void
47 47
     {
48
-        if (!\class_exists(Kernel::class)) {
48
+        if (!\class_exists(Kernel::class)){
49 49
             $this->markTestSkipped('A "spiral/framework" dependency is required to run these tests');
50 50
         }
51 51
 
@@ -56,14 +56,14 @@  discard block
 block discarded – undo
56 56
             'cache'  => sys_get_temp_dir()
57 57
         ], null, false);
58 58
 
59
-        foreach (static::STORE as $name) {
59
+        foreach (static::STORE as $name){
60 60
             $this->storage->store($name);
61 61
         }
62 62
     }
63 63
 
64 64
     public function tearDown(): void
65 65
     {
66
-        foreach (static::STORE as $name) {
66
+        foreach (static::STORE as $name){
67 67
             $this->storage->restore($name);
68 68
         }
69 69
     }
Please login to merge, or discard this patch.
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -45,7 +45,8 @@  discard block
 block discarded – undo
45 45
 
46 46
     public function setUp(): void
47 47
     {
48
-        if (!\class_exists(Kernel::class)) {
48
+        if (!\class_exists(Kernel::class))
49
+        {
49 50
             $this->markTestSkipped('A "spiral/framework" dependency is required to run these tests');
50 51
         }
51 52
 
@@ -56,14 +57,16 @@  discard block
 block discarded – undo
56 57
             'cache'  => sys_get_temp_dir()
57 58
         ], null, false);
58 59
 
59
-        foreach (static::STORE as $name) {
60
+        foreach (static::STORE as $name)
61
+        {
60 62
             $this->storage->store($name);
61 63
         }
62 64
     }
63 65
 
64 66
     public function tearDown(): void
65 67
     {
66
-        foreach (static::STORE as $name) {
68
+        foreach (static::STORE as $name)
69
+        {
67 70
             $this->storage->restore($name);
68 71
         }
69 72
     }
Please login to merge, or discard this patch.