Passed
Pull Request — master (#1139)
by Abdul Malik
11:09
created
src/Telemetry/src/Config/TelemetryConfig.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
      */
26 26
     public function getDefaultDriver(): string
27 27
     {
28
-        if (!\is_string($this->config['default'])) {
28
+        if (!\is_string($this->config['default'])){
29 29
             throw new InvalidArgumentException('Default trace driver config value must be a string');
30 30
         }
31 31
 
@@ -37,9 +37,9 @@  discard block
 block discarded – undo
37 37
      * @return class-string<TracerFactoryInterface>|Autowire|TracerFactoryInterface
38 38
      * @throws InvalidArgumentException
39 39
      */
40
-    public function getDriverConfig(string $name): string|Autowire|TracerFactoryInterface
40
+    public function getDriverConfig(string $name): string | Autowire | TracerFactoryInterface
41 41
     {
42
-        if (!isset($this->config['drivers'][$name])) {
42
+        if (!isset($this->config['drivers'][$name])){
43 43
             throw new InvalidArgumentException(
44 44
                 \sprintf('Config for telemetry driver `%s` is not defined.', $name)
45 45
             );
@@ -47,11 +47,11 @@  discard block
 block discarded – undo
47 47
 
48 48
         $driver = $this->config['drivers'][$name];
49 49
 
50
-        if ($driver instanceof TracerFactoryInterface) {
50
+        if ($driver instanceof TracerFactoryInterface){
51 51
             return $driver;
52 52
         }
53 53
 
54
-        if (!\is_string($driver) && !$driver instanceof Autowire) {
54
+        if (!\is_string($driver) && !$driver instanceof Autowire){
55 55
             throw new InvalidArgumentException(
56 56
                 \sprintf('Trace type value for `%s` must be a string or %s', $name, Autowire::class)
57 57
             );
Please login to merge, or discard this patch.
Braces   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -25,7 +25,8 @@  discard block
 block discarded – undo
25 25
      */
26 26
     public function getDefaultDriver(): string
27 27
     {
28
-        if (!\is_string($this->config['default'])) {
28
+        if (!\is_string($this->config['default']))
29
+        {
29 30
             throw new InvalidArgumentException('Default trace driver config value must be a string');
30 31
         }
31 32
 
@@ -39,7 +40,8 @@  discard block
 block discarded – undo
39 40
      */
40 41
     public function getDriverConfig(string $name): string|Autowire|TracerFactoryInterface
41 42
     {
42
-        if (!isset($this->config['drivers'][$name])) {
43
+        if (!isset($this->config['drivers'][$name]))
44
+        {
43 45
             throw new InvalidArgumentException(
44 46
                 \sprintf('Config for telemetry driver `%s` is not defined.', $name)
45 47
             );
@@ -47,11 +49,13 @@  discard block
 block discarded – undo
47 49
 
48 50
         $driver = $this->config['drivers'][$name];
49 51
 
50
-        if ($driver instanceof TracerFactoryInterface) {
52
+        if ($driver instanceof TracerFactoryInterface)
53
+        {
51 54
             return $driver;
52 55
         }
53 56
 
54
-        if (!\is_string($driver) && !$driver instanceof Autowire) {
57
+        if (!\is_string($driver) && !$driver instanceof Autowire)
58
+        {
55 59
             throw new InvalidArgumentException(
56 60
                 \sprintf('Trace type value for `%s` must be a string or %s', $name, Autowire::class)
57 61
             );
Please login to merge, or discard this patch.
src/Telemetry/src/LogTracerFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
         private readonly ClockInterface $clock,
20 20
         LogsInterface $logs,
21 21
         string $channel = self::LOG_CHANNEL
22
-    ) {
22
+    ){
23 23
         $this->logger = $logs->getLogger($channel);
24 24
     }
25 25
 
Please login to merge, or discard this patch.
src/Telemetry/src/Span/Status.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -11,8 +11,8 @@
 block discarded – undo
11 11
      * @param non-empty-string|null $description
12 12
      */
13 13
     public function __construct(
14
-        public readonly string|int $code,
14
+        public readonly string | int $code,
15 15
         public readonly ?string $description = null
16
-    ) {
16
+    ){
17 17
     }
18 18
 }
Please login to merge, or discard this patch.
src/Telemetry/src/Bootloader/TelemetryBootloader.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
 
37 37
     public function __construct(
38 38
         private readonly ConfiguratorInterface $config
39
-    ) {
39
+    ){
40 40
     }
41 41
 
42 42
     public function init(EnvironmentInterface $env): void
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     /**
48 48
      * @param class-string<TracerFactoryInterface>|TracerFactoryInterface|Autowire $driver
49 49
      */
50
-    public function registerTracer(string $name, string|TracerFactoryInterface|Autowire $driver): void
50
+    public function registerTracer(string $name, string | TracerFactoryInterface | Autowire $driver): void
51 51
     {
52 52
         $this->config->modify(
53 53
             TelemetryConfig::CONFIG,
Please login to merge, or discard this patch.
src/Telemetry/src/SpanInterface.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
      * @param non-empty-string|int $code
29 29
      * @param non-empty-string|null $description
30 30
      */
31
-    public function setStatus(string|int $code, string $description = null): self;
31
+    public function setStatus(string | int $code, string $description = null): self;
32 32
 
33 33
     /**
34 34
      * Get the current span status.
Please login to merge, or discard this patch.
src/Telemetry/src/LogTracer.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
         private readonly ClockInterface $clock,
24 24
         private readonly LoggerInterface $logger,
25 25
         private readonly UuidFactoryInterface $uuidFactory
26
-    ) {
26
+    ){
27 27
         parent::__construct($scope);
28 28
     }
29 29
 
Please login to merge, or discard this patch.
src/Queue/tests/Interceptor/Consume/HandlerTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@
 block discarded – undo
22 22
         $tracerFactory->shouldReceive('make')
23 23
             ->once()
24 24
             ->with(['some' => 'data'])
25
-            ->andReturn( $tracer = new NullTracer());
25
+            ->andReturn($tracer = new NullTracer());
26 26
 
27 27
         $handler = new Handler(
28 28
             core: $core = m::mock(CoreInterface::class),
Please login to merge, or discard this patch.
src/Queue/src/Core/QueueInjector.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -18,24 +18,24 @@
 block discarded – undo
18 18
 {
19 19
     public function __construct(
20 20
         private readonly QueueConnectionProviderInterface $queueManager
21
-    ) {
21
+    ){
22 22
     }
23 23
 
24 24
     public function createInjection(ReflectionClass $class, string $context = null): QueueInterface
25 25
     {
26
-        try {
27
-            if ($context === null) {
26
+        try{
27
+            if ($context === null){
28 28
                 $connection = $this->queueManager->getConnection();
29
-            } else {
29
+            }else{
30 30
                 // Get Queue by context
31
-                try {
31
+                try{
32 32
                     $connection = $this->queueManager->getConnection($context);
33
-                } catch (InvalidArgumentException) {
33
+                }catch (InvalidArgumentException){
34 34
                     // Case when context doesn't match to configured connections
35 35
                     return $this->queueManager->getConnection();
36 36
                 }
37 37
             }
38
-        } catch (\Throwable $e) {
38
+        }catch (\Throwable $e){
39 39
             throw new ContainerException(\sprintf("Can't inject the required queue. %s", $e->getMessage()), 0, $e);
40 40
         }
41 41
 
Please login to merge, or discard this patch.
Braces   +15 added lines, -6 removed lines patch added patch discarded remove patch
@@ -23,19 +23,28 @@
 block discarded – undo
23 23
 
24 24
     public function createInjection(ReflectionClass $class, string $context = null): QueueInterface
25 25
     {
26
-        try {
27
-            if ($context === null) {
26
+        try
27
+        {
28
+            if ($context === null)
29
+            {
28 30
                 $connection = $this->queueManager->getConnection();
29
-            } else {
31
+            }
32
+            else
33
+            {
30 34
                 // Get Queue by context
31
-                try {
35
+                try
36
+                {
32 37
                     $connection = $this->queueManager->getConnection($context);
33
-                } catch (InvalidArgumentException) {
38
+                }
39
+                catch (InvalidArgumentException)
40
+                {
34 41
                     // Case when context doesn't match to configured connections
35 42
                     return $this->queueManager->getConnection();
36 43
                 }
37 44
             }
38
-        } catch (\Throwable $e) {
45
+        }
46
+        catch (\Throwable $e)
47
+        {
39 48
             throw new ContainerException(\sprintf("Can't inject the required queue. %s", $e->getMessage()), 0, $e);
40 49
         }
41 50
 
Please login to merge, or discard this patch.
src/Queue/tests/Core/QueueInjectorTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@
 block discarded – undo
87 87
         $factory->shouldReceive('make')->andReturnUsing(function (string $name): QueueInterface {
88 88
             $result = ['sync' => $this->defaultQueue, 'test' => $this->testQueue][$name] ?? null;
89 89
 
90
-            if ($result === null) {
90
+            if ($result === null){
91 91
                 throw new NotFoundException();
92 92
             }
93 93
             return $result;
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,8 @@
 block discarded – undo
87 87
         $factory->shouldReceive('make')->andReturnUsing(function (string $name): QueueInterface {
88 88
             $result = ['sync' => $this->defaultQueue, 'test' => $this->testQueue][$name] ?? null;
89 89
 
90
-            if ($result === null) {
90
+            if ($result === null)
91
+            {
91 92
                 throw new NotFoundException();
92 93
             }
93 94
             return $result;
Please login to merge, or discard this patch.