Prefix all sprintf() calls

Author: derrabus

Adapter/AbstractConnection.php

@@ -42,7 +42,7 @@ protected function configureOptions(OptionsResolver $resolver): void
 
         $resolver->setDefault('port', fn (Options $options) => 'ssl' === $options['encryption'] ? 636 : 389);
 
-        $resolver->setDefault('connection_string', fn (Options $options) => sprintf('ldap%s://%s:%s', 'ssl' === $options['encryption'] ? 's' : '', $options['host'], $options['port']));
+        $resolver->setDefault('connection_string', fn (Options $options) => \sprintf('ldap%s://%s:%s', 'ssl' === $options['encryption'] ? 's' : '', $options['host'], $options['port']));
 
         $resolver->setAllowedTypes('host', 'string');
         $resolver->setAllowedTypes('port', 'numeric');

Adapter/ExtLdap/Connection.php

@@ -96,14 +96,14 @@ public function getResource(): ?LDAPConnection
     public function setOption(string $name, array|string|int|bool $value): void
     {
         if (!@ldap_set_option($this->connection, ConnectionOptions::getOption($name), $value)) {
-            throw new LdapException(sprintf('Could not set value "%s" for option "%s".', $value, $name));
+            throw new LdapException(\sprintf('Could not set value "%s" for option "%s".', $value, $name));
         }
     }
 
     public function getOption(string $name): array|string|int|null
     {
         if (!@ldap_get_option($this->connection, ConnectionOptions::getOption($name), $ret)) {
-            throw new LdapException(sprintf('Could not retrieve value for option "%s".', $name));
+            throw new LdapException(\sprintf('Could not retrieve value for option "%s".', $name));
         }
 
         return $ret;

Adapter/ExtLdap/ConnectionOptions.php

@@ -65,7 +65,7 @@ final class ConnectionOptions
 
     public static function getOptionName(string $name): string
     {
-        return sprintf('%s::%s', self::class, strtoupper($name));
+        return \sprintf('%s::%s', self::class, strtoupper($name));
     }
 
     /**
@@ -80,7 +80,7 @@ public static function getOption(string $name): int
         $constantName = self::getOptionName($name);
 
         if (!\defined($constantName)) {
-            throw new LdapException(sprintf('Unknown option "%s".', $name));
+            throw new LdapException(\sprintf('Unknown option "%s".', $name));
         }
 
         return \constant($constantName);

Adapter/ExtLdap/EntryManager.php

@@ -37,7 +37,7 @@ public function add(Entry $entry): static
         $con = $this->getConnectionResource();
 
         if (!@ldap_add($con, $entry->getDn(), $entry->getAttributes())) {
-            throw new LdapException(sprintf('Could not add entry "%s": ', $entry->getDn()).ldap_error($con), ldap_errno($con));
+            throw new LdapException(\sprintf('Could not add entry "%s": ', $entry->getDn()).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -51,7 +51,7 @@ public function update(Entry $entry): static
         $con = $this->getConnectionResource();
 
         if (!@ldap_modify($con, $entry->getDn(), $entry->getAttributes())) {
-            throw new LdapException(sprintf('Could not update entry "%s": ', $entry->getDn()).ldap_error($con), ldap_errno($con));
+            throw new LdapException(\sprintf('Could not update entry "%s": ', $entry->getDn()).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -65,7 +65,7 @@ public function remove(Entry $entry): static
         $con = $this->getConnectionResource();
 
         if (!@ldap_delete($con, $entry->getDn())) {
-            throw new LdapException(sprintf('Could not remove entry "%s": ', $entry->getDn()).ldap_error($con), ldap_errno($con));
+            throw new LdapException(\sprintf('Could not remove entry "%s": ', $entry->getDn()).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -84,7 +84,7 @@ public function addAttributeValues(Entry $entry, string $attribute, array $value
         $con = $this->getConnectionResource();
 
         if (!@ldap_mod_add($con, $entry->getDn(), [$attribute => $values])) {
-            throw new LdapException(sprintf('Could not add values to entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con), ldap_errno($con));
+            throw new LdapException(\sprintf('Could not add values to entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -103,7 +103,7 @@ public function removeAttributeValues(Entry $entry, string $attribute, array $va
         $con = $this->getConnectionResource();
 
         if (!@ldap_mod_del($con, $entry->getDn(), [$attribute => $values])) {
-            throw new LdapException(sprintf('Could not remove values from entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con), ldap_errno($con));
+            throw new LdapException(\sprintf('Could not remove values from entry "%s", attribute "%s": ', $entry->getDn(), $attribute).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -117,7 +117,7 @@ public function rename(Entry $entry, string $newRdn, bool $removeOldRdn = true):
         $con = $this->getConnectionResource();
 
         if (!@ldap_rename($con, $entry->getDn(), $newRdn, '', $removeOldRdn)) {
-            throw new LdapException(sprintf('Could not rename entry "%s" to "%s": ', $entry->getDn(), $newRdn).ldap_error($con), ldap_errno($con));
+            throw new LdapException(\sprintf('Could not rename entry "%s" to "%s": ', $entry->getDn(), $newRdn).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -137,7 +137,7 @@ public function move(Entry $entry, string $newParent): static
         $con = $this->getConnectionResource();
         // deleteOldRdn does not matter here, since the Rdn will not be changing in the move.
         if (!@ldap_rename($con, $entry->getDn(), $rdn, $newParent, true)) {
-            throw new LdapException(sprintf('Could not move entry "%s" to "%s": ', $entry->getDn(), $newParent).ldap_error($con), ldap_errno($con));
+            throw new LdapException(\sprintf('Could not move entry "%s" to "%s": ', $entry->getDn(), $newParent).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -172,7 +172,7 @@ public function applyOperations(string $dn, iterable $operations): static
 
         $con = $this->getConnectionResource();
         if (!@ldap_modify_batch($con, $dn, $operationsMapped)) {
-            throw new UpdateOperationException(sprintf('Error executing UpdateOperation on "%s": ', $dn).ldap_error($con), ldap_errno($con));
+            throw new UpdateOperationException(\sprintf('Error executing UpdateOperation on "%s": ', $dn).ldap_error($con), ldap_errno($con));
         }
 
         return $this;
@@ -181,7 +181,7 @@ public function applyOperations(string $dn, iterable $operations): static
     private function parseRdnFromEntry(Entry $entry): string
     {
         if (!preg_match('/(^[^,\\\\]*(?:\\\\.[^,\\\\]*)*),/', $entry->getDn(), $matches)) {
-            throw new LdapException(sprintf('Entry "%s" malformed, could not parse RDN.', $entry->getDn()));
+            throw new LdapException(\sprintf('Entry "%s" malformed, could not parse RDN.', $entry->getDn()));
         }
 
         return $matches[1];

Adapter/ExtLdap/Query.php

@@ -73,7 +73,7 @@ public function execute(): CollectionInterface
                 static::SCOPE_BASE => 'ldap_read',
                 static::SCOPE_ONE => 'ldap_list',
                 static::SCOPE_SUB => 'ldap_search',
-                default => throw new LdapException(sprintf('Could not search in scope "%s".', $this->options['scope'])),
+                default => throw new LdapException(\sprintf('Could not search in scope "%s".', $this->options['scope'])),
             };
 
             $itemsLeft = $maxItems = $this->options['maxItems'];
@@ -101,13 +101,13 @@ public function execute(): CollectionInterface
                 if (false === $search) {
                     $ldapError = '';
                     if ($errno = ldap_errno($con)) {
-                        $ldapError = sprintf(' LDAP error was [%d] %s', $errno, ldap_error($con));
+                        $ldapError = \sprintf(' LDAP error was [%d] %s', $errno, ldap_error($con));
                     }
                     if ($pageControl) {
                         $this->resetPagination();
                     }
 
-                    throw new LdapException(sprintf('Could not complete search with dn "%s", query "%s" and filters "%s".%s.', $this->dn, $this->query, implode(',', $this->options['filter']), $ldapError), $errno);
+                    throw new LdapException(\sprintf('Could not complete search with dn "%s", query "%s" and filters "%s".%s.', $this->dn, $this->query, implode(',', $this->options['filter']), $ldapError), $errno);
                 }
 
                 $this->results[] = $search;

Adapter/ExtLdap/UpdateOperation.php

@@ -34,10 +34,10 @@ public function __construct(
         private ?array $values,
     ) {
         if (!\in_array($operationType, self::VALID_OPERATION_TYPES, true)) {
-            throw new UpdateOperationException(sprintf('"%s" is not a valid modification type.', $operationType));
+            throw new UpdateOperationException(\sprintf('"%s" is not a valid modification type.', $operationType));
         }
         if (\LDAP_MODIFY_BATCH_REMOVE_ALL === $operationType && null !== $values) {
-            throw new UpdateOperationException(sprintf('$values must be null for LDAP_MODIFY_BATCH_REMOVE_ALL operation, "%s" given.', get_debug_type($values)));
+            throw new UpdateOperationException(\sprintf('$values must be null for LDAP_MODIFY_BATCH_REMOVE_ALL operation, "%s" given.', get_debug_type($values)));
         }
     }
 

Ldap.php

@@ -56,7 +56,7 @@ public function escape(string $subject, string $ignore = '', int $flags = 0): st
     public static function create(string $adapter, array $config = []): static
     {
         if ('ext_ldap' !== $adapter) {
-            throw new DriverNotFoundException(sprintf('Adapter "%s" not found. Only "ext_ldap" is supported at the moment.', $adapter));
+            throw new DriverNotFoundException(\sprintf('Adapter "%s" not found. Only "ext_ldap" is supported at the moment.', $adapter));
         }
 
         return new self(new Adapter($config));

Security/CheckLdapCredentialsListener.php

@@ -48,7 +48,7 @@ public function onCheckPassport(CheckPassportEvent $event): void
         }
 
         if (!$passport->hasBadge(PasswordCredentials::class)) {
-            throw new \LogicException(sprintf('LDAP authentication requires a passport containing password credentials, authenticator "%s" does not fulfill these requirements.', $event->getAuthenticator()::class));
+            throw new \LogicException(\sprintf('LDAP authentication requires a passport containing password credentials, authenticator "%s" does not fulfill these requirements.', $event->getAuthenticator()::class));
         }
 
         /** @var PasswordCredentials $passwordCredentials */
@@ -58,7 +58,7 @@ public function onCheckPassport(CheckPassportEvent $event): void
         }
 
         if (!$this->ldapLocator->has($ldapBadge->getLdapServiceId())) {
-            throw new \LogicException(sprintf('Cannot check credentials using the "%s" ldap service, as such service is not found. Did you maybe forget to add the "ldap" service tag to this service?', $ldapBadge->getLdapServiceId()));
+            throw new \LogicException(\sprintf('Cannot check credentials using the "%s" ldap service, as such service is not found. Did you maybe forget to add the "ldap" service tag to this service?', $ldapBadge->getLdapServiceId()));
         }
 
         $presentedPassword = $passwordCredentials->getPassword();

