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/DependencyInjection/Configuration.php

Spacing +24 added lines, -24 removed lines

@@ -101,7 +101,7 @@ 
                     ->scalarNode('base_url')
                         ->defaultValue(null)
                         ->validate()
-                            ->ifTrue(function ($v) {
+                            ->ifTrue(function($v) {
                                 return !is_string($v);
                             })
                             ->thenInvalid('base_url can be: string')
@@ -117,7 +117,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')
@@ -125,7 +125,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.')
@@ -138,7 +138,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')
@@ -146,7 +146,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')
@@ -154,7 +154,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')
@@ -163,16 +163,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
                                                     ));
@@ -189,7 +189,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)')
@@ -197,12 +197,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')
@@ -211,7 +211,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...)')
@@ -227,7 +227,7 @@ 
                             ->end()
                             ->scalarNode('sink')
                                 ->validate()
-                                    ->ifTrue(function ($v) {
+                                    ->ifTrue(function($v) {
                                         return !is_string($v);
                                     })
                                     ->thenInvalid('sink can be: string')
@@ -236,7 +236,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')
@@ -244,7 +244,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)')
@@ -254,12 +254,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')
@@ -267,12 +267,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')
@@ -280,7 +280,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')
@@ -310,7 +310,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')
@@ -339,7 +339,7 @@ 
         } elseif ($v === 0 || $v === false) {
             return Logger::LOG_MODE_NONE;
         } else {
-            return constant(Logger::class .'::LOG_MODE_' . strtoupper($v));
+            return constant(Logger::class.'::LOG_MODE_'.strtoupper($v));
         }
     }
 }

src/DependencyInjection/EightPointsGuzzleExtension.php

Spacing +4 added lines, -4 removed lines

@@ -94,11 +94,11 @@ 
             $container->setDefinition($serviceName, $client);
         }
 
-        $clientsWithLogging = array_filter($config['clients'], function ($options) use ($logging) {
+        $clientsWithLogging = array_filter($config['clients'], function($options) use ($logging) {
             return $options['logging'] !== false && $logging !== false;
         });
 
-        if (count($clientsWithLogging)>0) {
+        if (count($clientsWithLogging) > 0) {
             $this->defineTwigDebugExtension($container);
             $this->defineDataCollector($container, $config['slow_response_time'] / 1000);
             $this->defineFormatter($container);
@@ -126,7 +126,7 @@ 
         $container->setDefinition($eventServiceName, $eventService);
 
         // Create the event Dispatch Middleware
-        $eventExpression  = new Expression(sprintf("service('%s').dispatchEvent()", $eventServiceName));
+        $eventExpression = new Expression(sprintf("service('%s').dispatchEvent()", $eventServiceName));
 
         $handler = new Definition(HandlerStack::class);
         $handler->setFactory([HandlerStack::class, 'create']);
@@ -216,7 +216,7 @@ 
     {
         $dataCollectorDefinition = new Definition('%eight_points_guzzle.data_collector.class%');
 
-        $dataCollectorDefinition->addArgument(array_map(function ($loggerId) : Reference {
+        $dataCollectorDefinition->addArgument(array_map(function($loggerId) : Reference {
             return new Reference($loggerId);
         }, array_keys($container->findTaggedServiceIds('eight_points_guzzle.logger'))));
 

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]');
         }
     }