Prefix all sprintf() calls

Author: derrabus

BinaryFileResponse.php

@@ -256,13 +256,13 @@ public function prepare(Request $request): static
  $end = min($end, $fileSize - 1);
  if ($start < 0 || $start > $end) {
  $this->setStatusCode(416);
- $this->headers->set('Content-Range', sprintf('bytes */%s', $fileSize));
+ $this->headers->set('Content-Range', \sprintf('bytes */%s', $fileSize));
  } elseif ($end - $start < $fileSize - 1) {
  $this->maxlen = $end < $fileSize ? $end - $start + 1 : -1;
  $this->offset = $start;
 
  $this->setStatusCode(206);
- $this->headers->set('Content-Range', sprintf('bytes %s-%s/%s', $start, $end, $fileSize));
+ $this->headers->set('Content-Range', \sprintf('bytes %s-%s/%s', $start, $end, $fileSize));
  $this->headers->set('Content-Length', $end - $start + 1);
  }
  }

Cookie.php

@@ -101,7 +101,7 @@ public function __construct(
  ) {
  // from PHP source code
  if ($raw && false !== strpbrk($name, self::RESERVED_CHARS_LIST)) {
- throw new \InvalidArgumentException(sprintf('The cookie name "%s" contains invalid characters.', $name));
+ throw new \InvalidArgumentException(\sprintf('The cookie name "%s" contains invalid characters.', $name));
  }
 
  if (!$name) {
@@ -204,7 +204,7 @@ public function withHttpOnly(bool $httpOnly = true): static
  public function withRaw(bool $raw = true): static
  {
  if ($raw && false !== strpbrk($this->name, self::RESERVED_CHARS_LIST)) {
- throw new \InvalidArgumentException(sprintf('The cookie name "%s" contains invalid characters.', $this->name));
+ throw new \InvalidArgumentException(\sprintf('The cookie name "%s" contains invalid characters.', $this->name));
  }
 
  $cookie = clone $this;

File/Exception/AccessDeniedException.php

@@ -20,6 +20,6 @@ class AccessDeniedException extends FileException
 {
  public function __construct(string $path)
  {
- parent::__construct(sprintf('The file %s could not be accessed', $path));
+ parent::__construct(\sprintf('The file %s could not be accessed', $path));
  }
 }

File/Exception/FileNotFoundException.php

@@ -20,6 +20,6 @@ class FileNotFoundException extends FileException
 {
  public function __construct(string $path)
  {
- parent::__construct(sprintf('The file "%s" does not exist', $path));
+ parent::__construct(\sprintf('The file "%s" does not exist', $path));
  }
 }

File/Exception/UnexpectedTypeException.php

@@ -15,6 +15,6 @@ class UnexpectedTypeException extends FileException
 {
  public function __construct(mixed $value, string $expectedType)
  {
- parent::__construct(sprintf('Expected argument of type %s, %s given', $expectedType, get_debug_type($value)));
+ parent::__construct(\sprintf('Expected argument of type %s, %s given', $expectedType, get_debug_type($value)));
  }
 }

File/File.php

@@ -93,7 +93,7 @@ public function move(string $directory, ?string $name = null): self
  restore_error_handler();
  }
  if (!$renamed) {
- throw new FileException(sprintf('Could not move the file "%s" to "%s" (%s).', $this->getPathname(), $target, strip_tags($error)));
+ throw new FileException(\sprintf('Could not move the file "%s" to "%s" (%s).', $this->getPathname(), $target, strip_tags($error)));
  }
 
  @chmod($target, 0666 & ~umask());
@@ -106,7 +106,7 @@ public function getContent(): string
  $content = file_get_contents($this->getPathname());
 
  if (false === $content) {
- throw new FileException(sprintf('Could not get the content of the file "%s".', $this->getPathname()));
+ throw new FileException(\sprintf('Could not get the content of the file "%s".', $this->getPathname()));
  }
 
  return $content;
@@ -116,10 +116,10 @@ protected function getTargetFile(string $directory, ?string $name = null): self
  {
  if (!is_dir($directory)) {
  if (false === @mkdir($directory, 0777, true) && !is_dir($directory)) {
- throw new FileException(sprintf('Unable to create the "%s" directory.', $directory));
+ throw new FileException(\sprintf('Unable to create the "%s" directory.', $directory));
  }
  } elseif (!is_writable($directory)) {
- throw new FileException(sprintf('Unable to write in the "%s" directory.', $directory));
+ throw new FileException(\sprintf('Unable to write in the "%s" directory.', $directory));
  }
 
  $target = rtrim($directory, '/\\').\DIRECTORY_SEPARATOR.(null === $name ? $this->getBasename() : $this->getName($name));

File/UploadedFile.php

@@ -194,7 +194,7 @@ public function move(string $directory, ?string $name = null): File
  restore_error_handler();
  }
  if (!$moved) {
- throw new FileException(sprintf('Could not move the file "%s" to "%s" (%s).', $this->getPathname(), $target, strip_tags($error)));
+ throw new FileException(\sprintf('Could not move the file "%s" to "%s" (%s).', $this->getPathname(), $target, strip_tags($error)));
  }
 
  @chmod($target, 0666 & ~umask());
@@ -284,6 +284,6 @@ public function getErrorMessage(): string
  $maxFilesize = \UPLOAD_ERR_INI_SIZE === $errorCode ? self::getMaxFilesize() / 1024 : 0;
  $message = $errors[$errorCode] ?? 'The file "%s" was not uploaded due to an unknown error.';
 
- return sprintf($message, $this->getClientOriginalName(), $maxFilesize);
+ return \sprintf($message, $this->getClientOriginalName(), $maxFilesize);
  }
 }