Security/LdapAuthenticator.php

@@ -75,7 +75,7 @@ public function onAuthenticationFailure(Request $request, AuthenticationExceptio
     public function start(Request $request, ?AuthenticationException $authException = null): Response
     {
         if (!$this->authenticator instanceof AuthenticationEntryPointInterface) {
-            throw new NotAnEntryPointException(sprintf('Decorated authenticator "%s" does not implement interface "%s".', get_debug_type($this->authenticator), AuthenticationEntryPointInterface::class));
+            throw new NotAnEntryPointException(\sprintf('Decorated authenticator "%s" does not implement interface "%s".', get_debug_type($this->authenticator), AuthenticationEntryPointInterface::class));
         }
 
         return $this->authenticator->start($request, $authException);

Security/LdapUserProvider.php

@@ -72,7 +72,7 @@ public function loadUserByIdentifier(string $identifier): UserInterface
         $count = \count($entries);
 
         if (!$count) {
-            $e = new UserNotFoundException(sprintf('User "%s" not found.', $identifier));
+            $e = new UserNotFoundException(\sprintf('User "%s" not found.', $identifier));
             $e->setUserIdentifier($identifier);
 
             throw $e;
@@ -98,7 +98,7 @@ public function loadUserByIdentifier(string $identifier): UserInterface
     public function refreshUser(UserInterface $user): UserInterface
     {
         if (!$user instanceof LdapUser) {
-            throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', get_debug_type($user)));
+            throw new UnsupportedUserException(\sprintf('Instances of "%s" are not supported.', get_debug_type($user)));
         }
 
         return new LdapUser($user->getEntry(), $user->getUserIdentifier(), $user->getPassword(), $user->getRoles(), $user->getExtraFields());
@@ -110,7 +110,7 @@ public function refreshUser(UserInterface $user): UserInterface
     public function upgradePassword(PasswordAuthenticatedUserInterface $user, string $newHashedPassword): void
     {
         if (!$user instanceof LdapUser) {
-            throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', get_debug_type($user)));
+            throw new UnsupportedUserException(\sprintf('Instances of "%s" are not supported.', get_debug_type($user)));
         }
 
         if (null === $this->passwordAttribute) {
@@ -153,7 +153,7 @@ protected function loadUser(string $identifier, Entry $entry): UserInterface
     private function getAttributeValue(Entry $entry, string $attribute): mixed
     {
         if (!$entry->hasAttribute($attribute)) {
-            throw new InvalidArgumentException(sprintf('Missing attribute "%s" for user "%s".', $attribute, $entry->getDn()));
+            throw new InvalidArgumentException(\sprintf('Missing attribute "%s" for user "%s".', $attribute, $entry->getDn()));
         }
 
         $values = $entry->getAttribute($attribute);
@@ -162,7 +162,7 @@ private function getAttributeValue(Entry $entry, string $attribute): mixed
         }
 
         if (1 !== \count($values)) {
-            throw new InvalidArgumentException(sprintf('Attribute "%s" has multiple values.', $attribute));
+            throw new InvalidArgumentException(\sprintf('Attribute "%s" has multiple values.', $attribute));
         }
 
         return $values[0];