Passed
Pull Request — master (#817)
by Maxim
11:28 queued 05:29
created
src/Core/tests/Internal/Resolver/CommonCasesTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
         $this->expectExceptionMessage('Enum `Spiral\Tests\Core\Stub\EnumObject` can not be constructed.');
64 64
 
65 65
         $this->resolveClosure(
66
-            static function (EnumObject $enum) {},
66
+            static function (EnumObject $enum){},
67 67
         );
68 68
     }
69 69
 
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
         );
76 76
 
77 77
         $this->resolveClosure(
78
-            static function (TestTrait $enum) {},
78
+            static function (TestTrait $enum){},
79 79
         );
80 80
     }
81 81
 }
Please login to merge, or discard this patch.
src/Core/tests/ExceptionsTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -79,9 +79,9 @@  discard block
 block discarded – undo
79 79
 
80 80
         $this->expectException(ContainerException::class);
81 81
 
82
-        try {
82
+        try{
83 83
             $container->get(WithContainerInside::class);
84
-        } catch (ContainerException $e) {
84
+        }catch (ContainerException $e){
85 85
             $this->assertSame(
86 86
                 \preg_replace('/\s+/', '', 'Can\'tresolve`Spiral\Tests\Core\Fixtures\
87 87
                     WithContainerInside`:undefinedclassorbinding`Spiral\Tests\Core\Fixtures\InvalidClass`.
@@ -101,9 +101,9 @@  discard block
 block discarded – undo
101 101
     {
102 102
         $this->expectException(ContainerException::class);
103 103
 
104
-        try {
104
+        try{
105 105
             $container->get(ClassWithUndefinedDependency::class);
106
-        } catch (ContainerException $e) {
106
+        }catch (ContainerException $e){
107 107
             $this->assertSame(
108 108
                 \preg_replace('/\s+/', '', $message),
109 109
                 \preg_replace('/\s+/', '', $e->getMessage())
Please login to merge, or discard this patch.
src/Core/src/Internal/Trace.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -13,16 +13,16 @@
 block discarded – undo
13 13
         public readonly string $alias,
14 14
         public readonly string $information,
15 15
         public ?string $context = null
16
-    ) {
16
+    ){
17 17
         $this->context ??= '-';
18 18
     }
19 19
 
20 20
     public function __toString(): string
21 21
     {
22 22
         $result = [];
23
-        $result[] = '- ' . $this->alias;
24
-        $result[] = '    Info: ' . $this->information;
25
-        $result[] = '    Context: ' . $this->context;
23
+        $result[] = '- '.$this->alias;
24
+        $result[] = '    Info: '.$this->information;
25
+        $result[] = '    Context: '.$this->context;
26 26
 
27 27
         return \implode(PHP_EOL, $result);
28 28
     }
Please login to merge, or discard this patch.
src/Core/src/Exception/Container/ContainerException.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
         int $code = 0,
19 19
         ?\Throwable $previous = null,
20 20
         protected ?Tracer $tracer = null
21
-    ) {
22
-        parent::__construct($tracer !== null ? $message . PHP_EOL . $tracer : $message, $code, $previous);
21
+    ){
22
+        parent::__construct($tracer !== null ? $message.PHP_EOL.$tracer : $message, $code, $previous);
23 23
     }
24 24
 }
Please login to merge, or discard this patch.
src/Core/src/Config.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
         public readonly string $binder = Binder::class,
41 41
         public readonly string $invoker = Invoker::class,
42 42
         public readonly string $tracer = Tracer::class,
43
-    ) {
43
+    ){
44 44
     }
45 45
 
46 46
     public function getIterator(): Traversable
Please login to merge, or discard this patch.
src/Core/src/Internal/Factory.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
      */
54 54
     public function make(string $alias, array $parameters = [], string $context = null): mixed
55 55
     {
56
-        if (!isset($this->state->bindings[$alias])) {
56
+        if (!isset($this->state->bindings[$alias])){
57 57
             $this->tracer->traceAutowire($alias, $context);
58 58
             //No direct instructions how to construct class, make is automatically
59 59
             return $this->autowire($alias, $parameters, $context);
@@ -62,9 +62,9 @@  discard block
 block discarded – undo
62 62
         $binding = $this->state->bindings[$alias];
63 63
         $this->tracer->traceBinding($alias, $binding, $context);
64 64
 
65
-        if (\is_object($binding)) {
66
-            if ($binding::class === WeakReference::class) {
67
-                if ($binding->get() === null && \class_exists($alias)) {
65
+        if (\is_object($binding)){
66
+            if ($binding::class === WeakReference::class){
67
+                if ($binding->get() === null && \class_exists($alias)){
68 68
                     $object = $this->createInstance($alias, $parameters, $context);
69 69
                     $binding = $this->state->bindings[$alias] = WeakReference::create($object);
70 70
                 }
@@ -74,23 +74,23 @@  discard block
 block discarded – undo
74 74
             return $binding;
75 75
         }
76 76
 
77
-        if (\is_string($binding)) {
77
+        if (\is_string($binding)){
78 78
             //Binding is pointing to something else
79 79
             return $this->make($binding, $parameters, $context);
80 80
         }
81 81
 
82 82
         unset($this->state->bindings[$alias]);
83
-        try {
83
+        try{
84 84
             $instance = $binding[0] === $alias
85 85
                 ? $this->autowire($alias, $parameters, $context)
86 86
                 : $this->evaluateBinding($alias, $binding[0], $parameters, $context);
87
-        } finally {
87
+        }finally{
88 88
             /** @psalm-var class-string $alias */
89 89
             $this->state->bindings[$alias] = $binding;
90 90
             $this->tracer->clean();
91 91
         }
92 92
 
93
-        if ($binding[1]) {
93
+        if ($binding[1]){
94 94
             // Indicates singleton
95 95
             /** @psalm-var class-string $alias */
96 96
             $this->state->bindings[$alias] = $instance;
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
      */
110 110
     private function autowire(string $class, array $parameters, string $context = null): object
111 111
     {
112
-        if (!\class_exists($class) && !isset($this->state->injectors[$class])) {
112
+        if (!\class_exists($class) && !isset($this->state->injectors[$class])){
113 113
             throw new NotFoundException(\sprintf(
114 114
                 'Can\'t resolve `%s`: undefined class or binding `%s`.',
115 115
                 $this->tracer->getRootConstructedClass(),
@@ -136,18 +136,18 @@  discard block
 block discarded – undo
136 136
         array $parameters,
137 137
         string $context = null
138 138
     ): mixed {
139
-        if (\is_string($target)) {
139
+        if (\is_string($target)){
140 140
             // Reference
141 141
             return $this->make($target, $parameters, $context);
142 142
         }
143 143
 
144
-        if ($target instanceof Autowire) {
144
+        if ($target instanceof Autowire){
145 145
             return $target->resolve($this, $parameters);
146 146
         }
147 147
 
148
-        try {
148
+        try{
149 149
             return $this->invoker->invoke($target, $parameters);
150
-        } catch (NotCallableException $e) {
150
+        }catch (NotCallableException $e){
151 151
             throw new ContainerException(
152 152
                 \sprintf('Invalid binding for `%s`.', $alias),
153 153
                 $e->getCode(),
@@ -172,20 +172,20 @@  discard block
 block discarded – undo
172 172
      */
173 173
     private function createInstance(string $class, array $parameters, string $context = null): object
174 174
     {
175
-        try {
175
+        try{
176 176
             $reflection = new \ReflectionClass($class);
177
-        } catch (\ReflectionException $e) {
177
+        }catch (\ReflectionException $e){
178 178
             throw new ContainerException($e->getMessage(), $e->getCode(), $e, $this->tracer);
179 179
         }
180 180
 
181 181
         //We have to construct class using external injector when we know exact context
182
-        if ($parameters === [] && $this->binder->hasInjector($class)) {
182
+        if ($parameters === [] && $this->binder->hasInjector($class)){
183 183
             $injector = $this->state->injectors[$reflection->getName()];
184 184
 
185
-            try {
185
+            try{
186 186
                 $injectorInstance = $this->container->get($injector);
187 187
 
188
-                if (!$injectorInstance instanceof InjectorInterface) {
188
+                if (!$injectorInstance instanceof InjectorInterface){
189 189
                     throw new InjectionException(
190 190
                         \sprintf(
191 191
                             "Class '%s' must be an instance of InjectorInterface for '%s'.",
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
 
198 198
                 /** @var InjectorInterface<TObject> $injectorInstance */
199 199
                 $instance = $injectorInstance->createInjection($reflection, $context);
200
-                if (!$reflection->isInstance($instance)) {
200
+                if (!$reflection->isInstance($instance)){
201 201
                     throw new InjectionException(
202 202
                         \sprintf(
203 203
                             "Invalid injection response for '%s'.",
@@ -207,12 +207,12 @@  discard block
 block discarded – undo
207 207
                 }
208 208
 
209 209
                 return $instance;
210
-            } finally {
210
+            }finally{
211 211
                 $this->state->injectors[$reflection->getName()] = $injector;
212 212
             }
213 213
         }
214 214
 
215
-        if (!$reflection->isInstantiable()) {
215
+        if (!$reflection->isInstantiable()){
216 216
             $itIs = match (true) {
217 217
                 $reflection->isEnum() => 'Enum',
218 218
                 $reflection->isAbstract() => 'Abstract class',
@@ -228,14 +228,14 @@  discard block
 block discarded – undo
228 228
 
229 229
         $constructor = $reflection->getConstructor();
230 230
 
231
-        if ($constructor !== null) {
232
-            try {
231
+        if ($constructor !== null){
232
+            try{
233 233
                 // Using constructor with resolved arguments
234 234
                 $instance = new $class(...$this->resolver->resolveArguments($constructor, $parameters));
235
-            } catch (\TypeError $e) {
235
+            }catch (\TypeError $e){
236 236
                 throw new WrongTypeException($constructor, $e);
237 237
             }
238
-        } else {
238
+        }else{
239 239
             // No constructor specified
240 240
             $instance = $reflection->newInstance();
241 241
         }
@@ -253,9 +253,9 @@  discard block
 block discarded – undo
253 253
     private function registerInstance(object $instance, array $parameters): object
254 254
     {
255 255
         //Declarative singletons (only when class received via direct get)
256
-        if ($parameters === [] && $instance instanceof SingletonInterface) {
256
+        if ($parameters === [] && $instance instanceof SingletonInterface){
257 257
             $alias = $instance::class;
258
-            if (!isset($this->state->bindings[$alias])) {
258
+            if (!isset($this->state->bindings[$alias])){
259 259
                 $this->state->bindings[$alias] = $instance;
260 260
             }
261 261
         }
Please login to merge, or discard this patch.
src/Core/src/Internal/Tracer.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -17,11 +17,11 @@  discard block
 block discarded – undo
17 17
     public function __toString(): string
18 18
     {
19 19
         $result = [];
20
-        if ($this->traces !== []) {
20
+        if ($this->traces !== []){
21 21
             $result[] = 'Container trace list:';
22 22
 
23
-            foreach ($this->traces as $item) {
24
-                $result[] = (string) $item;
23
+            foreach ($this->traces as $item){
24
+                $result[] = (string)$item;
25 25
             }
26 26
         }
27 27
 
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
         $this->trace($alias, 'Autowiring', $context);
34 34
     }
35 35
 
36
-    public function traceBinding(string $alias, string|array|object $binding, string $context = null): void
36
+    public function traceBinding(string $alias, string | array | object $binding, string $context = null): void
37 37
     {
38 38
         $message = match (true) {
39 39
             \is_string($binding) => \sprintf('Binding found `%s`', $binding),
Please login to merge, or discard this patch.