Prefix all sprintf() calls

derrabus · derrabus · commit 1b381a95755c · 2024-06-20T17:52:34.000+02:00
diff --git a/Transport/Connection.php b/Transport/Connection.php
@@ -114,13 +114,13 @@ public static function fromDsn(#[\SensitiveParameter] string $dsn, array $option
         // check for extra keys in options
         $optionsExtraKeys = array_diff(array_keys($options), array_keys(self::DEFAULT_OPTIONS));
         if (0 < \count($optionsExtraKeys)) {
-            throw new InvalidArgumentException(sprintf('Unknown option found: [%s]. Allowed options are [%s].', implode(', ', $optionsExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));
+            throw new InvalidArgumentException(\sprintf('Unknown option found: [%s]. Allowed options are [%s].', implode(', ', $optionsExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));
         }
 
         // check for extra keys in options
         $queryExtraKeys = array_diff(array_keys($query), array_keys(self::DEFAULT_OPTIONS));
         if (0 < \count($queryExtraKeys)) {
-            throw new InvalidArgumentException(sprintf('Unknown option found in DSN: [%s]. Allowed options are [%s].', implode(', ', $queryExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));
+            throw new InvalidArgumentException(\sprintf('Unknown option found in DSN: [%s]. Allowed options are [%s].', implode(', ', $queryExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));
         }
 
         $options = $query + $options + self::DEFAULT_OPTIONS;
@@ -147,7 +147,7 @@ public static function fromDsn(#[\SensitiveParameter] string $dsn, array $option
         unset($query['region']);
 
         if ('default' !== ($params['host'] ?? 'default')) {
-            $clientConfiguration['endpoint'] = sprintf('%s://%s%s', ($query['sslmode'] ?? null) === 'disable' ? 'http' : 'https', $params['host'], ($params['port'] ?? null) ? ':'.$params['port'] : '');
+            $clientConfiguration['endpoint'] = \sprintf('%s://%s%s', ($query['sslmode'] ?? null) === 'disable' ? 'http' : 'https', $params['host'], ($params['port'] ?? null) ? ':'.$params['port'] : '');
             if (preg_match(';^sqs\.([^\.]++)\.amazonaws\.com$;', $params['host'], $matches)) {
                 $clientConfiguration['region'] = $matches[1];
             }
@@ -274,7 +274,7 @@ public function setup(): void
         }
 
         if (null !== $this->configuration['account']) {
-            throw new InvalidArgumentException(sprintf('The Amazon SQS queue "%s" does not exist (or you don\'t have permissions on it), and can\'t be created when an account is provided.', $this->configuration['queue_name']));
+            throw new InvalidArgumentException(\sprintf('The Amazon SQS queue "%s" does not exist (or you don\'t have permissions on it), and can\'t be created when an account is provided.', $this->configuration['queue_name']));
         }
 
         $parameters = ['QueueName' => $this->configuration['queue_name']];
@@ -290,7 +290,7 @@ public function setup(): void
         // Blocking call to wait for the queue to be created
         $exists->wait();
         if (!$exists->isSuccess()) {
-            throw new TransportException(sprintf('Failed to create the Amazon SQS queue "%s".', $this->configuration['queue_name']));
+            throw new TransportException(\sprintf('Failed to create the Amazon SQS queue "%s".', $this->configuration['queue_name']));
         }
         $this->queueUrl = null;
     }

Original file line number	Diff line number	Diff line change
`@@ -114,13 +114,13 @@ public static function fromDsn(#[\SensitiveParameter] string $dsn, array $option`
`114`	`114`	`// check for extra keys in options`
`115`	`115`	`$optionsExtraKeys = array_diff(array_keys($options), array_keys(self::DEFAULT_OPTIONS));`
`116`	`116`	`if (0 < \count($optionsExtraKeys)) {`
`117`		`- throw new InvalidArgumentException(sprintf('Unknown option found: [%s]. Allowed options are [%s].', implode(', ', $optionsExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));`
	`117`	`+ throw new InvalidArgumentException(\sprintf('Unknown option found: [%s]. Allowed options are [%s].', implode(', ', $optionsExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));`
`118`	`118`	`}`
`119`	`119`
`120`	`120`	`// check for extra keys in options`
`121`	`121`	`$queryExtraKeys = array_diff(array_keys($query), array_keys(self::DEFAULT_OPTIONS));`
`122`	`122`	`if (0 < \count($queryExtraKeys)) {`
`123`		`- throw new InvalidArgumentException(sprintf('Unknown option found in DSN: [%s]. Allowed options are [%s].', implode(', ', $queryExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));`
	`123`	`+ throw new InvalidArgumentException(\sprintf('Unknown option found in DSN: [%s]. Allowed options are [%s].', implode(', ', $queryExtraKeys), implode(', ', array_keys(self::DEFAULT_OPTIONS))));`
`124`	`124`	`}`
`125`	`125`
`126`	`126`	`$options = $query + $options + self::DEFAULT_OPTIONS;`
`@@ -147,7 +147,7 @@ public static function fromDsn(#[\SensitiveParameter] string $dsn, array $option`
`147`	`147`	`unset($query['region']);`
`148`	`148`
`149`	`149`	`if ('default' !== ($params['host'] ?? 'default')) {`
`150`		`- $clientConfiguration['endpoint'] = sprintf('%s://%s%s', ($query['sslmode'] ?? null) === 'disable' ? 'http' : 'https', $params['host'], ($params['port'] ?? null) ? ':'.$params['port'] : '');`
	`150`	`+ $clientConfiguration['endpoint'] = \sprintf('%s://%s%s', ($query['sslmode'] ?? null) === 'disable' ? 'http' : 'https', $params['host'], ($params['port'] ?? null) ? ':'.$params['port'] : '');`
`151`	`151`	`if (preg_match(';^sqs\.([^\.]++)\.amazonaws\.com$;', $params['host'], $matches)) {`
`152`	`152`	`$clientConfiguration['region'] = $matches[1];`
`153`	`153`	`}`
`@@ -274,7 +274,7 @@ public function setup(): void`
`274`	`274`	`}`
`275`	`275`
`276`	`276`	`if (null !== $this->configuration['account']) {`
`277`		`- throw new InvalidArgumentException(sprintf('The Amazon SQS queue "%s" does not exist (or you don\'t have permissions on it), and can\'t be created when an account is provided.', $this->configuration['queue_name']));`
	`277`	`+ throw new InvalidArgumentException(\sprintf('The Amazon SQS queue "%s" does not exist (or you don\'t have permissions on it), and can\'t be created when an account is provided.', $this->configuration['queue_name']));`
`278`	`278`	`}`
`279`	`279`
`280`	`280`	`$parameters = ['QueueName' => $this->configuration['queue_name']];`
`@@ -290,7 +290,7 @@ public function setup(): void`
`290`	`290`	`// Blocking call to wait for the queue to be created`
`291`	`291`	`$exists->wait();`
`292`	`292`	`if (!$exists->isSuccess()) {`
`293`		`- throw new TransportException(sprintf('Failed to create the Amazon SQS queue "%s".', $this->configuration['queue_name']));`
	`293`	`+ throw new TransportException(\sprintf('Failed to create the Amazon SQS queue "%s".', $this->configuration['queue_name']));`
`294`	`294`	`}`
`295`	`295`	`$this->queueUrl = null;`
`296`	`296`	`}`