Adding periods at the end of exceptions, and changing one class name to LogicException thanks to @iltar
This commit is contained in:
parent
eb158cbdb3
commit
d6937218be
@ -162,7 +162,7 @@ class GuardAuthenticationListener implements ListenerInterface
|
|||||||
|
|
||||||
if (null === $this->rememberMeServices) {
|
if (null === $this->rememberMeServices) {
|
||||||
if (null !== $this->logger) {
|
if (null !== $this->logger) {
|
||||||
$this->logger->info('Remember me skipped: it is not configured for the firewall', array('authenticator' => get_class($guardAuthenticator)));
|
$this->logger->info('Remember me skipped: it is not configured for the firewall.', array('authenticator' => get_class($guardAuthenticator)));
|
||||||
}
|
}
|
||||||
|
|
||||||
return;
|
return;
|
||||||
|
@ -68,7 +68,7 @@ class GuardAuthenticatorHandler
|
|||||||
}
|
}
|
||||||
|
|
||||||
throw new \UnexpectedValueException(sprintf(
|
throw new \UnexpectedValueException(sprintf(
|
||||||
'The %s::onAuthenticationSuccess method must return null or a Response object. You returned %s',
|
'The %s::onAuthenticationSuccess method must return null or a Response object. You returned %s.',
|
||||||
get_class($guardAuthenticator),
|
get_class($guardAuthenticator),
|
||||||
is_object($response) ? get_class($response) : gettype($response)
|
is_object($response) ? get_class($response) : gettype($response)
|
||||||
));
|
));
|
||||||
@ -117,7 +117,7 @@ class GuardAuthenticatorHandler
|
|||||||
}
|
}
|
||||||
|
|
||||||
throw new \UnexpectedValueException(sprintf(
|
throw new \UnexpectedValueException(sprintf(
|
||||||
'The %s::onAuthenticationFailure method must return null or a Response object. You returned %s',
|
'The %s::onAuthenticationFailure method must return null or a Response object. You returned %s.',
|
||||||
get_class($guardAuthenticator),
|
get_class($guardAuthenticator),
|
||||||
is_object($response) ? get_class($response) : gettype($response)
|
is_object($response) ? get_class($response) : gettype($response)
|
||||||
));
|
));
|
||||||
|
@ -52,7 +52,7 @@ class GuardAuthenticationProvider implements AuthenticationProviderInterface
|
|||||||
public function authenticate(TokenInterface $token)
|
public function authenticate(TokenInterface $token)
|
||||||
{
|
{
|
||||||
if (!$this->supports($token)) {
|
if (!$this->supports($token)) {
|
||||||
throw new \InvalidArgumentException('GuardAuthenticationProvider only supports NonAuthenticatedGuardToken');
|
throw new \InvalidArgumentException('GuardAuthenticationProvider only supports GuardTokenInterface.');
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!$token instanceof PreAuthenticationGuardToken) {
|
if (!$token instanceof PreAuthenticationGuardToken) {
|
||||||
@ -87,7 +87,7 @@ class GuardAuthenticationProvider implements AuthenticationProviderInterface
|
|||||||
}
|
}
|
||||||
|
|
||||||
throw new \LogicException(sprintf(
|
throw new \LogicException(sprintf(
|
||||||
'The correct GuardAuthenticator could not be found for unique key "%s". The listener and provider should be passed the same list of authenticators!?',
|
'The correct GuardAuthenticator could not be found for unique key "%s". The listener and provider should be passed the same list of authenticators.',
|
||||||
$token->getGuardProviderKey()
|
$token->getGuardProviderKey()
|
||||||
));
|
));
|
||||||
}
|
}
|
||||||
@ -99,7 +99,7 @@ class GuardAuthenticationProvider implements AuthenticationProviderInterface
|
|||||||
|
|
||||||
if (!$user instanceof UserInterface) {
|
if (!$user instanceof UserInterface) {
|
||||||
throw new \UnexpectedValueException(sprintf(
|
throw new \UnexpectedValueException(sprintf(
|
||||||
'The %s::authenticate method must return a UserInterface. You returned %s',
|
'The %s::authenticate method must return a UserInterface. You returned %s.',
|
||||||
get_class($guardAuthenticator),
|
get_class($guardAuthenticator),
|
||||||
is_object($user) ? get_class($user) : gettype($user)
|
is_object($user) ? get_class($user) : gettype($user)
|
||||||
));
|
));
|
||||||
@ -113,7 +113,7 @@ class GuardAuthenticationProvider implements AuthenticationProviderInterface
|
|||||||
$authenticatedToken = $guardAuthenticator->createAuthenticatedToken($user, $this->providerKey);
|
$authenticatedToken = $guardAuthenticator->createAuthenticatedToken($user, $this->providerKey);
|
||||||
if (!$authenticatedToken instanceof TokenInterface) {
|
if (!$authenticatedToken instanceof TokenInterface) {
|
||||||
throw new \UnexpectedValueException(sprintf(
|
throw new \UnexpectedValueException(sprintf(
|
||||||
'The %s::createAuthenticatedToken method must return a TokenInterface. You returned %s',
|
'The %s::createAuthenticatedToken method must return a TokenInterface. You returned %s.',
|
||||||
get_class($guardAuthenticator),
|
get_class($guardAuthenticator),
|
||||||
is_object($authenticatedToken) ? get_class($authenticatedToken) : gettype($authenticatedToken)
|
is_object($authenticatedToken) ? get_class($authenticatedToken) : gettype($authenticatedToken)
|
||||||
));
|
));
|
||||||
|
@ -51,6 +51,6 @@ class PreAuthenticationGuardToken extends AbstractToken implements GuardTokenInt
|
|||||||
|
|
||||||
public function setAuthenticated($authenticated)
|
public function setAuthenticated($authenticated)
|
||||||
{
|
{
|
||||||
throw new \Exception('The PreAuthenticationGuardToken is *always* not authenticated');
|
throw new \LogicException('The PreAuthenticationGuardToken is *always* not authenticated.');
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user