Apply code style fixes (#868)

Author: cleptric

src/Command/SentryTestCommand.php

@@ -25,7 +25,7 @@ public function __construct(?HubInterface $hub = null)
         parent::__construct();
 
         if (null === $hub) {
-            @trigger_error(sprintf('Not passing an instance of the "%s" interface as argument of the constructor is deprecated since version 4.12 and will not work since version 5.0.', HubInterface::class), \E_USER_DEPRECATED);
+            @trigger_error(\sprintf('Not passing an instance of the "%s" interface as argument of the constructor is deprecated since version 4.12 and will not work since version 5.0.', HubInterface::class), \E_USER_DEPRECATED);
         }
 
         $this->hub = $hub ?? SentrySdk::getCurrentHub();

src/DependencyInjection/Compiler/DbalTracingPass.php

@@ -45,8 +45,8 @@ public function process(ContainerBuilder $container): void
         }
 
         foreach ($connectionsToTrace as $connectionName) {
-            if (!\in_array(sprintf(self::CONNECTION_SERVICE_NAME_FORMAT, $connectionName), $connections, true)) {
-                throw new \InvalidArgumentException(sprintf('The Doctrine connection "%s" does not exists and cannot be instrumented.', $connectionName));
+            if (!\in_array(\sprintf(self::CONNECTION_SERVICE_NAME_FORMAT, $connectionName), $connections, true)) {
+                throw new \InvalidArgumentException(\sprintf('The Doctrine connection "%s" does not exists and cannot be instrumented.', $connectionName));
             }
 
             if (class_exists(Result::class)) {
@@ -65,7 +65,7 @@ private function configureConnectionForDoctrineDBALVersion3(ContainerBuilder $co
 
     private function configureConnectionForDoctrineDBALVersion2(ContainerBuilder $container, string $connectionName): void
     {
-        $connectionDefinition = $container->getDefinition(sprintf(self::CONNECTION_SERVICE_NAME_FORMAT, $connectionName));
+        $connectionDefinition = $container->getDefinition(\sprintf(self::CONNECTION_SERVICE_NAME_FORMAT, $connectionName));
         $connectionDefinition->setConfigurator([new Reference(ConnectionConfigurator::class), 'configure']);
     }
 

src/ErrorTypesParser.php

@@ -75,7 +75,7 @@ private static function convertErrorConstants(string $value): string
         }, $value);
 
         if (null === $output) {
-            throw new \InvalidArgumentException(sprintf('The "%s" value could not be parsed.', $value));
+            throw new \InvalidArgumentException(\sprintf('The "%s" value could not be parsed.', $value));
         }
 
         return $output;

src/EventListener/AbstractTracingRequestListener.php

@@ -67,7 +67,7 @@ protected function getRouteName(Request $request): string
             $route = $request->attributes->get('_controller');
 
             if (\is_array($route) && \is_callable($route, true)) {
-                $route = sprintf('%s::%s', \is_object($route[0]) ? get_debug_type($route[0]) : $route[0], $route[1]);
+                $route = \sprintf('%s::%s', \is_object($route[0]) ? get_debug_type($route[0]) : $route[0], $route[1]);
             }
         }
 

src/EventListener/TracingRequestListener.php

@@ -46,10 +46,10 @@ public function handleKernelRequestEvent(RequestEvent $event): void
 
         $routeName = $request->attributes->get('_route');
         if (null !== $routeName && \is_string($routeName)) {
-            $context->setName(sprintf('%s %s', $request->getMethod(), $routeName));
+            $context->setName(\sprintf('%s %s', $request->getMethod(), $routeName));
             $context->setSource(TransactionSource::route());
         } else {
-            $context->setName(sprintf('%s %s%s%s', $request->getMethod(), $request->getSchemeAndHttpHost(), $request->getBaseUrl(), $request->getPathInfo()));
+            $context->setName(\sprintf('%s %s%s%s', $request->getMethod(), $request->getSchemeAndHttpHost(), $request->getBaseUrl(), $request->getPathInfo()));
             $context->setSource(TransactionSource::url());
         }
 

src/EventListener/TracingSubRequestListener.php

@@ -36,7 +36,7 @@ public function handleKernelRequestEvent(RequestEvent $event): void
 
         $spanContext = new SpanContext();
         $spanContext->setOp('http.server');
-        $spanContext->setDescription(sprintf('%s %s%s%s', $request->getMethod(), $request->getSchemeAndHttpHost(), $request->getBaseUrl(), $request->getPathInfo()));
+        $spanContext->setDescription(\sprintf('%s %s%s%s', $request->getMethod(), $request->getSchemeAndHttpHost(), $request->getBaseUrl(), $request->getPathInfo()));
         $spanContext->setData([
             'http.request.method' => $request->getMethod(),
             'http.url' => $request->getUri(),

src/Tracing/Cache/TraceableCacheAdapterForV2.php

@@ -44,7 +44,7 @@ public function get(string $key, callable $callback, float $beta = null, array &
     {
         return $this->traceFunction('cache.get_item', function () use ($key, $callback, $beta, &$metadata) {
             if (!$this->decoratedAdapter instanceof CacheInterface) {
-                throw new \BadMethodCallException(sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
+                throw new \BadMethodCallException(\sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
             }
 
             return $this->decoratedAdapter->get($key, $callback, $beta, $metadata);

src/Tracing/Cache/TraceableCacheAdapterForV3.php

@@ -42,7 +42,7 @@ public function get(string $key, callable $callback, float $beta = null, array &
     {
         return $this->traceFunction('cache.get_item', function () use ($key, $callback, $beta, &$metadata) {
             if (!$this->decoratedAdapter instanceof CacheInterface) {
-                throw new \BadMethodCallException(sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
+                throw new \BadMethodCallException(\sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
             }
 
             return $this->decoratedAdapter->get($key, $callback, $beta, $metadata);

src/Tracing/Cache/TraceableCacheAdapterTrait.php

@@ -70,7 +70,7 @@ public function delete(string $key): bool
     {
         return $this->traceFunction('cache.delete_item', function () use ($key): bool {
             if (!$this->decoratedAdapter instanceof CacheInterface) {
-                throw new \BadMethodCallException(sprintf('The %s::delete() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
+                throw new \BadMethodCallException(\sprintf('The %s::delete() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
             }
 
             return $this->decoratedAdapter->delete($key);

src/Tracing/Cache/TraceableTagAwareCacheAdapterForV2.php

@@ -45,7 +45,7 @@ public function get(string $key, callable $callback, float $beta = null, array &
     {
         return $this->traceFunction('cache.get_item', function () use ($key, $callback, $beta, &$metadata) {
             if (!$this->decoratedAdapter instanceof CacheInterface) {
-                throw new \BadMethodCallException(sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
+                throw new \BadMethodCallException(\sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
             }
 
             return $this->decoratedAdapter->get($key, $callback, $beta, $metadata);

src/Tracing/Cache/TraceableTagAwareCacheAdapterForV3.php

@@ -43,7 +43,7 @@ public function get(string $key, callable $callback, float $beta = null, array &
     {
         return $this->traceFunction('cache.get_item', function () use ($key, $callback, $beta, &$metadata) {
             if (!$this->decoratedAdapter instanceof CacheInterface) {
-                throw new \BadMethodCallException(sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
+                throw new \BadMethodCallException(\sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "%s" interface.', self::class, CacheInterface::class));
             }
 
             return $this->decoratedAdapter->get($key, $callback, $beta, $metadata);

src/Tracing/Doctrine/DBAL/TracingDriverConnectionForV2V3.php

@@ -176,7 +176,7 @@ public function rollBack(): bool
     public function getNativeConnection()
     {
         if (!method_exists($this->decoratedConnection, 'getNativeConnection')) {
-            throw new \BadMethodCallException(sprintf('The connection "%s" does not support accessing the native connection.', \get_class($this->decoratedConnection)));
+            throw new \BadMethodCallException(\sprintf('The connection "%s" does not support accessing the native connection.', \get_class($this->decoratedConnection)));
         }
 
         return $this->decoratedConnection->getNativeConnection();
@@ -191,7 +191,7 @@ public function errorCode(): ?string
             return $this->decoratedConnection->errorCode();
         }
 
-        throw new \BadMethodCallException(sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
+        throw new \BadMethodCallException(\sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
     }
 
     /**
@@ -203,7 +203,7 @@ public function errorInfo(): array
             return $this->decoratedConnection->errorInfo();
         }
 
-        throw new \BadMethodCallException(sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
+        throw new \BadMethodCallException(\sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
     }
 
     public function getWrappedConnection(): DriverConnectionInterface

src/Tracing/Doctrine/DBAL/TracingDriverConnectionForV4.php

@@ -184,7 +184,7 @@ public function errorCode(): ?string
             return $this->decoratedConnection->errorCode();
         }
 
-        throw new \BadMethodCallException(sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
+        throw new \BadMethodCallException(\sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
     }
 
     /**
@@ -196,7 +196,7 @@ public function errorInfo(): array
             return $this->decoratedConnection->errorInfo();
         }
 
-        throw new \BadMethodCallException(sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
+        throw new \BadMethodCallException(\sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
     }
 
     public function getWrappedConnection(): DriverConnectionInterface

src/Tracing/Doctrine/DBAL/TracingDriverMiddleware.php

@@ -31,11 +31,11 @@ public function __construct($hubOrConnectionFactory)
         if ($hubOrConnectionFactory instanceof TracingDriverConnectionFactoryInterface) {
             $this->connectionFactory = $hubOrConnectionFactory;
         } elseif ($hubOrConnectionFactory instanceof HubInterface) {
-            @trigger_error(sprintf('Not passing an instance of the "%s" interface as argument of the constructor is deprecated since version 4.2 and will not work since version 5.0.', TracingDriverConnectionFactoryInterface::class), \E_USER_DEPRECATED);
+            @trigger_error(\sprintf('Not passing an instance of the "%s" interface as argument of the constructor is deprecated since version 4.2 and will not work since version 5.0.', TracingDriverConnectionFactoryInterface::class), \E_USER_DEPRECATED);
 
             $this->connectionFactory = new TracingDriverConnectionFactory($hubOrConnectionFactory);
         } else {
-            throw new \InvalidArgumentException(sprintf('The constructor requires either an instance of the "%s" interface or an instance of the "%s" interface.', HubInterface::class, TracingDriverConnectionFactoryInterface::class));
+            throw new \InvalidArgumentException(\sprintf('The constructor requires either an instance of the "%s" interface or an instance of the "%s" interface.', HubInterface::class, TracingDriverConnectionFactoryInterface::class));
         }
     }
 

src/Tracing/Doctrine/DBAL/TracingServerInfoAwareDriverConnection.php

@@ -109,7 +109,7 @@ public function getServerVersion(): string
         $wrappedConnection = $this->getWrappedConnection();
 
         if (!$wrappedConnection instanceof ServerInfoAwareConnection) {
-            throw new \BadMethodCallException(sprintf('The wrapped connection must be an instance of the "%s" interface.', ServerInfoAwareConnection::class));
+            throw new \BadMethodCallException(\sprintf('The wrapped connection must be an instance of the "%s" interface.', ServerInfoAwareConnection::class));
         }
 
         return $wrappedConnection->getServerVersion();
@@ -123,7 +123,7 @@ public function getServerVersion(): string
     public function getNativeConnection()
     {
         if (!method_exists($this->decoratedConnection, 'getNativeConnection')) {
-            throw new \BadMethodCallException(sprintf('The connection "%s" does not support accessing the native connection.', \get_class($this->decoratedConnection)));
+            throw new \BadMethodCallException(\sprintf('The connection "%s" does not support accessing the native connection.', \get_class($this->decoratedConnection)));
         }
 
         return $this->decoratedConnection->getNativeConnection();
@@ -137,11 +137,11 @@ public function requiresQueryForServerVersion(): bool
         $wrappedConnection = $this->getWrappedConnection();
 
         if (!$wrappedConnection instanceof ServerInfoAwareConnection) {
-            throw new \BadMethodCallException(sprintf('The wrapped connection must be an instance of the "%s" interface.', ServerInfoAwareConnection::class));
+            throw new \BadMethodCallException(\sprintf('The wrapped connection must be an instance of the "%s" interface.', ServerInfoAwareConnection::class));
         }
 
         if (!method_exists($wrappedConnection, 'requiresQueryForServerVersion')) {
-            throw new \BadMethodCallException(sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
+            throw new \BadMethodCallException(\sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
         }
 
         return $wrappedConnection->requiresQueryForServerVersion();
@@ -156,7 +156,7 @@ public function errorCode(): ?string
             return $this->decoratedConnection->errorCode();
         }
 
-        throw new \BadMethodCallException(sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
+        throw new \BadMethodCallException(\sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
     }
 
     /**
@@ -168,7 +168,7 @@ public function errorInfo(): array
             return $this->decoratedConnection->errorInfo();
         }
 
-        throw new \BadMethodCallException(sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
+        throw new \BadMethodCallException(\sprintf('The %s() method is not supported on Doctrine DBAL 3.0.', __METHOD__));
     }
 
     public function getWrappedConnection(): Connection

src/Tracing/HttpClient/AbstractTraceableHttpClient.php

@@ -111,7 +111,7 @@ public function stream($responses, float $timeout = null): ResponseStreamInterfa
         if ($responses instanceof AbstractTraceableResponse) {
             $responses = [$responses];
         } elseif (!is_iterable($responses)) {
-            throw new \TypeError(sprintf('"%s()" expects parameter 1 to be an iterable of TraceableResponse objects, "%s" given.', __METHOD__, get_debug_type($responses)));
+            throw new \TypeError(\sprintf('"%s()" expects parameter 1 to be an iterable of TraceableResponse objects, "%s" given.', __METHOD__, get_debug_type($responses)));
         }
 
         return new ResponseStream(AbstractTraceableResponse::stream($this->client, $responses, $timeout));

src/Tracing/HttpClient/AbstractTraceableResponse.php

@@ -107,7 +107,7 @@ public static function stream(HttpClientInterface $client, iterable $responses,
 
         foreach ($responses as $response) {
             if (!$response instanceof self) {
-                throw new \TypeError(sprintf('"%s::stream()" expects parameter 1 to be an iterable of TraceableResponse objects, "%s" given.', TraceableHttpClient::class, get_debug_type($response)));
+                throw new \TypeError(\sprintf('"%s::stream()" expects parameter 1 to be an iterable of TraceableResponse objects, "%s" given.', TraceableHttpClient::class, get_debug_type($response)));
             }
 
             $traceableMap[$response->response] = $response;

src/Tracing/Twig/TwigTracingExtension.php

@@ -95,7 +95,7 @@ private function getSpanDescription(Profile $profile): string
                 return $profile->getTemplate();
 
             default:
-                return sprintf('%s::%s(%s)', $profile->getTemplate(), $profile->getType(), $profile->getName());
+                return \sprintf('%s::%s(%s)', $profile->getTemplate(), $profile->getType(), $profile->getName());
         }
     }
 }

src/Twig/SentryExtension.php

@@ -38,22 +38,22 @@ public function getFunctions(): array
      */
     public function getTraceMeta(): string
     {
-        return sprintf('<meta name="sentry-trace" content="%s" />', getTraceparent());
+        return \sprintf('<meta name="sentry-trace" content="%s" />', getTraceparent());
     }
 
     /**
      * Returns an HTML meta tag named `traceparent`.
      */
     public function getW3CTraceMeta(): string
     {
-        return sprintf('<meta name="traceparent" content="%s" />', getW3CTraceparent());
+        return \sprintf('<meta name="traceparent" content="%s" />', getW3CTraceparent());
     }
 
     /**
      * Returns an HTML meta tag named `baggage`.
      */
     public function getBaggageMeta(): string
     {
-        return sprintf('<meta name="baggage" content="%s" />', getBaggage());
+        return \sprintf('<meta name="baggage" content="%s" />', getBaggage());
     }
 }

tests/Tracing/Cache/AbstractTraceableCacheAdapterTest.php

@@ -159,7 +159,7 @@ public function testGetThrowsExceptionIfDecoratedAdapterDoesNotImplementTheCache
         $adapter = $this->createCacheAdapter($this->createMock(static::getAdapterClassFqcn()));
 
         $this->expectException(\BadMethodCallException::class);
-        $this->expectExceptionMessage(sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "Symfony\\Contracts\\Cache\\CacheInterface" interface.', \get_class($adapter)));
+        $this->expectExceptionMessage(\sprintf('The %s::get() method is not supported because the decorated adapter does not implement the "Symfony\\Contracts\\Cache\\CacheInterface" interface.', \get_class($adapter)));
 
         $adapter->get('foo', static function () {});
     }
@@ -197,7 +197,7 @@ public function testDeleteThrowsExceptionIfDecoratedAdapterDoesNotImplementTheCa
         $adapter = $this->createCacheAdapter($this->createMock(static::getAdapterClassFqcn()));
 
         $this->expectException(\BadMethodCallException::class);
-        $this->expectExceptionMessage(sprintf('The %s::delete() method is not supported because the decorated adapter does not implement the "Symfony\\Contracts\\Cache\\CacheInterface" interface.', \get_class($adapter)));
+        $this->expectExceptionMessage(\sprintf('The %s::delete() method is not supported because the decorated adapter does not implement the "Symfony\\Contracts\\Cache\\CacheInterface" interface.', \get_class($adapter)));
 
         $adapter->delete('foo');
     }

tests/Tracing/HttpClient/TraceableResponseTest.php

@@ -55,7 +55,7 @@ public function testInstanceCannotBeUnserialized(): void
         $this->expectException(\BadMethodCallException::class);
         $this->expectExceptionMessage('Unserializing instances of this class is forbidden.');
 
-        unserialize(sprintf('O:%u:"%s":0:{}', \strlen(TraceableResponse::class), TraceableResponse::class));
+        unserialize(\sprintf('O:%u:"%s":0:{}', \strlen(TraceableResponse::class), TraceableResponse::class));
     }
 
     public function testDestructor(): void