Updating interface method per suggestion - makes sense to me, Request is redundant

Author: weaverryan

Guard/Firewall/GuardAuthenticationListener.php

@@ -74,11 +74,11 @@ private function executeGuardAuthenticator($uniqueGuardKey, GuardAuthenticatorIn
         $request = $event->getRequest();
         try {
             if (null !== $this->logger) {
-                $this->logger->info('Calling getCredentialsFromRequest on guard configurator', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));
+                $this->logger->info('Calling getCredentials on guard configurator.', array('firewall_key' => $this->providerKey, 'authenticator' => get_class($guardAuthenticator)));
             }
 
             // allow the authenticator to fetch authentication info from the request
-            $credentials = $guardAuthenticator->getCredentialsFromRequest($request);
+            $credentials = $guardAuthenticator->getCredentials($request);
 
             // allow null to be returned to skip authentication
             if (null === $credentials) {

Guard/GuardAuthenticatorInterface.php

@@ -44,13 +44,13 @@ interface GuardAuthenticatorInterface extends AuthenticationEntryPointInterface
      *
      * @return mixed|null
      */
-    public function getCredentialsFromRequest(Request $request);
+    public function getCredentials(Request $request);
 
     /**
      * Return a UserInterface object based on the credentials OR throw
      * an AuthenticationException.
      *
-     * The *credentials* are the return value from getCredentialsFromRequest()
+     * The *credentials* are the return value from getCredentials()
      *
      * @param mixed                 $credentials
      * @param UserProviderInterface $userProvider

Guard/Tests/Firewall/GuardAuthenticationListenerTest.php

@@ -38,7 +38,7 @@ public function testHandleSuccess()
         $credentials = array('username' => 'weaverryan', 'password' => 'all_your_base');
         $authenticator
             ->expects($this->once())
-            ->method('getCredentialsFromRequest')
+            ->method('getCredentials')
             ->with($this->equalTo($this->request))
             ->will($this->returnValue($credentials));
 
@@ -87,7 +87,7 @@ public function testHandleSuccessWithRememberMe()
 
         $authenticator
             ->expects($this->once())
-            ->method('getCredentialsFromRequest')
+            ->method('getCredentials')
             ->with($this->equalTo($this->request))
             ->will($this->returnValue(array('username' => 'anything_not_empty')));
 
@@ -130,7 +130,7 @@ public function testHandleCatchesAuthenticationException()
         $authException = new AuthenticationException('Get outta here crazy user with a bad password!');
         $authenticator
             ->expects($this->once())
-            ->method('getCredentialsFromRequest')
+            ->method('getCredentials')
             ->will($this->throwException($authException));
 
         // this is not called
@@ -162,11 +162,11 @@ public function testReturnNullToSkipAuth()
 
         $authenticatorA
             ->expects($this->once())
-            ->method('getCredentialsFromRequest')
+            ->method('getCredentials')
             ->will($this->returnValue(null));
         $authenticatorB
             ->expects($this->once())
-            ->method('getCredentialsFromRequest')
+            ->method('getCredentials')
             ->will($this->returnValue(null));
 
         // this is not called