HeaderBag.php

@@ -51,7 +51,7 @@ public function __toString(): string
  foreach ($headers as $name => $values) {
  $name = ucwords($name, '-');
  foreach ($values as $value) {
- $content .= sprintf("%-{$max}s %s\r\n", $name.':', $value);
+ $content .= \sprintf("%-{$max}s %s\r\n", $name.':', $value);
  }
  }
 
@@ -194,7 +194,7 @@ public function getDate(string $key, ?\DateTimeInterface $default = null): ?\Dat
  }
 
  if (false === $date = \DateTimeImmutable::createFromFormat(\DATE_RFC2822, $value)) {
- throw new \RuntimeException(sprintf('The "%s" HTTP header is not parseable (%s).', $key, $value));
+ throw new \RuntimeException(\sprintf('The "%s" HTTP header is not parseable (%s).', $key, $value));
  }
 
  return $date;

HeaderUtils.php

@@ -165,7 +165,7 @@ public static function unquote(string $s): string
  public static function makeDisposition(string $disposition, string $filename, string $filenameFallback = ''): string
  {
  if (!\in_array($disposition, [self::DISPOSITION_ATTACHMENT, self::DISPOSITION_INLINE])) {
- throw new \InvalidArgumentException(sprintf('The disposition must be either "%s" or "%s".', self::DISPOSITION_ATTACHMENT, self::DISPOSITION_INLINE));
+ throw new \InvalidArgumentException(\sprintf('The disposition must be either "%s" or "%s".', self::DISPOSITION_ATTACHMENT, self::DISPOSITION_INLINE));
  }
 
  if ('' === $filenameFallback) {

InputBag.php

@@ -29,13 +29,13 @@ final class InputBag extends ParameterBag
  public function get(string $key, mixed $default = null): string|int|float|bool|null
  {
  if (null !== $default && !\is_scalar($default) && !$default instanceof \Stringable) {
- throw new \InvalidArgumentException(sprintf('Expected a scalar value as a 2nd argument to "%s()", "%s" given.', __METHOD__, get_debug_type($default)));
+ throw new \InvalidArgumentException(\sprintf('Expected a scalar value as a 2nd argument to "%s()", "%s" given.', __METHOD__, get_debug_type($default)));
  }
 
  $value = parent::get($key, $this);
 
  if (null !== $value && $this !== $value && !\is_scalar($value) && !$value instanceof \Stringable) {
- throw new BadRequestException(sprintf('Input value "%s" contains a non-scalar value.', $key));
+ throw new BadRequestException(\sprintf('Input value "%s" contains a non-scalar value.', $key));
  }
 
  return $this === $value ? $default : $value;
@@ -68,7 +68,7 @@ public function add(array $inputs = []): void
  public function set(string $key, mixed $value): void
  {
  if (null !== $value && !\is_scalar($value) && !\is_array($value) && !$value instanceof \Stringable) {
- throw new \InvalidArgumentException(sprintf('Expected a scalar, or an array as a 2nd argument to "%s()", "%s" given.', __METHOD__, get_debug_type($value)));
+ throw new \InvalidArgumentException(\sprintf('Expected a scalar, or an array as a 2nd argument to "%s()", "%s" given.', __METHOD__, get_debug_type($value)));
  }
 
  $this->parameters[$key] = $value;
@@ -114,11 +114,11 @@ public function filter(string $key, mixed $default = null, int $filter = \FILTER
  }
 
  if (\is_array($value) && !(($options['flags'] ?? 0) & (\FILTER_REQUIRE_ARRAY | \FILTER_FORCE_ARRAY))) {
- throw new BadRequestException(sprintf('Input value "%s" contains an array, but "FILTER_REQUIRE_ARRAY" or "FILTER_FORCE_ARRAY" flags were not set.', $key));
+ throw new BadRequestException(\sprintf('Input value "%s" contains an array, but "FILTER_REQUIRE_ARRAY" or "FILTER_FORCE_ARRAY" flags were not set.', $key));
  }
 
  if ((\FILTER_CALLBACK & $filter) && !(($options['options'] ?? null) instanceof \Closure)) {
- throw new \InvalidArgumentException(sprintf('A Closure must be passed to "%s()" when FILTER_CALLBACK is used, "%s" given.', __METHOD__, get_debug_type($options['options'] ?? null)));
+ throw new \InvalidArgumentException(\sprintf('A Closure must be passed to "%s()" when FILTER_CALLBACK is used, "%s" given.', __METHOD__, get_debug_type($options['options'] ?? null)));
  }
 
  $options['flags'] ??= 0;
@@ -131,6 +131,6 @@ public function filter(string $key, mixed $default = null, int $filter = \FILTER
  return $value;
  }
 
- throw new BadRequestException(sprintf('Input value "%s" is invalid and flag "FILTER_NULL_ON_FAILURE" was not set.', $key));
+ throw new BadRequestException(\sprintf('Input value "%s" is invalid and flag "FILTER_NULL_ON_FAILURE" was not set.', $key));
  }
 }