8p / EightPointsGuzzleBundle

src/Middleware/EventDispatchMiddleware.php

Spacing +4 added lines, -4 removed lines

@@ -39,9 +39,9 @@ 
      */
     public function dispatchEvent() : \Closure
     {
-        return function (callable $handler) {
+        return function(callable $handler) {
 
-            return function (
+            return function(
                 RequestInterface $request,
                 array $options
             ) use ($handler) {
@@ -56,7 +56,7 @@ 
 
                 // Handle the response form the server.
                 return $promise->then(
-                    function (ResponseInterface $response) {
+                    function(ResponseInterface $response) {
                         // Create the Post Transaction event.
                         $postTransactionEvent = new PostTransactionEvent($response, $this->serviceName);
 
@@ -66,7 +66,7 @@ 
                         // Continue down the chain.
                         return $postTransactionEvent->getTransaction();
                     },
-                    function (Exception $reason) {
+                    function(Exception $reason) {
                         // Get the response. The response in a RequestException can be null too.
                         $response = $reason instanceof RequestException ? $reason->getResponse() : null;
 

src/Middleware/SymfonyLogMiddleware.php

Spacing +3 added lines, -3 removed lines

@@ -34,11 +34,11 @@ 
         $logger    = $this->logger;
         $formatter = $this->formatter;
 
-        return function ($request, array $options) use ($handler, $logger, $formatter) {
+        return function($request, array $options) use ($handler, $logger, $formatter) {
 
             return $handler($request, $options)->then(
 
-                function ($response) use ($logger, $request, $formatter) {
+                function($response) use ($logger, $request, $formatter) {
                     $message = $formatter->format($request, $response);
 
                     $logger->info($message);
@@ -46,7 +46,7 @@ 
                     return $response;
                 },
 
-                function ($reason) use ($logger, $request, $formatter) {
+                function($reason) use ($logger, $request, $formatter) {
                     $response = $reason instanceof RequestException ? $reason->getResponse() : null;
                     $message  = $formatter->format($request, $response, $reason);
 

src/Middleware/LogMiddleware.php

Spacing +4 added lines, -4 removed lines

@@ -34,9 +34,9 @@ 
         $logger    = $this->logger;
         $formatter = $this->formatter;
 
-        return function (callable $handler) use ($logger, $formatter) {
+        return function(callable $handler) use ($logger, $formatter) {
 
-            return function ($request, array $options) use ($handler, $logger, $formatter) {
+            return function($request, array $options) use ($handler, $logger, $formatter) {
                 // generate id that will be used to supplement the log with information
                 $requestId = uniqid('eight_points_guzzle_');
 
@@ -48,7 +48,7 @@ 
 
                 return $handler($request, $options)->then(
 
-                    function ($response) use ($logger, $request, $formatter, $requestId) {
+                    function($response) use ($logger, $request, $formatter, $requestId) {
 
                         $message = $formatter->format($request, $response);
                         $context = compact('request', 'response', 'requestId');
@@ -58,7 +58,7 @@ 
                         return $response;
                     },
 
-                    function ($reason) use ($logger, $request, $formatter, $requestId) {
+                    function($reason) use ($logger, $request, $formatter, $requestId) {
 
                         $response = $reason instanceof RequestException ? $reason->getResponse() : null;
                         $message  = $formatter->format($request, $response, $reason);

src/DataCollector/HttpDataCollector.php

Spacing +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
     {
         return array_filter(
             $this->getMessages(),
-            function (LogMessage $message) use ($type) {
+            function(LogMessage $message) use ($type) {
                 return $message->getLevel() === $type;
             }
         );

src/Middleware/ProfileMiddleware.php

Spacing +4 added lines, -4 removed lines

@@ -29,9 +29,9 @@ 
     {
         $stopwatch = $this->stopwatch;
 
-        return function (callable $handler) use ($stopwatch) {
+        return function(callable $handler) use ($stopwatch) {
 
-            return function ($request, array $options) use ($handler, $stopwatch) {
+            return function($request, array $options) use ($handler, $stopwatch) {
                 $event = $stopwatch->start(
                     sprintf('%s %s', $request->getMethod(), $request->getUri()),
                     'eight_points_guzzle'
@@ -39,13 +39,13 @@ 
 
                 return $handler($request, $options)->then(
 
-                    function ($response) use ($event) {
+                    function($response) use ($event) {
                         $event->stop();
 
                         return $response;
                     },
 
-                    function ($reason) use ($event) {
+                    function($reason) use ($event) {
                         $event->stop();
 
                         return \GuzzleHttp\Promise\rejection_for($reason);

src/Middleware/RequestTimeMiddleware.php

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function __invoke(callable $handler) : \Closure
     {
-        return function (RequestInterface $request, array $options) use ($handler) {
+        return function(RequestInterface $request, array $options) use ($handler) {
             $options['on_stats'] = $this->getOnStatsCallback(
                 isset($options['on_stats']) ? $options['on_stats'] : null,
                 isset($options['request_id']) ? $options['request_id'] : null
@@ -55,15 +55,15 @@ 
      */
     protected function getOnStatsCallback(?callable $initialOnStats, ?string $requestId) : \Closure
     {
-        return function (TransferStats $stats) use ($initialOnStats, $requestId) {
+        return function(TransferStats $stats) use ($initialOnStats, $requestId) {
             if (is_callable($initialOnStats)) {
                 call_user_func($initialOnStats, $stats);
             }
 
-            $this->dataCollector->addTotalTime((float)$stats->getTransferTime());
+            $this->dataCollector->addTotalTime((float) $stats->getTransferTime());
 
             if (($this->logger instanceof Logger) && $requestId) {
-                $this->logger->addTransferTimeByRequestId($requestId, (float)$stats->getTransferTime());
+                $this->logger->addTransferTimeByRequestId($requestId, (float) $stats->getTransferTime());
             }
         };
     }

src/Log/LogResponse.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
                 $response->getBody()->rewind();
             }
         } else {
-            $this->setBody(EightPointsGuzzleBundle::class . ': [response body log disabled]');
+            $this->setBody(EightPointsGuzzleBundle::class.': [response body log disabled]');
         }
     }
 

src/DependencyInjection/EightPointsGuzzleExtension.php

Indentation +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
         } elseif ($logMode === false) {
             return Logger::LOG_MODE_NONE;
         } else {
-           return $logMode;
+            return $logMode;
         }
     }
 

Spacing +2 added lines, -2 removed lines

@@ -96,9 +96,9 @@
             // Allowed only for Symfony 4.2+
             if (method_exists($container, 'registerAliasForArgument')) {
                 if ('%eight_points_guzzle.http_client.class%' !== $options['class']) {
-                    $container->registerAliasForArgument($serviceName, $options['class'], $name . 'Client');
+                    $container->registerAliasForArgument($serviceName, $options['class'], $name.'Client');
                 }
-                $container->registerAliasForArgument($serviceName, ClientInterface::class, $name . 'Client');
+                $container->registerAliasForArgument($serviceName, ClientInterface::class, $name.'Client');
             }
         }
 

src/DependencyInjection/Configuration.php

Spacing +25 added lines, -25 removed lines

@@ -96,7 +96,7 @@ 
                     ->scalarNode('base_url')
                         ->defaultValue(null)
                         ->validate()
-                            ->ifTrue(function ($v) {
+                            ->ifTrue(function($v) {
                                 return !is_string($v);
                             })
                             ->thenInvalid('base_url can be: string')
@@ -106,13 +106,13 @@ 
                     ->integerNode('logging')
                         ->defaultValue(null)
                         ->beforeNormalization()
-                            ->always(function ($value): int {
+                            ->always(function($value): int {
                                 if ($value === 1 || $value === true) {
                                     return Logger::LOG_MODE_REQUEST_AND_RESPONSE;
                                 } elseif ($value === 0 || $value === false) {
                                     return Logger::LOG_MODE_NONE;
                                 } else {
-                                    return constant(Logger::class .'::LOG_MODE_' . strtoupper($value));
+                                    return constant(Logger::class.'::LOG_MODE_'.strtoupper($value));
                                 }
                             })
                         ->end()
@@ -120,7 +120,7 @@ 
                     ->scalarNode('handler')
                         ->defaultValue(null)
                         ->validate()
-                            ->ifTrue(function ($v) {
+                            ->ifTrue(function($v) {
                                 return $v !== null && (!is_string($v) || !class_exists($v));
                             })
                             ->thenInvalid('handler must be a valid FQCN for a loaded class')
@@ -128,7 +128,7 @@ 
                     ->end()
                     ->arrayNode('options')
                         ->validate()
-                            ->ifTrue(function ($options) {
+                            ->ifTrue(function($options) {
                                 return count($options['form_params']) && count($options['multipart']);
                             })
                             ->thenInvalid('You cannot use form_params and multipart at the same time.')
@@ -141,7 +141,7 @@ 
                             ->end()
                             ->variableNode('allow_redirects')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_array($v) && !is_bool($v);
                                     })
                                     ->thenInvalid('allow_redirects can be: bool or array')
@@ -149,7 +149,7 @@ 
                             ->end()
                             ->variableNode('auth')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_array($v) && !is_string($v);
                                     })
                                     ->thenInvalid('auth can be: string or array')
@@ -157,7 +157,7 @@ 
                             ->end()
                             ->variableNode('query')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_string($v) && !is_array($v);
                                     })
                                     ->thenInvalid('query can be: string or array')
@@ -166,16 +166,16 @@ 
                             ->arrayNode('curl')
                                 ->beforeNormalization()
                                     ->ifArray()
-                                        ->then(function (array $curlOptions) {
+                                        ->then(function(array $curlOptions) {
                                             $result = [];
 
                                             foreach ($curlOptions as $key => $value) {
-                                                $optionName = 'CURLOPT_' . strtoupper($key);
+                                                $optionName = 'CURLOPT_'.strtoupper($key);
 
                                                 if (!defined($optionName)) {
                                                     throw new InvalidConfigurationException(sprintf(
-                                                        'Invalid curl option in eight_points_guzzle: %s. ' .
-                                                        'Ex: use sslversion for CURLOPT_SSLVERSION option. ' . PHP_EOL .
+                                                        'Invalid curl option in eight_points_guzzle: %s. '.
+                                                        'Ex: use sslversion for CURLOPT_SSLVERSION option. '.PHP_EOL.
                                                         'See all available options: http://php.net/manual/en/function.curl-setopt.php',
                                                         $key
                                                     ));
@@ -192,7 +192,7 @@ 
                             ->end()
                             ->variableNode('cert')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_string($v) && (!is_array($v) || count($v) !== 2);
                                     })
                                     ->thenInvalid('cert can be: string or array with two entries (path and password)')
@@ -200,12 +200,12 @@ 
                             ->end()
                             ->scalarNode('connect_timeout')
                                 ->beforeNormalization()
-                                    ->always(function ($v) {
+                                    ->always(function($v) {
                                         return is_numeric($v) ? (float) $v : $v;
                                     })
                                 ->end()
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_float($v) && !(is_string($v) && strpos($v, 'env_') === 0);
                                     })
                                     ->thenInvalid('connect_timeout can be: float')
@@ -214,7 +214,7 @@ 
                             ->booleanNode('debug')->end()
                             ->variableNode('decode_content')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_string($v) && !is_bool($v);
                                     })
                                     ->thenInvalid('decode_content can be: bool or string (gzip, compress, deflate, etc...)')
@@ -230,7 +230,7 @@ 
                             ->end()
                             ->scalarNode('sink')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_string($v);
                                     })
                                     ->thenInvalid('sink can be: string')
@@ -239,7 +239,7 @@ 
                             ->booleanNode('http_errors')->end()
                             ->variableNode('expect')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_bool($v) && !is_int($v);
                                     })
                                     ->thenInvalid('expect can be: bool or int')
@@ -247,7 +247,7 @@ 
                             ->end()
                             ->variableNode('ssl_key')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_string($v) && (!is_array($v) || count($v) !== 2);
                                     })
                                     ->thenInvalid('ssl_key can be: string or array with two entries (path and password)')
@@ -257,12 +257,12 @@ 
                             ->booleanNode('synchronous')->end()
                             ->scalarNode('read_timeout')
                                 ->beforeNormalization()
-                                    ->always(function ($v) {
+                                    ->always(function($v) {
                                         return is_numeric($v) ? (float) $v : $v;
                                     })
                                 ->end()
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_float($v) && !(is_string($v) && strpos($v, 'env_') === 0);
                                     })
                                     ->thenInvalid('read_timeout can be: float')
@@ -270,12 +270,12 @@ 
                             ->end()
                             ->scalarNode('timeout')
                                 ->beforeNormalization()
-                                    ->always(function ($v) {
+                                    ->always(function($v) {
                                         return is_numeric($v) ? (float) $v : $v;
                                     })
                                 ->end()
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_float($v) && !(is_string($v) && strpos($v, 'env_') === 0);
                                     })
                                     ->thenInvalid('timeout can be: float')
@@ -283,7 +283,7 @@ 
                             ->end()
                             ->variableNode('verify')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_bool($v) && !is_string($v);
                                     })
                                     ->thenInvalid('verify can be: bool or string')
@@ -313,7 +313,7 @@ 
                             ->end()
                             ->scalarNode('version')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_string($v) && !is_float($v);
                                     })
                                     ->thenInvalid('version can be: string or float')